linux-kernel - Re: [PATCH v2 0/2] drm/bridge: tc358767: Fix DRM_BRIDGE_ATTACH_NO

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Mon, 17 Jun 2024 07:40:32 +0200
From: Jan Kiszka <jan.kiszka@...mens.com>
To: Marek Vasut <marex@...x.de>,
 Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
 Alexander Stein <alexander.stein@...tq-group.com>,
 Aradhya Bhatia <a-bhatia1@...com>, Andrzej Hajda <andrzej.hajda@...el.com>,
 Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>,
 Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
 Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>,
 Sam Ravnborg <sam@...nborg.org>, dri-devel@...ts.freedesktop.org
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] drm/bridge: tc358767: Fix
 DRM_BRIDGE_ATTACH_NO_CONNECTOR case

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