From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Umang Jain <umang.jain@ideasonboard.com>
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Lee Jackson <lee.jackson@arducam.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Hans Verkuil <hverkuil-cisco@xs4all.nl>,
Shawn Tu <shawnx.tu@intel.com>,
kieran.bingham@ideasonboard.com, jacopo.mondi@ideasonboard.com
Subject: Re: [PATCH v6 2/2] media: i2c: Add driver for IMX519 sensor
Date: Fri, 8 Sep 2023 20:26:41 +0300 [thread overview]
Message-ID: <20230908172641.GF17610@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20230908124344.171662-3-umang.jain@ideasonboard.com>
Hi Umang,
Thank you for the patch.
On Fri, Sep 08, 2023 at 08:43:44AM -0400, Umang Jain wrote:
> From: Lee Jackson <lee.jackson@arducam.com>
>
> Add a driver for the 16MPix IMX519 CSI2 sensor.
> Whilst the sensor supports 2 or 4 CSI2 data lanes, this driver
> currently only supports 2 lanes.
>
> The following Bayer modes are currently available:
>
> 4656x3496 10-bit @ 10fps
> 3840x2160 10-bit (cropped) @ 21fps
> 2328x1748 10-bit (binned) @ 30fps
> 1920x1080 10-bit (cropped/binned) @ 60fps
> 1280x720 10-bit (cropped/binned) @ 120fps
>
> Signed-off-by: Lee Jackson <lee.jackson@arducam.com>
> Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
> ---
> MAINTAINERS | 1 +
> drivers/media/i2c/Kconfig | 14 +
> drivers/media/i2c/Makefile | 1 +
> drivers/media/i2c/imx519.c | 1842 ++++++++++++++++++++++++++++++++++++
> 4 files changed, 1858 insertions(+)
> create mode 100644 drivers/media/i2c/imx519.c
[snip]
> diff --git a/drivers/media/i2c/imx519.c b/drivers/media/i2c/imx519.c
> new file mode 100644
> index 000000000000..f818a3d0bd25
> --- /dev/null
> +++ b/drivers/media/i2c/imx519.c
> @@ -0,0 +1,1842 @@
[snip]
> +/* 16 mpix 10fps */
> +static const struct cci_reg_sequence mode_4656x3496_regs[] = {
> + { CCI_REG8(0x0111), 0x02 },
> + { CCI_REG8(0x0112), 0x0a },
> + { CCI_REG8(0x0113), 0x0a },
> + { CCI_REG8(0x0114), 0x01 },
> + { CCI_REG8(0x0342), 0x42 },
> + { CCI_REG8(0x0343), 0x00 },
> + { CCI_REG8(0x0340), 0x0d },
> + { CCI_REG8(0x0341), 0xf4 },
> + { CCI_REG8(0x0344), 0x00 },
> + { CCI_REG8(0x0345), 0x00 },
> + { CCI_REG8(0x0346), 0x00 },
> + { CCI_REG8(0x0347), 0x00 },
> + { CCI_REG8(0x0348), 0x12 },
> + { CCI_REG8(0x0349), 0x2f },
> + { CCI_REG8(0x034a), 0x0d },
> + { CCI_REG8(0x034b), 0xa7 },
> + { CCI_REG8(0x0220), 0x00 },
> + { CCI_REG8(0x0221), 0x11 },
> + { CCI_REG8(0x0222), 0x01 },
> + { CCI_REG8(0x0900), 0x00 },
> + { CCI_REG8(0x0901), 0x11 },
> + { CCI_REG8(0x0902), 0x0a },
> + { CCI_REG8(0x3f4c), 0x01 },
> + { CCI_REG8(0x3f4d), 0x01 },
> + { CCI_REG8(0x4254), 0x7f },
> + { CCI_REG8(0x0401), 0x00 },
> + { CCI_REG8(0x0404), 0x00 },
> + { CCI_REG8(0x0405), 0x10 },
> + { CCI_REG8(0x0408), 0x00 },
> + { CCI_REG8(0x0409), 0x00 },
> + { CCI_REG8(0x040a), 0x00 },
> + { CCI_REG8(0x040b), 0x00 },
> + { CCI_REG8(0x040c), 0x12 },
> + { CCI_REG8(0x040d), 0x30 },
> + { CCI_REG8(0x040e), 0x0d },
> + { CCI_REG8(0x040f), 0xa8 },
> + { CCI_REG8(0x034c), 0x12 },
> + { CCI_REG8(0x034d), 0x30 },
> + { CCI_REG8(0x034e), 0x0d },
> + { CCI_REG8(0x034f), 0xa8 },
> + { CCI_REG8(0x0301), 0x06 },
> + { CCI_REG8(0x0303), 0x04 },
> + { CCI_REG8(0x0305), 0x04 },
> + { CCI_REG8(0x0306), 0x01 },
> + { CCI_REG8(0x0307), 0x57 },
> + { CCI_REG8(0x0309), 0x0a },
> + { CCI_REG8(0x030b), 0x02 },
> + { CCI_REG8(0x030d), 0x04 },
> + { CCI_REG8(0x030e), 0x01 },
> + { CCI_REG8(0x030f), 0x49 },
> + { CCI_REG8(0x0310), 0x01 },
> + { CCI_REG8(0x0820), 0x07 },
> + { CCI_REG8(0x0821), 0xb6 },
> + { CCI_REG8(0x0822), 0x00 },
> + { CCI_REG8(0x0823), 0x00 },
> + { CCI_REG8(0x3e20), 0x01 },
> + { CCI_REG8(0x3e37), 0x00 },
> + { CCI_REG8(0x3e3b), 0x00 },
> + { CCI_REG8(0x0106), 0x00 },
> + { CCI_REG8(0x0b00), 0x00 },
> + { CCI_REG8(0x3230), 0x00 },
> + { CCI_REG8(0x3f14), 0x01 },
> + { CCI_REG8(0x3f3c), 0x01 },
> + { CCI_REG8(0x3f0d), 0x0a },
> + { CCI_REG8(0x3fbc), 0x00 },
> + { CCI_REG8(0x3c06), 0x00 },
> + { CCI_REG8(0x3c07), 0x48 },
> + { CCI_REG8(0x3c0a), 0x00 },
> + { CCI_REG8(0x3c0b), 0x00 },
> + { CCI_REG8(0x3f78), 0x00 },
> + { CCI_REG8(0x3f79), 0x40 },
> + { CCI_REG8(0x3f7c), 0x00 },
> + { CCI_REG8(0x3f7d), 0x00 },
> +};
Looking at the per-mode tables, it appears that all registers below
0x3000 are CCS-compliant. Furthermore, all registers from 0x3000 above
(defined in CCS as the manufacturer-specific register - or MSR - range)
are identical for all modes. I can't tell if other modes could require
different values for MSRs, so it's hard to assess real CCS compliance,
but it looks like giving the CCS driver a try would be a good idea.
I'm sure Sakari will be delighted :-)
[snip]
--
Regards,
Laurent Pinchart
prev parent reply other threads:[~2023-09-08 17:26 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-08 12:43 [PATCH v6 0/2] media: i2c: imx519: Support for Sony IMX519 sensor Umang Jain
2023-09-08 12:43 ` [PATCH v6 1/2] dt-bindings: media: i2c: Add IMX519 CMOS sensor Umang Jain
2023-09-08 12:43 ` [PATCH v6 2/2] media: i2c: Add driver for IMX519 sensor Umang Jain
2023-09-08 15:18 ` Dave Stevenson
2023-09-08 16:23 ` Jacopo Mondi
2023-09-08 16:57 ` Dave Stevenson
2023-09-08 17:29 ` Kieran Bingham
2023-09-08 17:49 ` Dave Stevenson
2023-09-08 17:26 ` Laurent Pinchart [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230908172641.GF17610@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=devicetree@vger.kernel.org \
--cc=hverkuil-cisco@xs4all.nl \
--cc=jacopo.mondi@ideasonboard.com \
--cc=kieran.bingham@ideasonboard.com \
--cc=lee.jackson@arducam.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=shawnx.tu@intel.com \
--cc=umang.jain@ideasonboard.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).