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@xxxxxxxxxx>
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@xxxxxxxxxxx/#r
[2] https://lore.kernel.org/dri-devel/20241217160655.2371138-1-vignesh.raman@xxxxxxxxxxxxx/
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.
