On 14/04/25 16:41, Tomi Valkeinen wrote: > The docs say about mode_valid(): > > "it is not allowed to look at anything else but the passed-in mode, and > validate it against configuration-invariant hardware constraints" > > We're doing a lot more than just looking at the mode. The main issue > here is that we're doing checks based on the pixel clock, before we know > what the pixel clock from the crtc actually is. > > So, drop the cdns_dsi_check_conf() call from .mode_valid(). > > Signed-off-by: Tomi Valkeinen <tomi.valkei...@ideasonboard.com> > --- > drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) >
Reviewed-by: Aradhya Bhatia <aradhya.bha...@linux.dev> -- Regards Aradhya
