* [PATCH] drm/bridge: lt9611: Put DSI nodes if device tree parsing fails
@ 2026-11-03 6:47 Wentao Liang
2026-11-03 7:11 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Wentao Liang @ 2026-11-03 6:47 UTC (permalink / raw)
To: andrzej.hajda, neil.armstrong, rfoss
Cc: Laurent.pinchart, jonas, jernej.skrabec, luca.ceresoli,
dri-devel, linux-kernel, Wentao Liang, stable
lt9611_parse_dt() obtains references to the remote DSI nodes before
looking up the downstream bridge. If that lookup fails, for example
with -EPROBE_DEFER, lt9611_probe() returns without releasing the DSI
node references.
Use the existing err_of_put cleanup path when parsing fails. Both node
pointers are zero-initialized, so this also handles a missing DSI node.
Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge")
Cc: stable@vger.kernel.org
Assisted-by: DeepSeek-V4.1-Flash and OpenAI Codex
Signed-off-by: Wentao Liang <vulab@iscas.ac.cn>
---
drivers/gpu/drm/bridge/lontium-lt9611.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c
index 2d1a28261037..3c3e91e19fbd 100644
--- a/drivers/gpu/drm/bridge/lontium-lt9611.c
+++ b/drivers/gpu/drm/bridge/lontium-lt9611.c
@@ -1116,7 +1116,7 @@ static int lt9611_probe(struct i2c_client *client)
ret = lt9611_parse_dt(dev, lt9611);
if (ret) {
dev_err(dev, "failed to parse device tree\n");
- return ret;
+ goto err_of_put;
}
ret = lt9611_gpio_init(lt9611);
--
2.34.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] drm/bridge: lt9611: Put DSI nodes if device tree parsing fails
2026-11-03 6:47 [PATCH] drm/bridge: lt9611: Put DSI nodes if device tree parsing fails Wentao Liang
@ 2026-11-03 7:11 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2026-11-03 7:11 UTC (permalink / raw)
To: Wentao Liang, andrzej.hajda, neil.armstrong, rfoss
Cc: Laurent.pinchart, jonas, jernej.skrabec, luca.ceresoli,
dri-devel, linux-kernel, stable
On 03/11/2026 07:47, Wentao Liang wrote:
> lt9611_parse_dt() obtains references to the remote DSI nodes before
> looking up the downstream bridge. If that lookup fails, for example
> with -EPROBE_DEFER, lt9611_probe() returns without releasing the DSI
> node references.
>
> Use the existing err_of_put cleanup path when parsing fails. Both node
> pointers are zero-initialized, so this also handles a missing DSI node.
>
> Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge")
> Cc: stable@vger.kernel.org
> Assisted-by: DeepSeek-V4.1-Flash and OpenAI Codex
> Signed-off-by: Wentao Liang <vulab@iscas.ac.cn>
Another fake identity of "vulab" doing exactly the same as other cases,
repeating same mistake of running unsupervised crappy bot even though
was asked multiple times to fix that.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-11-03 7:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-03 6:47 [PATCH] drm/bridge: lt9611: Put DSI nodes if device tree parsing fails Wentao Liang
2026-11-03 7:11 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®