| Message ID | 20230626143626.106584-1-alexghiti@rivosinc.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v4,1/3] Documentation: arm: Add bootargs to the table of added DT parameters | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | warning | Series does not have a cover letter |
| conchuod/tree_selection | success | Guessed tree name to be for-next at HEAD 488833ccdcac |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 6 and now 6 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 20 this patch: 20 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 14 lines checked |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On Mon, 26 Jun 2023 07:36:24 PDT (-0700), alexghiti@rivosinc.com wrote: > The bootargs node is also added by the EFI stub in the function > update_fdt(), so add it to the table. > > Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com> > Reviewed-by: Atish Patra <atishp@rivosinc.com> > Reviewed-by: Song Shuai <songshuaishuai@tinylab.org> > --- > Documentation/arm/uefi.rst | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/arm/uefi.rst b/Documentation/arm/uefi.rst > index baebe688a006..2b7ad9bd7cd2 100644 > --- a/Documentation/arm/uefi.rst > +++ b/Documentation/arm/uefi.rst > @@ -50,7 +50,7 @@ The stub populates the FDT /chosen node with (and the kernel scans for) the > following parameters: > > ========================== ====== =========================================== > -Name Size Description > +Name Type Description > ========================== ====== =========================================== > linux,uefi-system-table 64-bit Physical address of the UEFI System Table. > > @@ -67,4 +67,6 @@ linux,uefi-mmap-desc-ver 32-bit Version of the mmap descriptor format. > > kaslr-seed 64-bit Entropy used to randomize the kernel image > base address location. > + > +bootargs String Kernel command line > ========================== ====== =========================================== I remember having said this somewhere, but it's not on lore for this version. This touches Arm stuff, so I'd prefer an Ack from someone before touching it. Also: it looks like this missed e790a4ce5290 ("arm: docs: Move Arm documentation to Documentation/arch/").
Alexandre Ghiti <alexghiti@rivosinc.com> writes: > The bootargs node is also added by the EFI stub in the function > update_fdt(), so add it to the table. > > Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com> > Reviewed-by: Atish Patra <atishp@rivosinc.com> > Reviewed-by: Song Shuai <songshuaishuai@tinylab.org> > --- > Documentation/arm/uefi.rst | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) So I'm not sure if I'm supposed to be applying these patches, or whether they will go through the arch trees? In the former case, they don't apply to docs-next, so could I get a respin please? Thanks, jon
On 21/07/2023 23:07, Jonathan Corbet wrote: > Alexandre Ghiti <alexghiti@rivosinc.com> writes: > >> The bootargs node is also added by the EFI stub in the function >> update_fdt(), so add it to the table. >> >> Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com> >> Reviewed-by: Atish Patra <atishp@rivosinc.com> >> Reviewed-by: Song Shuai <songshuaishuai@tinylab.org> >> --- >> Documentation/arm/uefi.rst | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) > So I'm not sure if I'm supposed to be applying these patches, or whether > they will go through the arch trees? > > In the former case, they don't apply to docs-next, so could I get a > respin please? Yes sure, I'll do that now. BTW all the arch documentation was moved in Documentation/arch/, but not the riscv, are you working on this or should we? Thanks, Alex > > Thanks, > > jon > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv
Alexandre Ghiti <alex@ghiti.fr> writes: >> In the former case, they don't apply to docs-next, so could I get a >> respin please? > > Yes sure, I'll do that now. BTW all the arch documentation was moved in > Documentation/arch/, but not the riscv, are you working on this or > should we? I'll get there - unless somebody beats me to it :) Thanks, jon
diff --git a/Documentation/arm/uefi.rst b/Documentation/arm/uefi.rst index baebe688a006..2b7ad9bd7cd2 100644 --- a/Documentation/arm/uefi.rst +++ b/Documentation/arm/uefi.rst @@ -50,7 +50,7 @@ The stub populates the FDT /chosen node with (and the kernel scans for) the following parameters: ========================== ====== =========================================== -Name Size Description +Name Type Description ========================== ====== =========================================== linux,uefi-system-table 64-bit Physical address of the UEFI System Table. @@ -67,4 +67,6 @@ linux,uefi-mmap-desc-ver 32-bit Version of the mmap descriptor format. kaslr-seed 64-bit Entropy used to randomize the kernel image base address location. + +bootargs String Kernel command line ========================== ====== ===========================================