ARM: mvebu: use improved armada spi device tree compatible name
diff mbox

Message ID 1410147029-30067-1-git-send-email-gerg@uclinux.org
State New, archived
Headers show

Commit Message

Greg Ungerer Sept. 8, 2014, 3:30 a.m. UTC
From: Greg Ungerer <gerg@uclinux.org>

Switch the Armada SoC SPI port device tree binding to use the new improved
armada-370-spi compatible name. This allows for a wider range of baud rates
to be used.

Signed-off-by: Greg Ungerer <gerg@uclinux.org>
Tested-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Reviewed-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
 arch/arm/boot/dts/armada-370-xp.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ezequiel Garcia Sept. 8, 2014, 1:40 p.m. UTC | #1
Sending to Jason (just in case)

On 08 Sep 01:30 PM, gerg@uclinux.org wrote:
> From: Greg Ungerer <gerg@uclinux.org>
> 
> Switch the Armada SoC SPI port device tree binding to use the new improved
> armada-370-spi compatible name. This allows for a wider range of baud rates
> to be used.
> 
> Signed-off-by: Greg Ungerer <gerg@uclinux.org>
> Tested-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> Reviewed-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/boot/dts/armada-370-xp.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/armada-370-xp.dtsi b/arch/arm/boot/dts/armada-370-xp.dtsi
> index 23227e0..83286ec 100644
> --- a/arch/arm/boot/dts/armada-370-xp.dtsi
> +++ b/arch/arm/boot/dts/armada-370-xp.dtsi
> @@ -110,7 +110,7 @@
>  			};
>  
>  			spi0: spi@10600 {
> -				compatible = "marvell,orion-spi";
> +				compatible = "marvell,armada-370-spi", "marvell,orion-spi";
>  				reg = <0x10600 0x28>;
>  				#address-cells = <1>;
>  				#size-cells = <0>;
> @@ -121,7 +121,7 @@
>  			};
>  
>  			spi1: spi@10680 {
> -				compatible = "marvell,orion-spi";
> +				compatible = "marvell,armada-370-spi", "marvell,orion-spi";
>  				reg = <0x10680 0x28>;
>  				#address-cells = <1>;
>  				#size-cells = <0>;
> -- 
> 1.9.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-spi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jason Cooper Sept. 9, 2014, 4 p.m. UTC | #2
On Mon, Sep 08, 2014 at 01:30:29PM +1000, gerg@uclinux.org wrote:
> From: Greg Ungerer <gerg@uclinux.org>
> 
> Switch the Armada SoC SPI port device tree binding to use the new improved
> armada-370-spi compatible name. This allows for a wider range of baud rates
> to be used.
> 
> Signed-off-by: Greg Ungerer <gerg@uclinux.org>
> Tested-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> Reviewed-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/boot/dts/armada-370-xp.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to mvebu/dt.

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Mark Brown Sept. 28, 2014, 11:43 a.m. UTC | #3
On Tue, Sep 09, 2014 at 12:00:43PM -0400, Jason Cooper wrote:
> On Mon, Sep 08, 2014 at 01:30:29PM +1000, gerg@uclinux.org wrote:

> > Switch the Armada SoC SPI port device tree binding to use the new improved
> > armada-370-spi compatible name. This allows for a wider range of baud rates
> > to be used.

> Applied to mvebu/dt.

Note that the corresponding driver change hasn't been applied (or sent
to me for review) so this will break users.
Greg Ungerer Sept. 28, 2014, 1:19 p.m. UTC | #4
Hi Mark,

On 28/09/14 21:43, Mark Brown wrote:
> On Tue, Sep 09, 2014 at 12:00:43PM -0400, Jason Cooper wrote:
>> On Mon, Sep 08, 2014 at 01:30:29PM +1000, gerg@uclinux.org wrote:
>>> Switch the Armada SoC SPI port device tree binding to use the new improved
>>> armada-370-spi compatible name. This allows for a wider range of baud rates
>>> to be used.
>> Applied to mvebu/dt.
> Note that the corresponding driver change hasn't been applied (or sent
> to me for review) so this will break users.

It still maintains the compatible "marvell,orion-spi" as well, so its
shouldn't break anyone with only that applied.

I will resend the spi-orion.c change.

Regards
Greg


--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch
diff mbox

diff --git a/arch/arm/boot/dts/armada-370-xp.dtsi b/arch/arm/boot/dts/armada-370-xp.dtsi
index 23227e0..83286ec 100644
--- a/arch/arm/boot/dts/armada-370-xp.dtsi
+++ b/arch/arm/boot/dts/armada-370-xp.dtsi
@@ -110,7 +110,7 @@ 
 			};
 
 			spi0: spi@10600 {
-				compatible = "marvell,orion-spi";
+				compatible = "marvell,armada-370-spi", "marvell,orion-spi";
 				reg = <0x10600 0x28>;
 				#address-cells = <1>;
 				#size-cells = <0>;
@@ -121,7 +121,7 @@ 
 			};
 
 			spi1: spi@10680 {
-				compatible = "marvell,orion-spi";
+				compatible = "marvell,armada-370-spi", "marvell,orion-spi";
 				reg = <0x10680 0x28>;
 				#address-cells = <1>;
 				#size-cells = <0>;