[PATCH v3 09/17] drm/bridge: cdns-dsi: Drop checks that shouldn't be in .mode_valid()
Aradhya Bhatia
aradhya.bhatia at linux.dev
Tue Apr 15 13:18:53 UTC 2025
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.valkeinen at ideasonboard.com>
> ---
> drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
Reviewed-by: Aradhya Bhatia <aradhya.bhatia at linux.dev>
--
Regards
Aradhya
More information about the dri-devel
mailing list