of/fdt: Remove PPC32 longtrail hack in memory scan
authorMichael Ellerman <mpe@ellerman.id.au>
Fri, 27 Jul 2018 05:35:55 +0000 (15:35 +1000)
committerRob Herring <robh@kernel.org>
Tue, 7 Aug 2018 18:05:54 +0000 (12:05 -0600)
When the OF code was originally made common by Grant in commit
51975db0b733 ("of/flattree: merge early_init_dt_scan_memory() common
code") (Feb 2010), the common code inherited a hack to handle
PPC "longtrail" machines, which had a "memory@0" node with no
device_type.

That check was then made to only apply to PPC32 in b44aa25d20e2 ("of:
Handle memory@0 node on PPC32 only") (May 2014).

But according to Paul Mackerras the "longtrail" machines are long
dead, if they were ever seen in the wild at all. If someone does still
have one, we can handle this firmware wart in powerpc platform code.

So remove the hack once and for all.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Rob Herring <robh@kernel.org>
drivers/of/fdt.c

index 6da20b9688f7419aae4f806798eaa137c8c4f649..800ad252cf9c635935f1a0ab75b14ae0b16bc003 100644 (file)
@@ -1034,14 +1034,7 @@ int __init early_init_dt_scan_memory(unsigned long node, const char *uname,
        bool hotpluggable;
 
        /* We are scanning "memory" nodes only */
-       if (type == NULL) {
-               /*
-                * The longtrail doesn't have a device_type on the
-                * /memory node, so look for the node called /memory@0.
-                */
-               if (!IS_ENABLED(CONFIG_PPC32) || depth != 1 || strcmp(uname, "memory@0") != 0)
-                       return 0;
-       } else if (strcmp(type, "memory") != 0)
+       if (type == NULL || strcmp(type, "memory") != 0)
                return 0;
 
        reg = of_get_flat_dt_prop(node, "linux,usable-memory", &l);