On Tue, Nov 7, 2017 at 11:54 PM, Greentime Hu [off-list ref] wrote:

From: Greentime Hu <redacted>

Commit message needed.

Signed-off-by: Vincent Chen <redacted>
Signed-off-by: Greentime Hu <redacted>
---

+       /* 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.

+               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.