linux-kernel - Re: [PATCH v2 5/8] drm/atmel-hlcdc: use devm_drm_of_get

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 for Android: free password hash cracker in your pocket

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

Message-ID: <c477520f-0ac9-48ee-9c84-21df015c5b79@microchip.com>
Date: Mon, 5 Jan 2026 06:20:52 +0000
From: <Manikandan.M@...rochip.com>
To: <Ludovic.Desroches@...rochip.com>, <Dharma.B@...rochip.com>,
	<maarten.lankhorst@...ux.intel.com>, <mripard@...nel.org>,
	<tzimmermann@...e.de>, <airlied@...il.com>, <simona@...ll.ch>,
	<Nicolas.Ferre@...rochip.com>, <alexandre.belloni@...tlin.com>,
	<claudiu.beznea@...on.dev>
CC: <dri-devel@...ts.freedesktop.org>, <linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 5/8] drm/atmel-hlcdc: use devm_drm_of_get_bridge()

On 18/12/25 6:56 pm, Ludovic Desroches wrote:
> Get rid of drm_of_find_panel_or_bridge() as it is deprecated and use
> devm_drm_of_get_bridge() instead.
> 
> Signed-off-by: Ludovic Desroches <ludovic.desroches@...rochip.com>
Reviewed-by: Manikandan Muralidharan <manikandan.m@...rochip.com>

> ---
>   drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 25 +++++-------------------
>   1 file changed, 5 insertions(+), 20 deletions(-)
> 
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> index e582315f70a119f2b39057ff112bc427117b85f5..e8aea905fb10d20d89c0cfd7937039b44d8a55b7 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> @@ -69,16 +69,14 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
>   {
>   	struct atmel_hlcdc_rgb_output *output;
>   	struct device_node *ep;
> -	struct drm_panel *panel;
>   	struct drm_bridge *bridge;
>   	struct atmel_hlcdc_dc *dc = dev->dev_private;
>   	struct drm_crtc *crtc = dc->crtc;
> -	int ret;
> +	int ret = 0;
>   
> -	ret = drm_of_find_panel_or_bridge(dev->dev->of_node, 0, endpoint,
> -					  &panel, &bridge);
> -	if (ret)
> -		return ret;
> +	bridge = devm_drm_of_get_bridge(dev->dev, dev->dev->of_node, 0, endpoint);
> +	if (IS_ERR(bridge))
> +		return PTR_ERR(bridge);
>   
>   	output = drmm_simple_encoder_alloc(dev, struct atmel_hlcdc_rgb_output,
>   					   encoder, DRM_MODE_ENCODER_NONE);
> @@ -97,23 +95,10 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
>   	}
>   
>   
> -
> -	if (panel) {
> -		bridge = drm_panel_bridge_add_typed(panel,
> -						    DRM_MODE_CONNECTOR_Unknown);
> -		if (IS_ERR(bridge))
> -			return PTR_ERR(bridge);
> -	}
>   	output->encoder.possible_crtcs = drm_crtc_mask(crtc);
>   
> -	if (bridge) {
> +	if (bridge)
>   		ret = drm_bridge_attach(&output->encoder, bridge, NULL, 0);
> -		if (!ret)
> -			return 0;
> -
> -		if (panel)
> -			drm_panel_bridge_remove(bridge);
> -	}
>   
>   	return ret;
>   }
> 

-- 
Thanks and Regards,
Manikandan M.