Messages in this thread | | | Subject | Re: [PATCH v6 3/3] drm: bridge: cdns-mhdp: add j721e wrapper | From | Tomi Valkeinen <> | Date | Thu, 12 Mar 2020 09:01:23 +0200 |
| |
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
| |