From: Martin Kepplinger <martin.kepplinger@puri.sm>
To: pavel@ucw.cz, laurent.pinchart@ideasonboard.com,
mchehab@kernel.org, sakari.ailus@iki.fi
Cc: devicetree@vger.kernel.org, kernel@puri.sm,
krzysztof.kozlowski@canonical.com, linux-kernel@vger.kernel.org,
linux-media@vger.kernel.org, paul.kocialkowski@bootlin.com,
phone-devel@vger.kernel.org, robh@kernel.org,
shawnx.tu@intel.com
Subject: Re: [PATCH v9 0/4] Add support for the Hynix Hi-846 camera
Date: Tue, 28 Sep 2021 11:50:25 +0200 [thread overview]
Message-ID: <802a27f49cbde7d37179a9dfddca837f096a34b4.camel@puri.sm> (raw)
In-Reply-To: <20210906102837.2190387-1-martin.kepplinger@puri.sm>
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
>
prev parent reply other threads:[~2021-09-28 9:50 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-06 10:28 [PATCH v9 0/4] Add support for the Hynix Hi-846 camera Martin Kepplinger
2021-09-06 10:28 ` [PATCH v9 1/4] dt-bindings: vendor-prefixes: Add SK Hynix Inc Martin Kepplinger
2021-09-06 10:28 ` [PATCH v9 2/4] dt-bindings: media: document SK Hynix Hi-846 MIPI CSI-2 8M pixel sensor Martin Kepplinger
2021-11-25 11:15 ` Geert Uytterhoeven
2021-11-26 10:02 ` Martin Kepplinger
2021-11-26 10:18 ` Sakari Ailus
2021-11-26 12:03 ` Geert Uytterhoeven
2021-11-30 14:57 ` Martin Kepplinger
2021-12-01 10:32 ` Geert Uytterhoeven
2021-12-03 16:19 ` Rob Herring
2022-01-05 16:23 ` Rob Herring
2022-01-06 6:34 ` Martin Kepplinger
2021-09-06 10:28 ` [PATCH v9 3/4] media: i2c: add driver for the SK Hynix Hi-846 8M pixel camera Martin Kepplinger
2021-09-06 10:28 ` [PATCH v9 4/4] Documentation: i2c-cardlist: add the Hynix hi846 sensor Martin Kepplinger
2021-09-28 9:50 ` Martin Kepplinger [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=802a27f49cbde7d37179a9dfddca837f096a34b4.camel@puri.sm \
--to=martin.kepplinger@puri.sm \
--cc=devicetree@vger.kernel.org \
--cc=kernel@puri.sm \
--cc=krzysztof.kozlowski@canonical.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=paul.kocialkowski@bootlin.com \
--cc=pavel@ucw.cz \
--cc=phone-devel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=sakari.ailus@iki.fi \
--cc=shawnx.tu@intel.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).