On 10/06/13 14:52, Arnd Bergmann wrote:

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

quoted

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 did try this suggestion, and it simplified driver too, I will do this
change in next version.

Thanks,
srini