drivers/iio/common/ssp_sensors/ssp_spi.c | 9 +- .../iio/common/st_sensors/st_sensors_core.c | 7 +- .../media/platform/allegro-dvt/allegro-core.c | 95 +++++-------------- .../platform/chips-media/coda/coda-bit.c | 4 +- .../platform/chips-media/coda/coda-jpeg.c | 39 ++++---- .../mediatek/vcodec/common/mtk_vcodec_dbgfs.c | 3 +- drivers/staging/greybus/camera.c | 27 ++---- drivers/staging/greybus/loopback.c | 35 +++----- drivers/staging/greybus/raw.c | 6 +- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 13 +-- drivers/staging/rtl8723bs/hal/sdio_ops.c | 37 ++------ 11 files changed, 78 insertions(+), 197 deletions(-)
From: Sanjay Chitroda <sanjayembeddedse@gmail.com> Hi all, This patch series replaces manual cleanup and explicit kfree() calls with the __free attribute from <linux/cleanup.h>. This modernizes the memory management style and simplifies common error paths without altering any functional behavior. The __free attribute provides automatic scope-based cleanup, making resource management clearer and reducing the chances of missing cleanup on early returns. No functional changes are intended in this series. Testing: - Compiled with W=1 - Build-tested on i86_64 Based on: <linux-v7.0-rc2> Feel free to share your valuable input in context of the cleanup API. Thanks, Sanjay Chitroda Sanjay Chitroda (7): staging: greybus: simplify cleanup using __free iio: ssp_sensors: simplify cleanup using __free iio: st_sensors: simplify cleanup using __free media: mediatek: vcodec: simplify cleanup using __free media: chips-media: coda: simplify cleanup using __free media: allegro: simplify cleanup using __free staging: rtl8723bs: simplify cleanup using __free drivers/iio/common/ssp_sensors/ssp_spi.c | 9 +- .../iio/common/st_sensors/st_sensors_core.c | 7 +- .../media/platform/allegro-dvt/allegro-core.c | 95 +++++-------------- .../platform/chips-media/coda/coda-bit.c | 4 +- .../platform/chips-media/coda/coda-jpeg.c | 39 ++++---- .../mediatek/vcodec/common/mtk_vcodec_dbgfs.c | 3 +- drivers/staging/greybus/camera.c | 27 ++---- drivers/staging/greybus/loopback.c | 35 +++----- drivers/staging/greybus/raw.c | 6 +- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 13 +-- drivers/staging/rtl8723bs/hal/sdio_ops.c | 37 ++------ 11 files changed, 78 insertions(+), 197 deletions(-) -- 2.34.1
Dear Sanjay, Would you mind explaining why you put so many people into TO and CC fields when sending patch series. You should stick to maintainers and reviewers from maintainers file as others can still see it on the mailing list(that's why you CC kernel mailing.list related emails). Sincerely Luka Gejak P.S. If you are including me in further discussions or patch series please use luka.gejak@linux.dev instead of lukagejak5@gmail.com. Thanks in advance. On Tue, Mar 10, 2026 at 9:05 PM Sanjay Chitroda <sanjayembeddedse@gmail.com> wrote: > > From: Sanjay Chitroda <sanjayembeddedse@gmail.com> > > Hi all, > > This patch series replaces manual cleanup and explicit kfree() calls with > the __free attribute from <linux/cleanup.h>. This modernizes the memory > management style and simplifies common error paths without altering any > functional behavior. > > The __free attribute provides automatic scope-based cleanup, making > resource management clearer and reducing the chances of missing cleanup > on early returns. > > No functional changes are intended in this series. > > Testing: > - Compiled with W=1 > - Build-tested on i86_64 > > Based on: > <linux-v7.0-rc2> > > Feel free to share your valuable input in context of the cleanup API. > > Thanks, > Sanjay Chitroda > > Sanjay Chitroda (7): > staging: greybus: simplify cleanup using __free > iio: ssp_sensors: simplify cleanup using __free > iio: st_sensors: simplify cleanup using __free > media: mediatek: vcodec: simplify cleanup using __free > media: chips-media: coda: simplify cleanup using __free > media: allegro: simplify cleanup using __free > staging: rtl8723bs: simplify cleanup using __free > > drivers/iio/common/ssp_sensors/ssp_spi.c | 9 +- > .../iio/common/st_sensors/st_sensors_core.c | 7 +- > .../media/platform/allegro-dvt/allegro-core.c | 95 +++++-------------- > .../platform/chips-media/coda/coda-bit.c | 4 +- > .../platform/chips-media/coda/coda-jpeg.c | 39 ++++---- > .../mediatek/vcodec/common/mtk_vcodec_dbgfs.c | 3 +- > drivers/staging/greybus/camera.c | 27 ++---- > drivers/staging/greybus/loopback.c | 35 +++----- > drivers/staging/greybus/raw.c | 6 +- > .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 13 +-- > drivers/staging/rtl8723bs/hal/sdio_ops.c | 37 ++------ > 11 files changed, 78 insertions(+), 197 deletions(-) > > -- > 2.34.1 >
On 3/10/26 3:05 PM, Sanjay Chitroda wrote: > From: Sanjay Chitroda <sanjayembeddedse@gmail.com> > > Hi all, > > This patch series replaces manual cleanup and explicit kfree() calls with > the __free attribute from <linux/cleanup.h>. This modernizes the memory > management style and simplifies common error paths without altering any > functional behavior. > > The __free attribute provides automatic scope-based cleanup, making > resource management clearer and reducing the chances of missing cleanup > on early returns. > > No functional changes are intended in this series. > > Testing: > - Compiled with W=1 > - Build-tested on i86_64 > > Based on: > <linux-v7.0-rc2> > > Feel free to share your valuable input in context of the cleanup API. > > Thanks, > Sanjay Chitroda > > Sanjay Chitroda (7): > staging: greybus: simplify cleanup using __free > iio: ssp_sensors: simplify cleanup using __free > iio: st_sensors: simplify cleanup using __free > media: mediatek: vcodec: simplify cleanup using __free > media: chips-media: coda: simplify cleanup using __free > media: allegro: simplify cleanup using __free > staging: rtl8723bs: simplify cleanup using __free There is no reason to put patches from different subsystems in the same series when there is no dependency between them. It just make for more noise for everyone.
On 11 March 2026 3:22:28 am IST, David Lechner <dlechner@baylibre.com> wrote: >On 3/10/26 3:05 PM, Sanjay Chitroda wrote: >> From: Sanjay Chitroda <sanjayembeddedse@gmail.com> >> >> Hi all, >> >> This patch series replaces manual cleanup and explicit kfree() calls with >> the __free attribute from <linux/cleanup.h>. This modernizes the memory >> management style and simplifies common error paths without altering any >> functional behavior. >> >> The __free attribute provides automatic scope-based cleanup, making >> resource management clearer and reducing the chances of missing cleanup >> on early returns. >> >> No functional changes are intended in this series. >> >> Testing: >> - Compiled with W=1 >> - Build-tested on i86_64 >> >> Based on: >> <linux-v7.0-rc2> >> >> Feel free to share your valuable input in context of the cleanup API. >> >> Thanks, >> Sanjay Chitroda >> >> Sanjay Chitroda (7): >> staging: greybus: simplify cleanup using __free >> iio: ssp_sensors: simplify cleanup using __free >> iio: st_sensors: simplify cleanup using __free >> media: mediatek: vcodec: simplify cleanup using __free >> media: chips-media: coda: simplify cleanup using __free >> media: allegro: simplify cleanup using __free >> staging: rtl8723bs: simplify cleanup using __free > >There is no reason to put patches from different subsystems >in the same series when there is no dependency between them. >It just make for more noise for everyone. > Thanks for the feedback. I will split unrelated patches into separate series per subsystem or individual in future submissions.
On Wed, Mar 11, 2026 at 01:35:06AM +0530, Sanjay Chitroda wrote: > From: Sanjay Chitroda <sanjayembeddedse@gmail.com> > > Hi all, > > This patch series replaces manual cleanup and explicit kfree() calls with > the __free attribute from <linux/cleanup.h>. This modernizes the memory > management style and simplifies common error paths without altering any > functional behavior. > > The __free attribute provides automatic scope-based cleanup, making > resource management clearer and reducing the chances of missing cleanup > on early returns. > > No functional changes are intended in this series. > > Testing: > - Compiled with W=1 > - Build-tested on i86_64 > > Based on: > <linux-v7.0-rc2> > > Feel free to share your valuable input in context of the cleanup API. Do you put random people in the Cc list? You may try my script [1] to see the difference. [1]: https://github.com/andy-shev/home-bin-tools/blob/master/ge2maintainer.sh -- With Best Regards, Andy Shevchenko
© 2016 - 2026 Red Hat, Inc.