On Fri, Jan 1, 2021 at 10:32 PM Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx> wrote: > > Hi Shawn, > > it seems like by doing so you're overwriting the I2C host pinctrl, so (unless that's intentional) the more correct fix would be: > > pinctrl-0 = <&qup_i2c3_default &i2c3_hid_active>; Ha, yes, I missed that there is already some default pinctrl state set up in sdm845.dtsi. I will fix it with v2. Thanks for spotting it, Konrad! Shawn
