06:45 <mwalle> devarsh: downstream has vp_bus_type = INTERNAL, DPI for dss0 and vp_bus_type = INTERNAL, INTERNAL for dss1. With your patches it's INTERNAL, DPI for both. Is that correct?

07:17 <devarsh> mwalle: Yes, that should be fine from functionality w.r.t what is supported in downstream. The DSS2 ->VP2 which is marked as DPI actually feeds DPI output to DSI bridge and driver would still handle it correctly for DSI bridge as the check w.r.t vp_bus_type is only being used to set connector/encoder as DPI in case DSS port connects to DPI panel directly which is not the case here.

07:26 <mwalle> devarsh: but is that also future proof? i.e. you won't have a different compatible to distiguish it then

07:29 <devarsh> We think it should be fine since DSS controller reg space is same between AM62p and AM62x, the difference lies in only clocking and interfacing and that's what we are trying to decouple

07:34 <devarsh> I think we would probably need to decouple vp_bus_type as well in future. For AM62p, DSS2 VP1 can be used for either OLDI or DPI based on device-tree configuration, so ideally there can't be a fix vp_bus_type for this port.