Re: [PATCH] ARM: dts: am57xx-idk: tpic2810 is on I2C bus, not SPI

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

 



On 02/15/2017 09:12 AM, Sekhar Nori wrote:
> commit 50e95b6b854c ("ARM: dts: am57xx-idk: Add Industrial
> output support") added the TPIC2810 device-tree node under
> SPI bus instead of I2C1.
> 

Well that's odd, looks like git could get confused by the lack of
context around the node during rebases. Anyway,

Acked-by: Andrew F. Davis <afd@xxxxxx>

> Fix it. Tested on AM572x IDK by driving on-board LEDs
> connected to TPIC2810
> 
> Fixes: 50e95b6b854c ("ARM: dts: am57xx-idk: Add Industrial output support")
> Signed-off-by: Sekhar Nori <nsekhar@xxxxxx>
> ---
>  arch/arm/boot/dts/am57xx-idk-common.dtsi | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/am57xx-idk-common.dtsi b/arch/arm/boot/dts/am57xx-idk-common.dtsi
> index 814a720d5c3d..d0a55b845690 100644
> --- a/arch/arm/boot/dts/am57xx-idk-common.dtsi
> +++ b/arch/arm/boot/dts/am57xx-idk-common.dtsi
> @@ -311,6 +311,13 @@
>  			/* ID & VBUS GPIOs provided in board dts */
>  		};
>  	};
> +
> +	tpic2810: tpic2810@60 {
> +		compatible = "ti,tpic2810";
> +		reg = <0x60>;
> +		gpio-controller;
> +		#gpio-cells = <2>;
> +	};
>  };
>  
>  &mcspi3 {
> @@ -326,13 +333,6 @@
>  		spi-max-frequency = <1000000>;
>  		spi-cpol;
>  	};
> -
> -	tpic2810: tpic2810@60 {
> -		compatible = "ti,tpic2810";
> -		reg = <0x60>;
> -		gpio-controller;
> -		#gpio-cells = <2>;
> -	};
>  };
>  
>  &uart3 {
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux