On Wed, 2025-02-26 at 21:27 +0530, Aradhya Bhatia wrote: > Move the bridge pre_enable call before crtc enable, and the bridge > post_disable call after the crtc disable. > > The sequence of enable after this patch will look like: > > bridge[n]_pre_enable > ... > bridge[1]_pre_enable > > crtc_enable > encoder_enable > > bridge[1]_enable > ... > bridge[n]_enable > > And, the disable sequence for the display pipeline will look like: > > bridge[n]_disable > ... > bridge[1]_disable > > encoder_disable > crtc_disable > > bridge[1]_post_disable > ... > bridge[n]_post_disable > > The definition of bridge pre_enable hook says that, > "The display pipe (i.e. clocks and timing signals) feeding this bridge > will not yet be running when this callback is called". > > Since CRTC is also a source feeding the bridge, it should not be enabled > before the bridges in the pipeline are pre_enabled. Fix that by > re-ordering the sequence of bridge pre_enable and bridge post_disable. > > Acked-by: Dmitry Baryshkov <[email protected]> > Reviewed-by: Tomi Valkeinen <[email protected]> > Tested-by: Tomi Valkeinen <[email protected]> > Signed-off-by: Aradhya Bhatia <[email protected]> > Signed-off-by: Aradhya Bhatia <[email protected]>
Tested with AM62 OLDI series [1] on AM625 with single channel AUO 800x480 LVDS panel. No issues with the patch! Tested-by: Alexander Sverdlin <[email protected]> [1] Link: https://lore.kernel.org/all/[email protected]/ > --- > drivers/gpu/drm/drm_atomic_helper.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) -- Alexander Sverdlin Siemens AG www.siemens.com
