Hi Dave, On Mon, Oct 10, 2022 at 11:31:33AM +0100, Dave Stevenson wrote: > Hi Laurent > > Do you have plans for a v2 on this patch set? I also have a number of > patches for imx290 and there's little point in causing grief to each > other with conflicts. > Or I could take the non-controversial patches from your set and send a > combined patch set? I'm working on a v2, I'll send it shortly. Do I assume correctly you'll submit IMX327 support on top ? :-) > On Tue, 23 Aug 2022 at 02:12, Laurent Pinchart wrote: > > > > Hi Sakari, > > > > Could you already pick up patches 01/19, 02/19, 04/19, 05/19 and 06/19 > > in your tree ? Your opinion on 03/19 woud be appreciated too, I think > > it's a candidate for merge as well. > > > > On Thu, Jul 21, 2022 at 11:35:21AM +0300, Laurent Pinchart wrote: > > > Hello, > > > > > > This patch series gathers miscellaneous improvements for the imx290 > > > driver. The most notable changes on the kernel side is patch 07/19 that > > > simplifies register access, and on the userspace API side patches 14/19, > > > 15/19 and 18/19 that extend the driver with controls and selection > > > rectangles required by libcamera. > > > > > > Laurent Pinchart (19): > > > media: i2c: imx290: Use device lock for the control handler > > > media: i2c: imx290: Print error code when I2C transfer fails > > > media: i2c: imx290: Specify HMAX values in decimal > > > media: i2c: imx290: Replace macro with explicit ARRAY_SIZE() > > > media: i2c: imx290: Drop imx290_write_buffered_reg() > > > media: i2c: imx290: Drop regmap cache > > > media: i2c: imx290: Support variable-sized registers > > > media: i2c: imx290: Correct register sizes > > > media: i2c: imx290: Simplify error handling when writing registers > > > media: i2c: imx290: Define more register macros > > > media: i2c: imx290: Add exposure time control > > > media: i2c: imx290: Fix max gain value > > > media: i2c: imx290: Split control initialization to separate function > > > media: i2c: imx290: Implement HBLANK and VBLANK controls > > > media: i2c: imx290: Create controls for fwnode properties > > > media: i2c: imx290: Move registers with fixed value to init array > > > media: i2c: imx290: Factor out format retrieval to separate function > > > media: i2c: imx290: Add crop selection targets support > > > media: i2c: imx290: Replace GAIN control with ANALOGUE_GAIN > > > > > > drivers/media/i2c/imx290.c | 781 ++++++++++++++++++++++--------------- > > > 1 file changed, 458 insertions(+), 323 deletions(-) -- Regards, Laurent Pinchart