[PATCH v2 01/34] drm: convert many bridge drivers from devm_kzalloc() to devm_drm_bridge_alloc() API

Manikandan.M at microchip.com Manikandan.M at microchip.com
Mon May 5 05:20:26 UTC 2025


On 30/04/25 4:06 pm, Luca Ceresoli wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Hello Manikandan,
> 
> On Wed, 30 Apr 2025 09:42:16 +0000
> <Manikandan.M at microchip.com> wrote:
> 
> [...]
> 
>>> diff --git a/drivers/gpu/drm/bridge/microchip-lvds.c b/drivers/gpu/drm/bridge/microchip-lvds.c
>>> index 1d4ae0097df847d9f93c79eecff0c4587ae331ba..9f4ff82bc6b49010f8727da3b367f5a744a28edc 100644
>>> --- a/drivers/gpu/drm/bridge/microchip-lvds.c
>>> +++ b/drivers/gpu/drm/bridge/microchip-lvds.c
>>> @@ -157,9 +157,10 @@ static int mchp_lvds_probe(struct platform_device *pdev)
>>>           if (!dev->of_node)
>>>                   return -ENODEV;
>>>
>>> -       lvds = devm_kzalloc(&pdev->dev, sizeof(*lvds), GFP_KERNEL);
>>> -       if (!lvds)
>>> -               return -ENOMEM;
>>> +       lvds = devm_drm_bridge_alloc(&pdev->dev, struct mchp_lvds, bridge,
>>> +                                    &mchp_lvds_bridge_funcs);
>>> +       if (IS_ERR(lvds))
>>> +               return PTR_ERR(lvds);
>>>
>>>           lvds->dev = dev;
>>>
>>> @@ -192,7 +193,6 @@ static int mchp_lvds_probe(struct platform_device *pdev)
>>>
>>>           lvds->bridge.of_node = dev->of_node;
>>>           lvds->bridge.type = DRM_MODE_CONNECTOR_LVDS;
>>> -       lvds->bridge.funcs = &mchp_lvds_bridge_funcs;
>>>
>>>           dev_set_drvdata(dev, lvds);
>>>           ret = devm_pm_runtime_enable(dev);
>>
>> Reviewed-by: Manikandan Muralidharan <manikandan.m at microchip.com>
> 
> Thanks for reviewing!
> 
> In v3 this patch will be slightly different from v2. See the reply I
> just sent to Doug for the details.
> 
> If your Reviewed-by tag refers only to the microchip-lvds driver, for
> which there will be no change in v3, I think it's correct to take your
> tag and add a comment like:
> 
>   Reviewed-by: Manikandan Muralidharan <manikandan.m at microchip.com> # microchip-lvds.c
> 
> Anything against this?
> 
No, Luca. Please proceed with the above mentioned tag # microchip-lvds.c
> Best regards,
> Luca
> 
> --
> Luca Ceresoli, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com

-- 
Thanks and Regards,
Manikandan M.



More information about the dri-devel mailing list