On Mon, Jun 10, 2013 at 3:52 PM, Arnd Bergmann [off-list ref] wrote:

On Monday 10 June 2013 10:27:05 Srinivas KANDAGATLA wrote:

quoted

+     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 think that in my comments to the pinctrl driver I had some
similar comment, so it's probably a good idea to look into
this.

Yours,
Linus Walleij