Re: [PATCH v2 0/9] media: i2c: imx334: Miscellaneous cleanups and improvements

[Tarang Raval]

Hi Sakari, Kieran

Do you have any further comments or feedback on this patch series? 

Best regards,
Tarang

> This patch series is a collection of miscellaneous cleanups and                
> improvements to the imx334 driver.                                             
>                                                                                
> The changes simplify error handling, optimize register access,                 
> enhance power management, and add support for different lane                   
> configurations.
>                                                                                
> This series is on top of Shravan's patches [1]                                 
> (media: i2c: imx334: Add support for 1280x720 & 640x480 resolutions)           
>                                                                                
> Link [1]: https://lore.kernel.org/linux-media/20250305051442.3716817-1-shravan.chippa@xxxxxxxxxxxxx/T/#t
> 
> 
> Changelog:
> 
> v1 -> v2:
> 
> - In patch 2/9: Added space before braces.
> - In patch 4/9: Corrected the commit message.
> - Added new patches (6/9, 8/9, 9/9): Fixed PM handling, used subdev state
>   lock for synchronization, and switched to {enable, disable}_streams.
> 
> Tarang Raval (9):
>   media: i2c: imx334: Simplify with dev_err_probe()
>   media: i2c: imx334: Convert to CCI register access helpers
>   media: i2c: imx334: Remove redundant register entries
>   media: i2c: imx334: Configure lane mode Dynamically
>   media: i2c: imx334: Fix power management and control handling
>   media: i2c: imx334: Fix runtime PM handling in remove function
>   media: i2c: imx334: Enable runtime PM before sub-device registration
>   media: i2c: imx334: Use subdev state lock for synchronization
>   media: i2c: imx334: switch to {enable,disable}_streams
>
>  drivers/media/i2c/Kconfig  |   1 +
>  drivers/media/i2c/imx334.c | 959 ++++++++++++++++---------------------
>  2 files changed, 421 insertions(+), 539 deletions(-)
> 
> --
> 2.34.1