Hi Mukesh On Sat, Jul 04, 2026 at 01:08:34AM +0530, Mukesh Ojha wrote: > Use the devres-managed devm_of_reserved_mem_device_init() instead of > the manual of_reserved_mem_device_init()/of_reserved_mem_device_release() > pair, letting the device resource manager handle cleanup automatically. > > Signed-off-by: Mukesh Ojha <[email protected]>
Reviewed-by: Jacopo Mondi <[email protected]> I'll collect this single patch for the Mali-C55 PR for v7.4 > --- > drivers/media/platform/arm/mali-c55/mali-c55-core.c | 12 +++--------- > 1 file changed, 3 insertions(+), 9 deletions(-) > > diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-core.c > b/drivers/media/platform/arm/mali-c55/mali-c55-core.c > index 94a389b3f833..afbec11f69f8 100644 > --- a/drivers/media/platform/arm/mali-c55/mali-c55-core.c > +++ b/drivers/media/platform/arm/mali-c55/mali-c55-core.c > @@ -801,14 +801,12 @@ static int mali_c55_probe(struct platform_device *pdev) > if (ret) > return dev_err_probe(dev, ret, "failed to acquire resets\n"); > > - of_reserved_mem_device_init(dev); > + devm_of_reserved_mem_device_init(dev); > vb2_dma_contig_set_max_seg_size(dev, UINT_MAX); > > ret = __mali_c55_power_on(mali_c55); > - if (ret) { > - dev_err_probe(dev, ret, "failed to power on\n"); > - goto err_release_mem; > - } > + if (ret) > + return dev_err_probe(dev, ret, "failed to power on\n"); > > ret = mali_c55_check_hwcfg(mali_c55); > if (ret) > @@ -847,9 +845,6 @@ static int mali_c55_probe(struct platform_device *pdev) > kfree(mali_c55->context.registers); > err_power_off: > __mali_c55_power_off(mali_c55); > -err_release_mem: > - of_reserved_mem_device_release(dev); > - > return ret; > } > > @@ -864,7 +859,6 @@ static void mali_c55_remove(struct platform_device *pdev) > } > pm_runtime_disable(&pdev->dev); > kfree(mali_c55->context.registers); > - of_reserved_mem_device_release(&pdev->dev); > } > > static const struct of_device_id mali_c55_of_match[] = { > -- > 2.53.0 >
