Hi Simon, On 14/01/2014 15:50, Simon Guinot wrote: > This patch updates the Armada 370/XP SATA node with the new compatible > string "marvell,armada-370-xp-sata". Usually we try to use the name of the first SoC instead of a family name. So here it should be "marvell,armada-370-sata" Thanks, Gregory > > Signed-off-by: Simon Guinot <simon.guinot@xxxxxxxxxxxx> > Cc: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxxxxxxxxx> > Cc: Jason Cooper <jason@xxxxxxxxxxxxxx> > Cc: Andrew Lunn <andrew@xxxxxxx> > Cc: Gregory Clement <gregory.clement@xxxxxxxxxxxxxxxxxx> > Cc: Sebastian Hesselbarth <sebastian.hesselbarth@xxxxxxxxx> > Cc: Lior Amsalem <alior@xxxxxxxxxxx> > Cc: stable@xxxxxxxxxxxxxxx > --- > arch/arm/boot/dts/armada-370-xp.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/armada-370-xp.dtsi b/arch/arm/boot/dts/armada-370-xp.dtsi > index 7f10f627ae5b..b5def2416800 100644 > --- a/arch/arm/boot/dts/armada-370-xp.dtsi > +++ b/arch/arm/boot/dts/armada-370-xp.dtsi > @@ -152,7 +152,7 @@ > }; > > sata@a0000 { > - compatible = "marvell,orion-sata"; > + compatible = "marvell,armada-370-xp-sata"; > reg = <0xa0000 0x5000>; > interrupts = <55>; > clocks = <&gateclk 15>, <&gateclk 30>; > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html