On Fri, May 9, 2025 at 9:56 AM Luca Ceresoli <luca.ceres...@bootlin.com>
wrote:
> This is the new API for allocating DRM bridges.
>
> Signed-off-by: Luca Ceresoli <luca.ceres...@bootlin.com>
> ---
> drivers/gpu/drm/bridge/nxp-ptn3460.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/nxp-ptn3460.c
> b/drivers/gpu/drm/bridge/nxp-ptn3460.c
> index
> 25d7c415478b14ef634bff4185a8dd8e866be0c6..7acb11f16dc19e87a84cc765b1cebef158662c00
> 100644
> --- a/drivers/gpu/drm/bridge/nxp-ptn3460.c
> +++ b/drivers/gpu/drm/bridge/nxp-ptn3460.c
> @@ -261,10 +261,10 @@ static int ptn3460_probe(struct i2c_client *client)
> struct drm_bridge *panel_bridge;
> int ret;
>
> - ptn_bridge = devm_kzalloc(dev, sizeof(*ptn_bridge), GFP_KERNEL);
> - if (!ptn_bridge) {
> - return -ENOMEM;
> - }
> + ptn_bridge = devm_drm_bridge_alloc(dev, struct ptn3460_bridge,
> bridge,
> + &ptn3460_bridge_funcs);
> + if (IS_ERR(ptn_bridge))
> + return PTR_ERR(ptn_bridge);
>
> panel_bridge = devm_drm_of_get_bridge(dev, dev->of_node, 0, 0);
> if (IS_ERR(panel_bridge))
> @@ -300,7 +300,6 @@ static int ptn3460_probe(struct i2c_client *client)
> return ret;
> }
>
> - ptn_bridge->bridge.funcs = &ptn3460_bridge_funcs;
> ptn_bridge->bridge.ops = DRM_BRIDGE_OP_EDID;
> ptn_bridge->bridge.type = DRM_MODE_CONNECTOR_LVDS;
> ptn_bridge->bridge.of_node = dev->of_node;
>
>
Looks good to me!
Reviewed-by: Anusha Srivatsa <asriv...@redhat.com>
> --
> 2.49.0
>
>
