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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <PN3P287MB1829D27A21BFFEC97C8F5B678BB22@PN3P287MB1829.INDP287.PROD.OUTLOOK.COM>
Date: Tue, 15 Apr 2025 09:23:41 +0000
From: Tarang Raval <tarang.raval@...iconsignals.io>
To: "sakari.ailus@...ux.intel.com" <sakari.ailus@...ux.intel.com>,
	"kieran.bingham@...asonboard.com" <kieran.bingham@...asonboard.com>
CC: "Shravan.Chippa@...rochip.com" <Shravan.Chippa@...rochip.com>, Mauro
 Carvalho Chehab <mchehab@...nel.org>, Hans Verkuil <hverkuil@...all.nl>,
	Laurent Pinchart <laurent.pinchart@...asonboard.com>, Julien Massot
	<julien.massot@...labora.com>, Zhi Mao <zhi.mao@...iatek.com>, Mikhail
 Rudenko <mike.rudenko@...il.com>, Kory Maincent <kory.maincent@...tlin.com>,
	Benjamin Mugnier <benjamin.mugnier@...s.st.com>,
	"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/9] media: i2c: imx334: Miscellaneous cleanups and
 improvements

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@microchip.com/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