On Fri, Jun 28, 2013 at 04:16:44PM +0900, Jingoo Han wrote:
> Add PHY provider node for the DP PHY.
> 
> Signed-off-by: Jingoo Han <jg1....@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5250.dtsi |   13 ++++++++-----
>  1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos5250.dtsi 
> b/arch/arm/boot/dts/exynos5250.dtsi
> index 41cd625..f7bac75 100644
> --- a/arch/arm/boot/dts/exynos5250.dtsi
> +++ b/arch/arm/boot/dts/exynos5250.dtsi
> @@ -614,6 +614,12 @@
>               interrupts = <0 94 0>;
>       };
>  
> +     dp_phy: video-phy@10040720 {
> +             compatible = "samsung,exynos5250-dp-video-phy";
> +             reg = <0x10040720 4>;
> +             #phy-cells = <0>;
> +     };
> +
>       dp-controller {
>               compatible = "samsung,exynos5-dp";
>               reg = <0x145b0000 0x1000>;
> @@ -623,11 +629,8 @@
>               clock-names = "dp";
>               #address-cells = <1>;
>               #size-cells = <0>;
> -
> -             dptx-phy {
> -                     reg = <0x10040720>;
> -                     samsung,enable-mask = <1>;
> -             };
> +             phys = <&dp_phy 0>;

phy-cells being 0, means that this would become:

                phys = <&dp_phy>;

> +             phy-names = "dp";

for the label, I would use something more descriptive such as
'display-port'.

other than that:

Acked-by: Felipe Balbi <ba...@ti.com>

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to