linux-kernel - Re: [PATCH v3 0/2] drm: Allow encoder modeset when connectors are changed

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]

Message-ID: <d9181abb-f23d-4f16-9254-6794a9c8ce21@quicinc.com>
Date: Tue, 7 Jan 2025 13:43:49 -0800
From: Abhinav Kumar <quic_abhinavk@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Maxime Ripard
	<mripard@...nel.org>
CC: Jessica Zhang <quic_jesszhan@...cinc.com>,
        Maarten Lankhorst
	<maarten.lankhorst@...ux.intel.com>,
        Thomas Zimmermann <tzimmermann@...e.de>,
        David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
        <dri-devel@...ts.freedesktop.org>, <linux-kernel@...r.kernel.org>,
        <robdclark@...il.com>
Subject: Re: [PATCH v3 0/2] drm: Allow encoder modeset when connectors are
 changed



On 12/19/2024 6:05 PM, Dmitry Baryshkov wrote:
> On Tue, Dec 17, 2024 at 04:02:21PM +0100, Maxime Ripard wrote:
>> On Mon, Dec 16, 2024 at 10:27:44AM -0800, Abhinav Kumar wrote:
>>>
>>>
>>> On 12/16/2024 3:06 AM, Maxime Ripard wrote:
>>>> On Wed, Dec 11, 2024 at 01:18:41PM -0800, Jessica Zhang wrote:
>>>>> Call encoder mode_set() when connectors are changed. This avoids issues
>>>>> for cases where the connectors are changed but CRTC mode is not.
>>>>
>>>> Looks great, thanks a lot for doing the tests :)
>>>>
>>>> Reviewed-by: Maxime Ripard <mripard@...nel.org>
>>>>
>>>> Maxime
>>>
>>> Thanks for your feedback.
>>>
>>> Can we get an ack to land this through msm tree as part of the series which
>>> needed it?
>>
>> If possible, I'd rather merge it through drm-misc. We merge a
>> significant number of patches affecting the framework there, so a
>> conflict would be less likely there.
> 
> I think it should be fine to merge this patchset + core part of the
> Jessica's CWB patches ([1]) through drm-misc, then merge drm-misc-next
> into msm-next. I'd ask for such a merge anyway, once Vignesh's IGT uprev
> [2] lands in drm-misc as I need it to finally land the patchset
> converting msm/hdmi to use the HDMI Connector framework.
> 
> [1] https://lore.kernel.org/dri-devel/20241216-concurrent-wb-v4-0-fe220297a7f0@quicinc.com/#r
> [2] https://lore.kernel.org/dri-devel/20241217160655.2371138-1-vignesh.raman@collabora.com/
> 

Ok sounds good, I am fine to land this through drm-misc as well.

But we dont need to wait for CWB or the HDMI connector framework. This 
is needed not just for CWB but also other features as well including the 
bug we had fixed earlier ( mentioned here : 
https://patchwork.freedesktop.org/patch/612740/#comment_1115262 )

So, if we can land this into drm-misc first it will unblock many other 
things not just CWB.