[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251225184401.414021-1-alperyasinak1@gmail.com>
Date: Thu, 25 Dec 2025 21:44:01 +0300
From: Alper Ak <alperyasinak1@...il.com>
To: andy.yan@...k-chips.com
Cc: Alper Ak <alperyasinak1@...il.com>,
Andrzej Hajda <andrzej.hajda@...el.com>,
Neil Armstrong <neil.armstrong@...aro.org>,
Robert Foss <rfoss@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Jonas Karlman <jonas@...boo.se>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>,
Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>,
dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org
Subject: [PATCH] drm/bridge: synopsys: dw-dp: fix signedness bug in dw_dp_video_need_vsc_sdp()
Since dw_dp_video_need_vsc_sdp() returns bool, have it return false
instead of -EINVAL if state is NULL.
Fixes: 86eecc3a9c2e ("drm/bridge: synopsys: Add DW DPTX Controller support library")
Signed-off-by: Alper Ak <alperyasinak1@...il.com>
---
drivers/gpu/drm/bridge/synopsys/dw-dp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/synopsys/dw-dp.c b/drivers/gpu/drm/bridge/synopsys/dw-dp.c
index 82aaf74e1bc0..36382956bbe0 100644
--- a/drivers/gpu/drm/bridge/synopsys/dw-dp.c
+++ b/drivers/gpu/drm/bridge/synopsys/dw-dp.c
@@ -1149,7 +1149,7 @@ static bool dw_dp_video_need_vsc_sdp(struct dw_dp *dp)
state = dw_dp_get_bridge_state(dp);
if (!state)
- return -EINVAL;
+ return false;
if (!link->vsc_sdp_supported)
return false;
--
2.43.0
Powered by blists - more mailing lists