linux-kernel - Re: [PATCH] drm/panel-simple: fix connector type for DataImage SCF0700C48GGU18 panel

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: <bf36363c-9e28-402e-932d-7844a947c73f@linaro.org>
Date: Tue, 13 Jan 2026 09:17:08 +0100
From: Neil Armstrong <neil.armstrong@...aro.org>
To: Marek Vasut <marex@...ladev.com>, dri-devel@...ts.freedesktop.org
Cc: stable@...r.kernel.org, David Airlie <airlied@...il.com>,
 Jessica Zhang <jesszhan0024@...il.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Simona Vetter <simona@...ll.ch>,
 Thomas Zimmermann <tzimmermann@...e.de>, kernel@...electronics.com,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/panel-simple: fix connector type for DataImage
 SCF0700C48GGU18 panel

On 1/10/26 16:27, Marek Vasut wrote:
> The connector type for the DataImage SCF0700C48GGU18 panel is missing and
> devm_drm_panel_bridge_add() requires connector type to be set. This leads
> to a warning and a backtrace in the kernel log and panel does not work:
> "
> WARNING: CPU: 3 PID: 38 at drivers/gpu/drm/bridge/panel.c:379 devm_drm_of_get_bridge+0xac/0xb8
> "
> The warning is triggered by a check for valid connector type in
> devm_drm_panel_bridge_add(). If there is no valid connector type
> set for a panel, the warning is printed and panel is not added.
> Fill in the missing connector type to fix the warning and make
> the panel operational once again.
> 
> Cc: stable@...r.kernel.org
> Fixes: 97ceb1fb08b6 ("drm/panel: simple: Add support for DataImage SCF0700C48GGU18")
> Signed-off-by: Marek Vasut <marex@...ladev.com>
> ---
> Cc: David Airlie <airlied@...il.com>
> Cc: Jessica Zhang <jesszhan0024@...il.com>
> Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
> Cc: Maxime Ripard <mripard@...nel.org>
> Cc: Neil Armstrong <neil.armstrong@...aro.org>
> Cc: Simona Vetter <simona@...ll.ch>
> Cc: Thomas Zimmermann <tzimmermann@...e.de>
> Cc: dri-devel@...ts.freedesktop.org
> Cc: kernel@...electronics.com
> Cc: linux-kernel@...r.kernel.org
> ---
>   drivers/gpu/drm/panel/panel-simple.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 3acc9f3dac16a..e33ee2308e715 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -1900,6 +1900,7 @@ static const struct panel_desc dataimage_scf0700c48ggu18 = {
>   	},
>   	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
>   	.bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE,
> +	.connector_type = DRM_MODE_CONNECTOR_DPI,
>   };
>   
>   static const struct display_timing dlc_dlc0700yzg_1_timing = {

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

Thanks,
Neil