On Fri, 12 Mar 2021, Rob Herring wrote: > On Fri, Mar 12, 2021 at 4:08 AM Lee Jones <lee.jo...@linaro.org> wrote: > > > > Fixes the following W=1 kernel build warning(s): > > > > drivers/of/fdt.c:478: warning: Function parameter or member 'node' not > > described in '__reserved_mem_reserve_reg' > > drivers/of/fdt.c:478: warning: Function parameter or member 'uname' not > > described in '__reserved_mem_reserve_reg' > > drivers/of/fdt.c:525: warning: Function parameter or member 'node' not > > described in '__reserved_mem_check_root' > > drivers/of/fdt.c:547: warning: Function parameter or member 'node' not > > described in '__fdt_scan_reserved_mem' > > drivers/of/fdt.c:547: warning: Function parameter or member 'uname' not > > described in '__fdt_scan_reserved_mem' > > drivers/of/fdt.c:547: warning: Function parameter or member 'depth' not > > described in '__fdt_scan_reserved_mem' > > drivers/of/fdt.c:547: warning: Function parameter or member 'data' not > > described in '__fdt_scan_reserved_mem' > > drivers/of/fdt.c:547: warning: expecting prototype for > > fdt_scan_reserved_mem(). Prototype was for __fdt_scan_reserved_mem() instead > > drivers/of/fdt.c:663: warning: Function parameter or member 'parent' not > > described in 'of_scan_flat_dt_subnodes' > > drivers/of/fdt.c:708: warning: Function parameter or member 'node' not > > described in 'of_get_flat_dt_prop' > > drivers/of/fdt.c:708: warning: Function parameter or member 'name' not > > described in 'of_get_flat_dt_prop' > > drivers/of/fdt.c:708: warning: Function parameter or member 'size' not > > described in 'of_get_flat_dt_prop' > > drivers/of/fdt.c:758: warning: Function parameter or member 'node' not > > described in 'of_flat_dt_match' > > drivers/of/fdt.c:758: warning: Function parameter or member 'compat' not > > described in 'of_flat_dt_match' > > drivers/of/fdt.c:778: warning: Function parameter or member 'node' not > > described in 'of_get_flat_dt_phandle' > > drivers/of/fdt.c:778: warning: expecting prototype for > > of_get_flat_dt_prop(). Prototype was for of_get_flat_dt_phandle() instead > > drivers/of/fdt.c:955: warning: Function parameter or member 'node' not > > described in 'early_init_dt_scan_root' > > drivers/of/fdt.c:955: warning: Function parameter or member 'uname' not > > described in 'early_init_dt_scan_root' > > drivers/of/fdt.c:955: warning: Function parameter or member 'depth' not > > described in 'early_init_dt_scan_root' > > drivers/of/fdt.c:955: warning: Function parameter or member 'data' not > > described in 'early_init_dt_scan_root' > > drivers/of/fdt.c:991: warning: Function parameter or member 'node' not > > described in 'early_init_dt_scan_memory' > > drivers/of/fdt.c:991: warning: Function parameter or member 'uname' not > > described in 'early_init_dt_scan_memory' > > drivers/of/fdt.c:991: warning: Function parameter or member 'depth' not > > described in 'early_init_dt_scan_memory' > > drivers/of/fdt.c:991: warning: Function parameter or member 'data' not > > described in 'early_init_dt_scan_memory' > > > > Cc: Rob Herring <robh...@kernel.org> > > Cc: Frank Rowand <frowand.l...@gmail.com> > > Cc: b...@kernel.crashing.org > > Cc: devicet...@vger.kernel.org > > Signed-off-by: Lee Jones <lee.jo...@linaro.org> > > --- > > drivers/of/fdt.c | 19 ++++++++++--------- > > 1 file changed, 10 insertions(+), 9 deletions(-) > > > > diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c > > index dcc1dd96911a9..1fb3348eb9516 100644 > > --- a/drivers/of/fdt.c > > +++ b/drivers/of/fdt.c > > @@ -470,7 +470,7 @@ void *initial_boot_params __ro_after_init; > > > > static u32 of_fdt_crc32; > > > > -/** > > +/* > > * __reserved_mem_reserve_reg() - reserve all memory described in 'reg' > > property > > */ > > static int __init __reserved_mem_reserve_reg(unsigned long node, > > @@ -516,7 +516,7 @@ static int __init __reserved_mem_reserve_reg(unsigned > > long node, > > return 0; > > } > > > > -/** > > +/* > > * __reserved_mem_check_root() - check if #size-cells, #address-cells > > provided > > * in /reserved-memory matches the values supported by the current > > implementation, > > * also check if ranges property has been provided > > @@ -539,7 +539,7 @@ static int __init __reserved_mem_check_root(unsigned > > long node) > > return 0; > > } > > > > -/** > > +/* > > * fdt_scan_reserved_mem() - scan a single FDT node for reserved memory > > This is still wrong. Should be __fdt_scan_reserved_mem.
Ah, the warning went away because I demoted it. Will fix. The others too. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
