[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0e9b73b7-a960-e2a1-d8a1-f12309755176@ti.com>
Date: Thu, 12 Mar 2020 09:01:23 +0200
From: Tomi Valkeinen <tomi.valkeinen@...com>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Yuti Amonkar <yamonkar@...ence.com>
CC: <linux-kernel@...r.kernel.org>, <dri-devel@...ts.freedesktop.org>,
<devicetree@...r.kernel.org>, <robh+dt@...nel.org>,
<maxime@...no.tech>, <airlied@...ux.ie>, <daniel@...ll.ch>,
<mark.rutland@....com>, <a.hajda@...sung.com>,
<narmstrong@...libre.com>, <jonas@...boo.se>,
<jernej.skrabec@...l.net>, <praneeth@...com>, <jsarha@...com>,
<mparab@...ence.com>, <sjakhade@...ence.com>
Subject: Re: [PATCH v6 3/3] drm: bridge: cdns-mhdp: add j721e wrapper
Hi Laurent,
On 11/03/2020 22:52, Laurent Pinchart wrote:
>> +void cdns_mhdp_j721e_enable(struct cdns_mhdp_device *mhdp)
>> +{
>> + /*
>> + * Eneble VIF_0 and select DPI2 as its input. DSS0 DPI0 is connected
>> + * to eDP DPI2. This is the only supported SST configuration on
>> + * J721E.
>
> Without hardware documentation I can't really comment on this, but I'd
> like to make sure it doesn't imply that the MHDP has more than one input
> and one output.
You can download the TRM for j721e here:
http://www.ti.com/lit/pdf/spruil1
MHDP has one DP output, but 4 inputs to support MST and split/dual panel modes. None of those are
supported by the drivers, but perhaps some thought should be paid to figure out if adding these
features affect the DT bindings.
Tomi
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
Powered by blists - more mailing lists