A semicolon at the end of the macro means that the for loop has an empty body, and so TSEC/MDIO will not work with older device trees.
This fix only applies to 2.6.28; apparently, this code is gone for 2.6.29, according to Grant Likely! Signed-off-by: Johns Daniel <johns.dan...@gmail.com> --- --- linux-2.6.28.7/arch/powerpc/sysdev/fsl_soc.c.orig 2009-02-20 16:41:27.000000000 -0600 +++ linux-2.6.28.7/arch/powerpc/sysdev/fsl_soc.c 2009-03-10 15:56:47.000000000 -0500 @@ -257,7 +257,7 @@ gfar_mdio_of_init_one(np); /* try the deprecated version */ - for_each_compatible_node(np, "mdio", "gianfar"); + for_each_compatible_node(np, "mdio", "gianfar") gfar_mdio_of_init_one(np); return 0; --- _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev