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 <[email protected]>
> ---
>  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 devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to