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

[Yakir Yang]

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@xxxxxxxxx> 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".
>