| Message ID | 20211007195601.677474-1-anders.roxell@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: asm: setup.h: export common variables | expand |
On Thu, 7 Oct 2021 21:56:01 +0200, Anders Roxell wrote: > When building the kernel with sparse enabled 'C=1' the following > warnings can be seen: > > arch/arm64/kernel/setup.c:58:13: warning: symbol '__fdt_pointer' was not declared. Should it be static? > arch/arm64/kernel/setup.c:84:25: warning: symbol 'boot_args' was not declared. Should it be static? > > Rework so the variables are exported, since these two variable are > created and used in setup.c, also used in head.S. > > [...] Applied to arm64 (for-next/misc), thanks! [1/1] arm64: asm: setup.h: export common variables https://git.kernel.org/arm64/c/1dfde0892b32 Cheers,
diff --git a/arch/arm64/include/asm/setup.h b/arch/arm64/include/asm/setup.h index d3320618ed14..6437df661700 100644 --- a/arch/arm64/include/asm/setup.h +++ b/arch/arm64/include/asm/setup.h @@ -8,4 +8,10 @@ void *get_early_fdt_ptr(void); void early_fdt_map(u64 dt_phys); +/* + * These two variables are used in the head.S file. + */ +extern phys_addr_t __fdt_pointer __initdata; +extern u64 __cacheline_aligned boot_args[4]; + #endif
When building the kernel with sparse enabled 'C=1' the following warnings can be seen: arch/arm64/kernel/setup.c:58:13: warning: symbol '__fdt_pointer' was not declared. Should it be static? arch/arm64/kernel/setup.c:84:25: warning: symbol 'boot_args' was not declared. Should it be static? Rework so the variables are exported, since these two variable are created and used in setup.c, also used in head.S. Signed-off-by: Anders Roxell <anders.roxell@linaro.org> --- arch/arm64/include/asm/setup.h | 6 ++++++ 1 file changed, 6 insertions(+)