Hi Fabio, On Fri, Jul 13, 2018 at 7:08 PM, Fabio Estevam <festevam@xxxxxxxxx> wrote: > Hi Peter, > > On Fri, Jul 13, 2018 at 2:11 PM, Peter Robinson <pbrobinson@xxxxxxxxx> wrote: > >> +&i2c4 { /* Onboard Motion sensors */ >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_i2c4>; >> + clock-frequency = <100000>; >> + status = "okay"; >> +}; > > Would it be possible to add i2c4 to the common dtsi file instead? > > I haven't checked the schematics, but if it is only the > imx6sx-udoo-neo-basic.dts that does not have the i2c4 node, then > maybe you could add i2c4 to the dtsi and disable it inside > imx6sx-udoo-neo-basic.dts? Enabling it to turn around and then disable it again seemed counter intuitive to me hence the way I did, the WiFi in done that way even though it's not on all boards and I would imagine it would cause errors on the devices where there isn't wifi although I don't have those model(s) to verify. Peter -- 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