On 27/05/2019 11:35, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > Two of the Tegra I2C controllers share pads with the DPAUX controllers. > In order for the I2C controllers to use these pads, they have to be set > into I2C mode. Use the I2C and off pin control states defined in the DT > nodes for DPAUX as "default" and "idle" states, respectively. This > ensures that the I2C controller driver can properly configure the pins > when it needs to perform I2C transactions. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > arch/arm64/boot/dts/nvidia/tegra186.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186.dtsi b/arch/arm64/boot/dts/nvidia/tegra186.dtsi > index 426ac0bdf6a6..00bd4ce80fa3 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra186.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra186.dtsi > @@ -173,6 +173,9 @@ > clock-names = "div-clk"; > resets = <&bpmp TEGRA186_RESET_I2C4>; > reset-names = "i2c"; > + pinctrl-names = "default", "idle"; > + pinctrl-0 = <&state_dpaux1_i2c>; > + pinctrl-1 = <&state_dpaux1_off>; > status = "disabled"; > }; > > @@ -201,6 +204,9 @@ > clock-names = "div-clk"; > resets = <&bpmp TEGRA186_RESET_I2C6>; > reset-names = "i2c"; > + pinctrl-names = "default", "idle"; > + pinctrl-0 = <&state_dpaux_i2c>; > + pinctrl-1 = <&state_dpaux_off>; > status = "disabled"; > }; Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx> Cheers Jon -- nvpublic