Re: [PATCH v3 1/3] drm/vc4: dpi: stop using deprecated drmm_of_get_bridge()

["Maíra Canal" <mcanal@igalia.com>]

On 13/10/26 12:23, Luca Ceresoli wrote:
> drmm_of_get_bridge() is deprecated, switch to
> of_drm_get_bridge_by_endpoint(). As of_drm_get_bridge_by_endpoint() returns
> a refcounted bridge, ensure it is put.
> 
> drm_panel.h becomes unused, drop the include.
> 
> Also remove a comment that is now obvious: now panels are just bridges.
> 
> Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>

Reviewed-by: Maíra Canal <mcanal@igalia.com>

Best regards,
- Maíra
> 
> ---
> 
> Changes in v3:
> - re-add and update comment
> ---
>   drivers/gpu/drm/vc4/vc4_dpi.c | 10 +++-------
>   1 file changed, 3 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/vc4/vc4_dpi.c b/drivers/gpu/drm/vc4/vc4_dpi.c
> index 53f36626a50d..feefefcf3ad7 100644
> --- a/drivers/gpu/drm/vc4/vc4_dpi.c
> +++ b/drivers/gpu/drm/vc4/vc4_dpi.c
> @@ -16,7 +16,6 @@
>   #include <drm/drm_drv.h>
>   #include <drm/drm_edid.h>
>   #include <drm/drm_of.h>
> -#include <drm/drm_panel.h>
>   #include <drm/drm_print.h>
>   #include <drm/drm_probe_helper.h>
>   #include <drm/drm_simple_kms_helper.h>
> @@ -279,16 +278,13 @@ static const struct of_device_id vc4_dpi_dt_match[] = {
>   	{}
>   };
>   
> -/* Sets up the next link in the display chain, whether it's a panel or
> - * a bridge.
> - */
> +/* Sets up the next link in the display chain */
>   static int vc4_dpi_init_bridge(struct vc4_dpi *dpi)
>   {
> -	struct drm_device *drm = dpi->encoder.base.dev;
>   	struct device *dev = &dpi->pdev->dev;
> -	struct drm_bridge *bridge;
>   
> -	bridge = drmm_of_get_bridge(drm, dev->of_node, 0, 0);
> +	struct drm_bridge *bridge __free(drm_bridge_put) =
> +		of_drm_get_bridge_by_endpoint(dev->of_node, 0, 0);
>   	if (IS_ERR(bridge)) {
>   		/* If nothing was connected in the DT, that's not an
>   		 * error.
>