Re: [PATCH v3 00/28] media: staging: atomisp: Remove unnecessary return statements from void functions
From: Andy Shevchenko
Date: Tue Mar 10 2026 - 18:47:21 EST
On Tue, Mar 10, 2026 at 10:39:11PM +0100, Tomasz Unger wrote:
> Signed-off-by: Tomasz Unger <tomasz.unger@xxxxxxxx>
> ---
> Changes in v3:
> - Extended cleanup to all 27 files in the driver containing redundant
> 'return;' statements at the end of void functions (as suggested by
> Andy Shevchenko): inputfifo.c, bufq.c, ifmtr.c, frame.c, rx.c,
> binary.c, ia_css_debug.c, mmu_public.h,
> isp2400_input_system_private.h, hmm_bo.c, sh_css.c, sp_private.h,
> fifo_monitor_private.h, isp.c, input_formatter_private.h, irq.c,
> mmu.c, fifo_monitor.c, gp_device.c, gdc.c, event_fifo_private.h,
> isp_private.h, input_formatter.c, irq_private.h, gpio_private.h,
> gp_device_private.h, atomisp_cmd.c
> - Each removed 'return;' was verified to be at the end of a void function
> - Compiled successfully: make M=drivers/staging/media/atomisp/
> - Module loaded successfully in QEMU with Linux 7.0-rc3
> - Link to v2: https://lore.kernel.org/r/20260310-atomisp-remove-void-return-v2b-v2-1-b0e73f498e25@xxxxxxxx
You sent the series 3 times and all of the times it's not sent in full.
Besides that it must not be split by file! This kind of change is okay
to have in one patch.
--
With Best Regards,
Andy Shevchenko