Hi, On Wednesday 04 December 2013 10:46 PM, Andrew Lunn wrote: > Add nodes for the two SATA PHYs on kirkewood. > Add node for the one SATA PHY on Dove. > Add pHandles to the PHYs in the sata nodes. > > Signed-off-by: Andrew Lunn <andrew@xxxxxxx> > --- > arch/arm/boot/dts/dove.dtsi | 11 +++++++++++ > arch/arm/boot/dts/kirkwood-6281.dtsi | 2 ++ > arch/arm/boot/dts/kirkwood-6282.dtsi | 2 ++ > arch/arm/boot/dts/kirkwood.dtsi | 18 ++++++++++++++++++ > 4 files changed, 33 insertions(+) > > diff --git a/arch/arm/boot/dts/dove.dtsi b/arch/arm/boot/dts/dove.dtsi > index 113a8bc7bee7..d56b3c675249 100644 > --- a/arch/arm/boot/dts/dove.dtsi > +++ b/arch/arm/boot/dts/dove.dtsi > @@ -490,10 +490,21 @@ > reg = <0xa0000 0x2400>; > interrupts = <62>; > clocks = <&gate_clk 3>; > + phys = <&sata_phy0>; > + phy-names = "0"; A more descriptive phy-names would be good here and below. > nr-ports = <1>; > status = "disabled"; > }; > > + sata_phy0: sata_phy@0 { > + compatible = "marvell,mvebu-sata-phy"; > + reg = <0xa2000 0x0334>; > + clocks = <&gate_clk 3>; > + clock-names = "sata"; > + #phy-cells = <0>; Ah.. only you documentation was having this value as '1'. Only that needs to be fixed then. Thanks Kishon -- 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