Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support

From	Yakir Yang <ykk@rock-chips.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support
Date	2016-05-25 08:00 +0200
Message-ID	<rCA1k-5Cj-5@gated-at.bofh.it> (permalink)
References	<rCcLn-7ot-13@gated-at.bofh.it> <rChBo-1Ye-25@gated-at.bofh.it> <rCp5T-6JY-13@gated-at.bofh.it> <rCpfz-6NU-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 05/25/2016 02:23 AM, Heiko Stuebner wrote:
> Am Dienstag, 24. Mai 2016, 11:12:20 schrieb Doug Anderson:
>> Hi,
>>
>> On Tue, May 24, 2016 at 3:17 AM, Heiko Stuebner <heiko@sntech.de> wrote:
>>>> --- a/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
>>>> +++ b/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
>>>>
>>>> @@ -5,6 +5,7 @@ Required properties for dp-controller:
>>>>                platform specific such as:
>>>>                 * "samsung,exynos5-dp"
>>>>                 * "rockchip,rk3288-dp"
>>>>
>>>> +              * "rockchip,rk3399-edp"
>>> the cleanlines-freak in my likes to know if there is a difference
>>> between
>>> the rk3399 being called -edp here and -dp on the rk3288 :-)
>>>
>>> [...]
>> If I was a guessing man (which I'm not, but if I was), I'd guess that
>> this is because on rk3288 there was only one DP port and it was an EDP
>> port.  ...but since there was only one people just referred to it as
>> the "DP" port and that was codified in the bindings.  On rk3399 there
>> are two ports: one EDP and one DP.  All of a sudden we need to
>> differentiate.
>>
>> Any better suggestions for how to deal with that?
> nope - everything as it should be in that regard then - the rk3288-dp is
> permanent now anyway.
>
> But it looks like I'm sort of blind blind and only now have seen the
> separate DP controller chapter in the TRM (probably because I somehow didn't
> expect a second controller).
RK3399 eDP controller have cut off the DP-Audio function and DP-HDCP 
function, so it can't be treated as DP interfaces any more, that's why I 
call it "eDP".
>
>

Thread

[PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support Yakir Yang <ykk@rock-chips.com> - 2016-05-24 07:10 +0200
  [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support Yakir Yang <ykk@rock-chips.com> - 2016-05-24 09:00 +0200
    Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support Heiko Stuebner <heiko@sntech.de> - 2016-05-24 12:20 +0200
      Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support Doug Anderson <dianders@chromium.org> - 2016-05-24 20:20 +0200
        Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP support Heiko Stuebner <heiko@sntech.de> - 2016-05-24 20:30 +0200
          Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP  support Yakir Yang <ykk@rock-chips.com> - 2016-05-25 08:00 +0200
      Re: [PATCH v2 05/10] drm/rockchip: analogix_dp: add rk3399 eDP  support Yakir Yang <ykk@rock-chips.com> - 2016-05-25 08:00 +0200

csiph-web