On Sat, Mar 08, 2014 at 07:37:58AM +0000, Grant Likely wrote: > I believe we already talked about the above test. This function must > only process devices created by of_platform_populate(). Merely checking > the of_node pointer is not a sufficient test because there are other > paths in the kernel for creating platform_devices that might get a node > pointer attached to it. There definitely are: there are a number of platform device drivers which create a new platform device structure, and copy the of_node pointer across. However, that's really sick code, and can result in the platform device driver probing the device it just created... this practise should be outlawed. -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html