Re: [PATCH 3/4] drm/panel: ilitek-ili9882t: Set prepare_prev_first

On 8/9/26 21:58, Marek Vasut wrote:
> Sending DSI commands from the prepare() callback requires DSI link to be
> up at that point. For DSI hosts is guaranteed only if the panel driver
> sets the .prepare_prev_first flag. Set it to let these panels work with
> the DSI hosts which don't power on the link in their .mode_set callback.
> 
> Signed-off-by: Marek Vasut <[email protected]>
> ---
> Cc: Conor Dooley <[email protected]>
> Cc: Dave Stevenson <[email protected]>
> Cc: David Airlie <[email protected]>
> Cc: Jessica Zhang <[email protected]>
> Cc: Krzysztof Kozlowski <[email protected]>
> Cc: Maarten Lankhorst <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Neil Armstrong <[email protected]>
> Cc: Rob Herring <[email protected]>
> Cc: Simona Vetter <[email protected]>
> Cc: Thomas Zimmermann <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> ---
>   drivers/gpu/drm/panel/panel-ilitek-ili9882t.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c b/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c
> index 219d048cb883f..24e72ad0d29a1 100644
> --- a/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c
> +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c
> @@ -808,6 +808,7 @@ static int ili9882t_add(struct ili9882t *ili)
>   
>   	ili->base.funcs = &ili9882t_funcs;
>   	ili->base.dev = &ili->dsi->dev;
> +	ili->base.prepare_prev_first = true;
>   
>   	drm_panel_add(&ili->base);
>   

Reviewed-by: Neil Armstrong <[email protected]>

Thanks,
Neil