staging: media: atomisp: clean up ia_css_stream_destroy

[PATCH v2 0/3] staging: media: atomisp: clean up ia_css_stream_destroy

Posted by Jose A. Perez de Azpillaga 6 days, 19 hours ago

This series fixes a loop shadowing bug and refactors the ISP2401 cleanup
logic inside ia_css_stream_destroy().

Changes in v2:
- Split the original patch into a 3-patch series as requested by
  Dan Carpenter.
- Added a Fixes tag for the loop shadowing bug.
- Separated the extraction from the logical improvements.
- Added clarification that assert() is a wrapper around BUG().

Jose A. Perez de Azpillaga (3):
  staging: media: atomisp: fix loop shadowing in ia_css_stream_destroy()
  staging: media: atomisp: extract ISP2401 cleanup into helper function
  staging: media: atomisp: improve ia_css_stream_destroy_isp2401

 drivers/staging/media/atomisp/pci/sh_css.c | 79 ++++++++++------------
 1 file changed, 37 insertions(+), 42 deletions(-)

--
2.53.0

Re: [PATCH v2 0/3] staging: media: atomisp: clean up ia_css_stream_destroy

Posted by Dan Carpenter 6 days, 9 hours ago

On Thu, Mar 26, 2026 at 10:34:06PM +0100, Jose A. Perez de Azpillaga wrote:
> This series fixes a loop shadowing bug and refactors the ISP2401 cleanup
> logic inside ia_css_stream_destroy().
> 
> Changes in v2:
> - Split the original patch into a 3-patch series as requested by
>   Dan Carpenter.
> - Added a Fixes tag for the loop shadowing bug.
> - Separated the extraction from the logical improvements.
> - Added clarification that assert() is a wrapper around BUG().

Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>

regards,
dan carpenter