linux-kernel - Re: [PATCH v8 3/4] media: i2c: add driver for the SK Hynix Hi-846 8M pixel camera

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <99d6902d8e4ed68de4c011d618344485552002ea.camel@puri.sm>
Date:   Mon, 06 Sep 2021 09:49:09 +0200
From:   Martin Kepplinger <martin.kepplinger@...i.sm>
To:     Pavel Machek <pavel@....cz>
Cc:     devicetree@...r.kernel.org, kernel@...i.sm,
        krzysztof.kozlowski@...onical.com,
        laurent.pinchart@...asonboard.com, linux-kernel@...r.kernel.org,
        linux-media@...r.kernel.org, mchehab@...nel.org,
        paul.kocialkowski@...tlin.com, phone-devel@...r.kernel.org,
        robh@...nel.org, sakari.ailus@....fi, shawnx.tu@...el.com
Subject: Re: [PATCH v8 3/4] media: i2c: add driver for the SK Hynix Hi-846
 8M pixel camera

Am Freitag, dem 03.09.2021 um 18:17 +0200 schrieb Pavel Machek:
> Hi!
> 
> > The SK Hynix Hi-846 is a 1/4" 8M Pixel CMOS Image Sensor. It
> > supports
> > usual features like I2C control, CSI-2 for frame data,
> > digital/analog
> > gain control or test patterns.
> > 
> > This driver supports the 640x480, 1280x720 and 1632x1224 resolution
> > modes. It supports runtime PM in order not to draw any unnecessary
> > power.
> > 
> > The part is also called YACG4D0C9SHC and a datasheet can be found
> > at
> > https://product.skhynix.com/products/cis/cis.go
> > 
> > The large sets of partly undocumented register values are for
> > example
> > found when searching for the hi846_mipi_raw_Sensor.c Android
> > driver.
> > 
> > Signed-off-by: Martin Kepplinger <martin.kepplinger@...i.sm>
> 
> Reviewed-by: Pavel Machek <pavel@....cz>
> 
> Some nitpicks below..
> 
> > +config VIDEO_HI846
> > +       tristate "Hynix Hi-846 sensor support"
> > +       depends on I2C && VIDEO_V4L2
> > +       select MEDIA_CONTROLLER
> > +       select VIDEO_V4L2_SUBDEV_API
> > +       select V4L2_FWNODE
> > +       help
> > +         This is a Video4Linux2 sensor driver for the Hynix
> > +         Hi-846 camera.
> > +
> > +         To compile this driver as a module, choose M here: the
> > +         module will be called hi846.
> 
> hi846.ko?

hi Pavel, thanks a lot for taking the time! the module name doesn't
include the file ending, according to all other Kconfig descriptions.

the rest looks good and I'll change it according to your review.

                           martin