Message ID | 20190530103927.20952-1-yamada.masahiro@socionext.com |
---|---|
State | New |
Headers | show |
Series | of/fdt: pass early_init_dt_reserve_memory_arch() with bool type nomap | expand |
On 5/30/19 3:39 AM, Masahiro Yamada wrote: > The third argument 'nomap' of early_init_dt_reserve_memory_arch() is > bool. It is preferred to pass it with a bool type parameter. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > drivers/of/fdt.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c > index de893c9616a1..b165e8b3a347 100644 > --- a/drivers/of/fdt.c > +++ b/drivers/of/fdt.c > @@ -551,7 +551,8 @@ static int __init __reserved_mem_reserve_reg(unsigned long node, > phys_addr_t base, size; > int len; > const __be32 *prop; > - int nomap, first = 1; > + int first = 1; > + bool nomap; > > prop = of_get_flat_dt_prop(node, "reg", &len); > if (!prop) > @@ -666,7 +667,7 @@ void __init early_init_fdt_scan_reserved_mem(void) > fdt_get_mem_rsv(initial_boot_params, n, &base, &size); > if (!size) > break; > - early_init_dt_reserve_memory_arch(base, size, 0); > + early_init_dt_reserve_memory_arch(base, size, false); > } > > of_scan_flat_dt(__fdt_scan_reserved_mem, NULL); > @@ -684,7 +685,7 @@ void __init early_init_fdt_reserve_self(void) > /* Reserve the dtb region */ > early_init_dt_reserve_memory_arch(__pa(initial_boot_params), > fdt_totalsize(initial_boot_params), > - 0); > + false); > } > > /** > Reviewed-by: Frank Rowand <frank.rowand@sony.com>
On Thu, 30 May 2019 19:39:27 +0900, Masahiro Yamada wrote: > The third argument 'nomap' of early_init_dt_reserve_memory_arch() is > bool. It is preferred to pass it with a bool type parameter. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > drivers/of/fdt.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > Applied, thanks. Rob
diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c index de893c9616a1..b165e8b3a347 100644 --- a/drivers/of/fdt.c +++ b/drivers/of/fdt.c @@ -551,7 +551,8 @@ static int __init __reserved_mem_reserve_reg(unsigned long node, phys_addr_t base, size; int len; const __be32 *prop; - int nomap, first = 1; + int first = 1; + bool nomap; prop = of_get_flat_dt_prop(node, "reg", &len); if (!prop) @@ -666,7 +667,7 @@ void __init early_init_fdt_scan_reserved_mem(void) fdt_get_mem_rsv(initial_boot_params, n, &base, &size); if (!size) break; - early_init_dt_reserve_memory_arch(base, size, 0); + early_init_dt_reserve_memory_arch(base, size, false); } of_scan_flat_dt(__fdt_scan_reserved_mem, NULL); @@ -684,7 +685,7 @@ void __init early_init_fdt_reserve_self(void) /* Reserve the dtb region */ early_init_dt_reserve_memory_arch(__pa(initial_boot_params), fdt_totalsize(initial_boot_params), - 0); + false); } /**
The third argument 'nomap' of early_init_dt_reserve_memory_arch() is bool. It is preferred to pass it with a bool type parameter. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- drivers/of/fdt.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) -- 2.17.1