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/ -- With best wishes Dmitry
