On Wed, Jun 22, 2022 at 04:36:35PM +0200, Konrad Dybcio wrote: > > > On 22.06.2022 15:43, Johan Hovold wrote: > > On Wed, Jun 22, 2022 at 02:33:02PM +0200, Konrad Dybcio wrote: > >> On 22.06.2022 06:12, Bjorn Andersson wrote: > > > >>> +&qup2_i2c5 { > >>> + clock-frequency = <400000>; > >>> + > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&qup2_i2c5_default>, <&kybd_default>, <&tpad_default>; > >>> + > >>> + status = "okay"; > >>> + > >> I think all device DTs generally have 'status = "okay"' at the beginning. Should we change that? > >> > > > > No, quite the opposite, status go at the end. > Then all other device DTs should be updated, as in dts/qcom/ > everybody keeps it first in non-SoC/PMIC files. Seems like a lot of churn so maybe not worth it. Johan