On 23/02/17 17:30, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > The P3310 processor modules use seven I2C controllers for various > peripherals. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 40 +++++++++++++++++++++++++- > 1 file changed, 39 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > index 3653d0dd5f4f..762f4756346c 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > @@ -6,6 +6,14 @@ > > aliases { > serial0 = &uarta; > + i2c0 = "/bpmp/i2c"; > + i2c1 = "/i2c@3160000"; > + i2c2 = "/i2c@c240000"; > + i2c3 = "/i2c@3180000"; > + i2c4 = "/i2c@3190000"; > + i2c5 = "/i2c@31c0000"; > + i2c6 = "/i2c@c250000"; > + i2c7 = "/i2c@31e0000"; Does it matter that these aliases do not align with the TRM numbering? I know that if we did it would create holes but I was not sure if that would be clearer? > }; > > chosen { > @@ -22,10 +30,38 @@ > status = "okay"; > }; > > + i2c@3160000 { > + status = "okay"; > + }; > + > + i2c@3180000 { > + status = "okay"; > + }; > + > + i2c@3190000 { > + status = "okay"; > + }; > + > + i2c@31c0000 { > + status = "okay"; > + }; > + > + i2c@31e0000 { > + status = "okay"; > + }; > + > hsp@3c00000 { > status = "okay"; > }; > > + i2c@c240000 { > + status = "okay"; > + }; > + > + i2c@c250000 { > + status = "okay"; > + }; > + > pmc@c360000 { > nvidia,invert-interrupt; > }; > @@ -57,7 +93,9 @@ > }; > > bpmp { > - status = "okay"; > + i2c { > + status = "okay"; > + }; > }; > > psci { > Otherwise ... Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx> Cheers Jon -- nvpublic -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html