On 16.02.24 15:57, Marek Vasut wrote: > On 2/16/24 10:10, Tomi Valkeinen wrote: >> On 15/02/2024 11:03, Alexander Stein wrote: >>> Hi everyone, >>> >>> Am Donnerstag, 15. Februar 2024, 09:53:54 CET schrieb Jan Kiszka: >>>> On 11.12.23 09:07, Aradhya Bhatia wrote: >>>>> On 06/12/23 17:41, Tomi Valkeinen wrote: >>>>>> Hi, >>>>>> >>>>>> On 08/11/2023 14:45, Alexander Stein wrote: >>>>>>> Hi Tomi, >>>>>>> >>>>>>> Am Mittwoch, 8. November 2023, 12:27:21 CET schrieb Tomi Valkeinen: >>>>>>>> These two patches are needed to make tc358767 work in the >>>>>>>> DRM_BRIDGE_ATTACH_NO_CONNECTOR case, at least when using a DP >>>>>>>> connector. >>>>>>>> >>>>>>>> I have tested this with TI AM654 EVM with a tc358767 add-on card >>>>>>>> connected to a DP monitor. >>>>>>> >>>>>>> Just a question regarding the usage of this DSI-DP bridge. >>>>>>> What is the state of the DSI lanes after the DSI host has been >>>>>>> initialized, >>>>>>> but before calling atomic_pre_enable? AFAIK this bridge requires >>>>>>> LP-11 >>>>>>> on DSI >>>>>>> at any time for accessing the AUX channel. >>>>> >>>>> + Marek >>>>> >>>>> Marek, Alexander, >>>>> >>>>> A quick grep tells me that you have added devicetree for tc358767 >>>>> in DSI >>>>> to (e)DP mode on other platforms. Could you please test these patches >>>>> and report if you find any issue? >>> >>> Sorry, I can't provide any feedback here. I've yet to setup the DSI-DP >>> correctly. >> >> Ok. Does anyone have a worry that these patches make the situation >> worse for the DSI case than it was before? Afaics, if the DSI lanes >> are not set up early enough by the DSI host, the driver would break >> with and without these patches. >> >> These do fix the driver for DRM_BRIDGE_ATTACH_NO_CONNECTOR and DPI, so >> I'd like to merge these unless these cause a regression with the DSI >> case. > > 1/2 looks good to me, go ahead and apply . My local patches still apply on top of 6.10-rc4, so I don't think this ever happened. What's still holding up this long-pending fix (at least for our devices)? Jan -- Siemens AG, Technology Linux Expert Center