linux-kernel - Re: [PATCH v2 1/2] drm/mipi-dsi: Add mipi_dsi_dual

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4bdeab38-d884-4864-8654-c6a70ef06288@linaro.org>
Date: Fri, 7 Mar 2025 11:03:30 +0100
From: neil.armstrong@...aro.org
To: Tejas Vipin <tejasvipin76@...il.com>, maarten.lankhorst@...ux.intel.com,
 mripard@...nel.org, tzimmermann@...e.de, airlied@...il.com, simona@...ll.ch,
 lujianhua000@...il.com
Cc: quic_jesszhan@...cinc.com, dianders@...omium.org,
 dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
 asrivats@...hat.com
Subject: Re: [PATCH v2 1/2] drm/mipi-dsi: Add mipi_dsi_dual_msleep

On 07/03/2025 10:15, Tejas Vipin wrote:
> Introduces mipi_dsi_dual_msleep to make it easier to use msleep in the
> case of a double-DSI panel.
> 
> Signed-off-by: Tejas Vipin <tejasvipin76@...il.com>
> ---
>   include/drm/drm_mipi_dsi.h | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/include/drm/drm_mipi_dsi.h b/include/drm/drm_mipi_dsi.h
> index bd40a443385c..d9bc0d79dd82 100644
> --- a/include/drm/drm_mipi_dsi.h
> +++ b/include/drm/drm_mipi_dsi.h
> @@ -306,6 +306,12 @@ ssize_t mipi_dsi_generic_read(struct mipi_dsi_device *dsi, const void *params,
>   			usleep_range(min, max);	\
>   	} while (0)
>   
> +#define mipi_dsi_dual_msleep(ctx1, ctx2, delay)				\
> +	do {								\
> +		if ((!(ctx1)->accum_err) || (!(ctx2)->accum_err))	\
> +			msleep(delay);					\
> +	} while (0)
> +
>   /**
>    * enum mipi_dsi_dcs_tear_mode - Tearing Effect Output Line mode
>    * @MIPI_DSI_DCS_TEAR_MODE_VBLANK: the TE output line consists of V-Blanking

Reviewed-by: Neil Armstrong <neil.armstrong@...aro.org>