[PATCH v2] media: imx-jpeg: convert kzalloc_obj() to devm_kzalloc()

From: Sanjay Chitroda

Date: Sun Mar 08 2026 - 01:06:11 EST


From: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>

The driver allcoates memory using kzalloc_obj() and frees it in the relase
path. since the allocated memory is tied to the lifetime of the device,
devm_kzalloc() can be used instead.

Using device-managed allocation simplifies the error handling paths and
remove the need for manual cleanup.

No functional change intended.

Signed-off-by: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>
---
Changes in v2:
- Rebase on tip of latest tip
- Updated commit message accordingly.
- Link to v1 https://lore.kernel.org/all/20260307210404.1428894-1-sanjayembedded@xxxxxxxxx/
---
drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c | 11 +++--------
1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
index b442dcba02e7..bd4b5f08a85c 100644
--- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
+++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
@@ -2200,14 +2200,12 @@ static int mxc_jpeg_open(struct file *file)
struct mxc_jpeg_ctx *ctx;
int ret = 0;

- ctx = kzalloc_obj(*ctx);
+ ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
if (!ctx)
return -ENOMEM;

- if (mutex_lock_interruptible(&mxc_jpeg->lock)) {
- ret = -ERESTARTSYS;
- goto free;
- }
+ if (mutex_lock_interruptible(&mxc_jpeg->lock))
+ return -ERESTARTSYS;

v4l2_fh_init(&ctx->fh, mxc_vfd);
v4l2_fh_add(&ctx->fh, file);
@@ -2246,8 +2244,6 @@ static int mxc_jpeg_open(struct file *file)
v4l2_fh_del(&ctx->fh, file);
v4l2_fh_exit(&ctx->fh);
mutex_unlock(&mxc_jpeg->lock);
-free:
- kfree(ctx);
return ret;
}

@@ -2754,7 +2750,6 @@ static int mxc_jpeg_release(struct file *file)
v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
v4l2_fh_del(&ctx->fh, file);
v4l2_fh_exit(&ctx->fh);
- kfree(ctx);
mutex_unlock(&mxc_jpeg->lock);

return 0;
--
2.34.1