[PATCH 6/7] media: allegro: simplify cleanup using __free
Sanjay Chitroda
sanjayembeddedse at gmail.com
Tue Mar 10 13:05:12 PDT 2026
From: Sanjay Chitroda <sanjayembeddedse at gmail.com>
Replace manual cleanup logic with __free attribute from cleanup.h. This
removes explicit kfree() calls and simplifies the error handling paths.
Remove unused variable e.g. ‘size’ after usage of cleanup API.
No functional change intended for kernel memory allocation.
Signed-off-by: Sanjay Chitroda <sanjayembeddedse at gmail.com>
---
.../media/platform/allegro-dvt/allegro-core.c | 95 +++++--------------
1 file changed, 24 insertions(+), 71 deletions(-)
diff --git a/drivers/media/platform/allegro-dvt/allegro-core.c b/drivers/media/platform/allegro-dvt/allegro-core.c
index eac3bc9af990..197a368fcb57 100644
--- a/drivers/media/platform/allegro-dvt/allegro-core.c
+++ b/drivers/media/platform/allegro-dvt/allegro-core.c
@@ -934,25 +934,20 @@ static int allegro_mbox_send(struct allegro_mbox *mbox, void *msg)
{
struct allegro_dev *dev = mbox->dev;
ssize_t size;
- int err;
- u32 *tmp;
+ int err = 0;
- tmp = kzalloc(mbox->size, GFP_KERNEL);
- if (!tmp) {
- err = -ENOMEM;
- goto out;
- }
+ u32 *tmp __free(kfree) = kzalloc(mbox->size, GFP_KERNEL);
+ if (!tmp)
+ return -ENOMEM;
size = allegro_encode_mail(tmp, msg);
err = allegro_mbox_write(mbox, tmp, size);
- kfree(tmp);
if (err)
- goto out;
+ return err;
allegro_mcu_interrupt(dev);
-out:
return err;
}
@@ -963,36 +958,29 @@ static int allegro_mbox_send(struct allegro_mbox *mbox, void *msg)
static int allegro_mbox_notify(struct allegro_mbox *mbox)
{
struct allegro_dev *dev = mbox->dev;
- union mcu_msg_response *msg;
- u32 *tmp;
- int err;
+ int err = 0;
- msg = kmalloc_obj(*msg);
+ union mcu_msg_response *msg __free(kfree) = kmalloc_obj(*msg);
if (!msg)
return -ENOMEM;
msg->header.version = dev->fw_info->mailbox_version;
- tmp = kmalloc(mbox->size, GFP_KERNEL);
+ u32 *tmp __free(kfree) = kmalloc(mbox->size, GFP_KERNEL);
if (!tmp) {
- err = -ENOMEM;
- goto out;
+ return -ENOMEM;
}
err = allegro_mbox_read(mbox, tmp, mbox->size);
if (err < 0)
- goto out;
+ return err;
err = allegro_decode_mail(msg, tmp);
if (err)
- goto out;
+ return err;
allegro_handle_message(dev, msg);
-out:
- kfree(tmp);
- kfree(msg);
-
return err;
}
@@ -1480,13 +1468,11 @@ static int allegro_mcu_push_buffer_internal(struct allegro_channel *channel,
enum mcu_msg_type type)
{
struct allegro_dev *dev = channel->dev;
- struct mcu_msg_push_buffers_internal *msg;
struct mcu_msg_push_buffers_internal_buffer *buffer;
unsigned int num_buffers = 0;
size_t size;
struct allegro_buffer *al_buffer;
struct list_head *list;
- int err;
switch (type) {
case MCU_MSG_TYPE_PUSH_BUFFER_REFERENCE:
@@ -1501,9 +1487,9 @@ static int allegro_mcu_push_buffer_internal(struct allegro_channel *channel,
list_for_each_entry(al_buffer, list, head)
num_buffers++;
- size = struct_size(msg, buffer, num_buffers);
+ size = struct_size((struct mcu_msg_push_buffers_internal *)NULL, buffer, num_buffers);
- msg = kmalloc(size, GFP_KERNEL);
+ struct mcu_msg_push_buffers_internal *msg __free(kfree) = kmalloc(size, GFP_KERNEL);
if (!msg)
return -ENOMEM;
@@ -1521,10 +1507,7 @@ static int allegro_mcu_push_buffer_internal(struct allegro_channel *channel,
buffer++;
}
- err = allegro_mbox_send(dev->mbox_command, msg);
-
- kfree(msg);
- return err;
+ return allegro_mbox_send(dev->mbox_command, msg);
}
static int allegro_mcu_push_buffer_intermediate(struct allegro_channel *channel)
@@ -1621,8 +1604,6 @@ static ssize_t allegro_h264_write_sps(struct allegro_channel *channel,
void *dest, size_t n)
{
struct allegro_dev *dev = channel->dev;
- struct nal_h264_sps *sps;
- ssize_t size;
unsigned int size_mb = SIZE_MACROBLOCK;
/* Calculation of crop units in Rec. ITU-T H.264 (04/2017) p. 76 */
unsigned int crop_unit_x = 2;
@@ -1632,7 +1613,7 @@ static ssize_t allegro_h264_write_sps(struct allegro_channel *channel,
unsigned int cpb_size;
unsigned int cpb_size_scale;
- sps = kzalloc_obj(*sps);
+ struct nal_h264_sps *sps __free(kfree) = kzalloc_obj(*sps);
if (!sps)
return -ENOMEM;
@@ -1715,21 +1696,15 @@ static ssize_t allegro_h264_write_sps(struct allegro_channel *channel,
sps->vui.pic_struct_present_flag = 1;
sps->vui.bitstream_restriction_flag = 0;
- size = nal_h264_write_sps(&dev->plat_dev->dev, dest, n, sps);
-
- kfree(sps);
-
- return size;
+ return nal_h264_write_sps(&dev->plat_dev->dev, dest, n, sps);
}
static ssize_t allegro_h264_write_pps(struct allegro_channel *channel,
void *dest, size_t n)
{
struct allegro_dev *dev = channel->dev;
- struct nal_h264_pps *pps;
- ssize_t size;
- pps = kzalloc_obj(*pps);
+ struct nal_h264_pps *pps __free(kfree) = kzalloc_obj(*pps);
if (!pps)
return -ENOMEM;
@@ -1752,11 +1727,7 @@ static ssize_t allegro_h264_write_pps(struct allegro_channel *channel,
pps->pic_scaling_matrix_present_flag = 0;
pps->second_chroma_qp_index_offset = 0;
- size = nal_h264_write_pps(&dev->plat_dev->dev, dest, n, pps);
-
- kfree(pps);
-
- return size;
+ return nal_h264_write_pps(&dev->plat_dev->dev, dest, n, pps);
}
static void allegro_channel_eos_event(struct allegro_channel *channel)
@@ -1772,15 +1743,13 @@ static ssize_t allegro_hevc_write_vps(struct allegro_channel *channel,
void *dest, size_t n)
{
struct allegro_dev *dev = channel->dev;
- struct nal_hevc_vps *vps;
struct nal_hevc_profile_tier_level *ptl;
- ssize_t size;
unsigned int num_ref_frames = channel->num_ref_idx_l0;
s32 profile = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_profile);
s32 level = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_level);
s32 tier = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_tier);
- vps = kzalloc_obj(*vps);
+ struct nal_hevc_vps *vps __free(kfree) = kzalloc_obj(*vps);
if (!vps)
return -ENOMEM;
@@ -1800,29 +1769,23 @@ static ssize_t allegro_hevc_write_vps(struct allegro_channel *channel,
vps->max_dec_pic_buffering_minus1[0] = num_ref_frames;
vps->max_num_reorder_pics[0] = num_ref_frames;
- size = nal_hevc_write_vps(&dev->plat_dev->dev, dest, n, vps);
-
- kfree(vps);
-
- return size;
+ return nal_hevc_write_vps(&dev->plat_dev->dev, dest, n, vps);
}
static ssize_t allegro_hevc_write_sps(struct allegro_channel *channel,
void *dest, size_t n)
{
struct allegro_dev *dev = channel->dev;
- struct nal_hevc_sps *sps;
struct nal_hevc_profile_tier_level *ptl;
struct nal_hevc_vui_parameters *vui;
struct nal_hevc_hrd_parameters *hrd;
- ssize_t size;
unsigned int cpb_size;
unsigned int num_ref_frames = channel->num_ref_idx_l0;
s32 profile = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_profile);
s32 level = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_level);
s32 tier = v4l2_ctrl_g_ctrl(channel->mpeg_video_hevc_tier);
- sps = kzalloc_obj(*sps);
+ struct nal_hevc_sps *sps __free(kfree) = kzalloc_obj(*sps);
if (!sps)
return -ENOMEM;
@@ -1913,11 +1876,7 @@ static ssize_t allegro_hevc_write_sps(struct allegro_channel *channel,
hrd->vcl_hrd[0].cbr_flag[0] = !v4l2_ctrl_g_ctrl(channel->mpeg_video_frame_rc_enable);
- size = nal_hevc_write_sps(&dev->plat_dev->dev, dest, n, sps);
-
- kfree(sps);
-
- return size;
+ return nal_hevc_write_sps(&dev->plat_dev->dev, dest, n, sps);
}
static ssize_t allegro_hevc_write_pps(struct allegro_channel *channel,
@@ -1925,11 +1884,9 @@ static ssize_t allegro_hevc_write_pps(struct allegro_channel *channel,
void *dest, size_t n)
{
struct allegro_dev *dev = channel->dev;
- struct nal_hevc_pps *pps;
- ssize_t size;
int i;
- pps = kzalloc_obj(*pps);
+ struct nal_hevc_pps *pps = kzalloc_obj(*pps);
if (!pps)
return -ENOMEM;
@@ -1960,11 +1917,7 @@ static ssize_t allegro_hevc_write_pps(struct allegro_channel *channel,
pps->lists_modification_present_flag = channel->enable_reordering;
- size = nal_hevc_write_pps(&dev->plat_dev->dev, dest, n, pps);
-
- kfree(pps);
-
- return size;
+ return nal_hevc_write_pps(&dev->plat_dev->dev, dest, n, pps);
}
static u64 allegro_put_buffer(struct allegro_channel *channel,
--
2.34.1
More information about the Linux-mediatek
mailing list