On 10/06/13 14:52, Arnd Bergmann wrote: > On Monday 10 June 2013 10:27:05 Srinivas KANDAGATLA wrote: > >> > + soc { >> > + pin-controller-sbc { >> > + #address-cells = <1>; >> > + #size-cells = <1>; >> > + compatible = "st,stih416-pinctrl", "simple-bus"; > Why is this both its own device with a compatible string and a > "simple-bus" at the same time? Wouldn't it be simpler to just > scan the child device nodes from the "st,stih416-pinctrl" > driver instead of having a separate platform_driver for them? > I did try this suggestion, and it simplified driver too, I will do this change in next version. Thanks, srini -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html