Re: [PATCH 14/19] powerpc: mpc83xx: Switch to of_platform_populate

From: "J. Neuschäfer" <redacted>

Quoting from drivers/of/platform.c:

quoted

of_platform_populate() - [...]
Similar to of_platform_bus_probe(), this function walks the device
tree and creates devices from nodes.  It differs in that it follows
the modern convention of requiring all device nodes to have a
'compatible' property, and it is suitable for creating devices which
are children of the root node (of_platform_bus_probe will only create
children of the root which are selected by the @matches argument).

This is useful for new board ports because it means that the C code does
not have to anticipate every node that is placed directly under the root.

As a consequence, the of_bus_ids list can be much shorter, and I've
trimmed it to the necessary parts:

 - device-type = "soc" and compatible = "simple-bus" for the SoC bus
 - compatible = "gianfar" for the Ethernet controller (TSEC), which
   may contain an MDIO bus, which needs to be probed, as a subnode

Signed-off-by: J. Neuschäfer <redacted>
---
 arch/powerpc/platforms/83xx/misc.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/arch/powerpc/platforms/83xx/misc.c b/arch/powerpc/platforms/83xx/misc.c
index 1135c1ab923cc120f377a0d98767fef686cad1fe..bf522ee007bbb1429233355f668fc8563d8ca4e2 100644
--- a/arch/powerpc/platforms/83xx/misc.c
+++ b/arch/powerpc/platforms/83xx/misc.c

@@ -94,18 +94,14 @@ void __init mpc83xx_ipic_init_IRQ(void)

 static const struct of_device_id of_bus_ids[] __initconst = {
        { .type = "soc", },

-       { .compatible = "soc", },
        { .compatible = "simple-bus" },
        { .compatible = "gianfar" },
-       { .compatible = "gpio-leds", },
-       { .type = "qe", },
-       { .compatible = "fsl,qe", },