diff mbox

ARM: dts: armada-385-linksys: fix DSA compatible property

Message ID 20170209152702.23036-1-ralph.sennhauser@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ralph Sennhauser Feb. 9, 2017, 3:27 p.m. UTC
The switch to the new DSA binding used "marvell,mv88e6095" for the
compatible property which doesn't exist, use "marvell,mv88e6085"
instead.

Fixes: 455b82f03f52 ("ARM: dts: armada-385-linksys: Utilize new DSA binding")
Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>
---
 arch/arm/boot/dts/armada-385-linksys.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Feb. 9, 2017, 3:51 p.m. UTC | #1
On Thu, Feb 09, 2017 at 04:27:02PM +0100, Ralph Sennhauser wrote:
> The switch to the new DSA binding used "marvell,mv88e6095" for the
> compatible property which doesn't exist, use "marvell,mv88e6085"
> instead.
> 
> Fixes: 455b82f03f52 ("ARM: dts: armada-385-linksys: Utilize new DSA binding")
> Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>

Hi Ralph

Good catch.

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

Gregory, this is in the mvebu dt branch and need fixing there.

[Goes away to look at the other dts files]

O.K. Florian only made this typ0 once in the patchset.

     Andrew
Gregory CLEMENT Feb. 15, 2017, 2:56 p.m. UTC | #2
Hi Ralph and Andrew,
 
 On jeu., févr. 09 2017, Andrew Lunn <andrew@lunn.ch> wrote:

> On Thu, Feb 09, 2017 at 04:27:02PM +0100, Ralph Sennhauser wrote:
>> The switch to the new DSA binding used "marvell,mv88e6095" for the
>> compatible property which doesn't exist, use "marvell,mv88e6085"
>> instead.
>> 
>> Fixes: 455b82f03f52 ("ARM: dts: armada-385-linksys: Utilize new DSA binding")
>> Signed-off-by: Ralph Sennhauser <ralph.sennhauser@gmail.com>
>
> Hi Ralph
>
> Good catch.
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
>
> Gregory, this is in the mvebu dt branch and need fixing there.


Applied on mvebu/dt if it can't go through arm-soc I will submitted as a
fix for v4.11-rc1.

Thanks,

Gregory


>
> [Goes away to look at the other dts files]
>
> O.K. Florian only made this typ0 once in the patchset.
>
>      Andrew
diff mbox

Patch

diff --git a/arch/arm/boot/dts/armada-385-linksys.dtsi b/arch/arm/boot/dts/armada-385-linksys.dtsi
index 20d5e8b..df47bf1 100644
--- a/arch/arm/boot/dts/armada-385-linksys.dtsi
+++ b/arch/arm/boot/dts/armada-385-linksys.dtsi
@@ -107,7 +107,7 @@ 
 				status = "okay";
 
 				switch@0 {
-					compatible = "marvell,mv88e6095";
+					compatible = "marvell,mv88e6085";
 					#address-cells = <1>;
 					#size-cells = <0>;
 					reg = <0>;