linux-kernel - Re: [PATCH v9 0/4] Add support for the Hynix Hi-846 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: <802a27f49cbde7d37179a9dfddca837f096a34b4.camel@puri.sm>
Date:   Tue, 28 Sep 2021 11:50:25 +0200
From:   Martin Kepplinger <martin.kepplinger@...i.sm>
To:     pavel@....cz, laurent.pinchart@...asonboard.com,
        mchehab@...nel.org, sakari.ailus@....fi
Cc:     devicetree@...r.kernel.org, kernel@...i.sm,
        krzysztof.kozlowski@...onical.com, linux-kernel@...r.kernel.org,
        linux-media@...r.kernel.org, paul.kocialkowski@...tlin.com,
        phone-devel@...r.kernel.org, robh@...nel.org, shawnx.tu@...el.com
Subject: Re: [PATCH v9 0/4] Add support for the Hynix Hi-846 camera

Am Montag, dem 06.09.2021 um 12:28 +0200 schrieb Martin Kepplinger:
> hi,
> 
> This series adds support for the SK Hynix Hi-846 CMOS images sensor.
> It includes the dt-bindings and the driver.
> 
> best wishes,
> 
>                               martin

hi all, are there any objections, doubts or other thoughts about this
that would prevent it from being merged?

thanks for all the good review,

                                martin

> 
> revision history
> ----------------
> v9: (thank you Pavel and Sakari)
> * fix the probe() error paths (that I broke in v8)
> * cheanups
> 
> v8: (thank you Laurent)
> * add fwnode properties support for orientation and rotation on the
> board
> * remove the arm64 defconfig addition patch. deal with that later.
> * 
> https://lore.kernel.org/linux-media/20210831134344.1673318-1-martin.kepplinger@puri.sm/
> 
> v7: (thank you Sakari)
> * move the check for supported nr_lanes for a mode to set_format()
> * change get_selection() according to the Sakaris' review
> * check for the mipi link frequencies from DT
> * check for the external clock rate and add assigned-clock-rates
> requirement
> * 
> https://lore.kernel.org/linux-media/20210712084137.3779628-1-martin.kepplinger@puri.sm/
> 
> v6:
> * better digital gain defaults
> * lane config fix found by smatch
> * fix regulator usage in power_on()
> * 
> https://lore.kernel.org/linux-media/20210628101054.828579-1-martin.kepplinger@puri.sm/
> 
> v5: (thank you Laurent and Rob)
> * minor dt-bindings fixes
> * driver: disable lens shading correcting (no seed values yet used
> from "otp" for that)
> * add reviewed-tags
> * 
> https://lore.kernel.org/linux-media/20210611101404.2553818-1-martin.kepplinger@puri.sm/
> 
> v4: (thank you Laurent, Sakari and Rob) many driver changes, see v3
> review for
> details. they include:
> * add get_selection(), remove open() callback
> * use gpiod API
> * use regulator_bulk API
> * fix power supply timing sequence and bindings
> * 
> https://lore.kernel.org/linux-media/20210607105213.1211722-1-martin.kepplinger@puri.sm/
> 
> v3: (thank you, Laurent)
> * use do_div() for divisions
> * reset-gpios DT property name instead of rst-gpios
> * improve the dt-bindings
> * add the phone-devel list
> * 
> https://lore.kernel.org/linux-media/20210531120737.168496-1-martin.kepplinger@puri.sm/
> 
> v2:
> sent a bit early due to stupid mistakes
> * fix build issues
> * fix dtschema issues
> * add enable for arm64 defconfig
> * 
> https://lore.kernel.org/linux-media/20210530212337.GA15366@duo.ucw.cz/T/#t
> 
> v1:
> * 
> https://lore.kernel.org/linux-media/20210527091221.3335998-1-martin.kepplinger@puri.sm/
> 
> 
> Martin Kepplinger (4):
>   dt-bindings: vendor-prefixes: Add SK Hynix Inc.
>   dt-bindings: media: document SK Hynix Hi-846 MIPI CSI-2 8M pixel
>     sensor
>   media: i2c: add driver for the SK Hynix Hi-846 8M pixel camera
>   Documentation: i2c-cardlist: add the Hynix hi846 sensor
> 
>  .../admin-guide/media/i2c-cardlist.rst        |    1 +
>  .../bindings/media/i2c/hynix,hi846.yaml       |  120 +
>  .../devicetree/bindings/vendor-prefixes.yaml  |    2 +
>  MAINTAINERS                                   |    6 +
>  drivers/media/i2c/Kconfig                     |   13 +
>  drivers/media/i2c/Makefile                    |    1 +
>  drivers/media/i2c/hi846.c                     | 2190
> +++++++++++++++++
>  7 files changed, 2333 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml
>  create mode 100644 drivers/media/i2c/hi846.c
>