Messages in this thread | | | From | Greentime Hu <> | Date | Thu, 9 Nov 2017 17:49:24 +0800 | Subject | Re: [PATCH 02/31] nds32: Kernel booting and initialization |
| |
2017-11-08 21:38 GMT+08:00 Rob Herring <robh+dt@kernel.org>: > On Tue, Nov 7, 2017 at 11:54 PM, Greentime Hu <green.hu@gmail.com> wrote: >> From: Greentime Hu <greentime@andestech.com> >> > > Commit message needed.
Thanks. I will add more commit messages in the next version patch.
> >> Signed-off-by: Vincent Chen <vincentc@andestech.com> >> Signed-off-by: Greentime Hu <greentime@andestech.com> >> --- > >> + /* it could update max_pfn */ >> + if (max_pfn - ram_start_pfn <= MAXMEM_PFN) >> + max_low_pfn = max_pfn; >> + else { >> + max_low_pfn = MAXMEM_PFN + ram_start_pfn; >> +#ifndef CONFIG_HIGHMEM > > Use IS_ENABLED here and other ifdef's if possible. >
Thanks I will try to use IS_ENABLED here in the next version patch.
>> + max_pfn = MAXMEM_PFN + ram_start_pfn; >> +#endif >> + } > > >> +static int __init nds32_device_probe(void) >> +{ >> + return of_platform_populate(NULL, NULL, NULL, NULL); >> +} >> + >> +device_initcall(nds32_device_probe); > > You can drop this. The core code will call of_platform_populate for you.
Thanks. I will drop this in the next version patch.
| |