Re: [PATCH 04/11] arm64: tegra: Enable I2C controllers on P3310

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux