Re: [PATCH 05/11] drm/bridge: panel: add a panel_bridge to every panel

"Luca Ceresoli" <[email protected]>
Newsgroups dev.linux.lists.sashiko-reviews,org.freedesktop.lists.dri-devel
Message-ID <[email protected]>
On Fri Aug 14, 2026 at 4:25 PM CEST, sashiko-bot wrote:

[...]

>> diff --git a/drivers/gpu/drm/drm_panel.c b/drivers/gpu/drm/drm_panel.c
>> index f8f6082e637fd..95fc95d1b4b7a 100644
>> --- a/drivers/gpu/drm/drm_panel.c
>> +++ b/drivers/gpu/drm/drm_panel.c
>> @@ -453,6 +453,9 @@ void *__devm_drm_panel_alloc(struct device *dev, size_t size, size_t offset,
>>
>>  	drm_panel_init(panel, dev, funcs, connector_type);
>>
>> +	panel->bridge = devm_drm_panel_bridge_add(dev, panel);
>> +	WARN_ON(!panel->bridge);
>
> [Severity: Medium]
> Will this check actually catch failures? devm_drm_panel_bridge_add()
> returns an ERR_PTR on failure, rather than NULL.

Indeed, will fix in v2.

Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.