From: Drew Davenport <ddavenport@chromium.org>
To: Louis Kuo <louis.kuo@mediatek.com>
Cc: hans.verkuil@cisco.com,
laurent.pinchart+renesas@ideasonboard.com, tfiga@chromium.org,
keiichiw@chromium.org, matthias.bgg@gmail.com,
mchehab@kernel.org, yuzhao@chromium.org, zwisler@chromium.org,
linux-mediatek@lists.infradead.org,
linux-arm-kernel@lists.infradead.org, Sean.Cheng@mediatek.com,
sj.huang@mediatek.com, christie.yu@mediatek.com,
holmes.chiou@mediatek.com, frederic.chen@mediatek.com,
Jerry-ch.Chen@mediatek.com, jungo.lin@mediatek.com,
Rynn.Wu@mediatek.com, linux-media@vger.kernel.org,
srv_heupstream@mediatek.com, devicetree@vger.kernel.org
Subject: Re: [RFC PATCH V2 0/4] media: support Mediatek sensor interface driver
Date: Tue, 14 May 2019 14:19:52 -0600 [thread overview]
Message-ID: <20190514201952.GA107202@chromium.org> (raw)
In-Reply-To: <20190512060005.5444-1-louis.kuo@mediatek.com>
Hello Louis,
On Sun, May 12, 2019 at 02:00:01PM +0800, Louis Kuo wrote:
> Hello,
>
> This is the RFC patch adding Sensor Inferface(seninf) driver on
> Mediatek mt8183 SoC, which will be used in camera features on CrOS application.
> It belongs to the first Mediatek's camera driver series based on V4L2 and media controller framework.
> I posted the main part of the seninf driver as RFC to discuss first and would like some review comments
> on the overall structure of the driver.
>
> The driver is implemented with V4L2 framework.
> 1. Register as a V4L2 sub-device.
> 2. Only one entity with sink pads linked to camera sensors for choosing desired camera sensor by setup link
> and with source pads linked to cam-io for routing different types of decoded packet datas to PASS1 driver
> to generate sensor image frame and meta-data.
>
> The overall file structure of the seninf driver is as following:
>
> * mtk_seninf.c: Implement software and HW control flow of seninf driver.
> * mtk_seninf_def.h: Define data structure and enumeration.
> * mtk_seninf_reg.h: Define HW register R/W macros and HW register names.
>
> [ v2: use recommended coding style, fix v4l2-compliance issue, add v4l2 async notifier operations, remove redundant code]
Thanks for the style fixes. I made a number of review comments on V1 of
the driver patch. Please take a look at those if you haven't already.
>
> media: platform: mtk-isp: Add Mediatek sensor interface driver
> media: platform: Add Mediatek sensor interface driver KConfig
> dt-bindings: mt8183: Add sensor interface dt-bindings
> dts: arm64: mt8183: Add sensor interface nodes
>
> .../bindings/media/mediatek-seninf.txt | 52 +
> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 34 +
> drivers/media/platform/Makefile | 2 +
> drivers/media/platform/mtk-isp/Kconfig | 16 +
> drivers/media/platform/mtk-isp/Makefile | 14 +
> .../media/platform/mtk-isp/isp_50/Makefile | 17 +
> .../platform/mtk-isp/isp_50/seninf/Makefile | 4 +
> .../mtk-isp/isp_50/seninf/mtk_seninf.c | 1366 +++++++++++++++++
> .../mtk-isp/isp_50/seninf/mtk_seninf_def.h | 170 ++
> .../mtk-isp/isp_50/seninf/mtk_seninf_reg.h | 992 ++++++++++++
> 10 files changed, 2667 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/media/mediatek-seninf.txt
> create mode 100644 drivers/media/platform/mtk-isp/Kconfig
> create mode 100644 drivers/media/platform/mtk-isp/Makefile
> create mode 100644 drivers/media/platform/mtk-isp/isp_50/Makefile
> create mode 100644 drivers/media/platform/mtk-isp/isp_50/seninf/Makefile
> create mode 100644 drivers/media/platform/mtk-isp/isp_50/seninf/mtk_seninf.c
> create mode 100644 drivers/media/platform/mtk-isp/isp_50/seninf/mtk_seninf_def.h
> create mode 100644 drivers/media/platform/mtk-isp/isp_50/seninf/mtk_seninf_reg.h
>
>
prev parent reply other threads:[~2019-05-14 20:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-12 6:00 [RFC PATCH V2 0/4] media: support Mediatek sensor interface driver Louis Kuo
2019-05-12 6:00 ` [RFC PATCH V2 1/4] media: platform: mtk-isp: Add " Louis Kuo
2019-05-13 8:27 ` Hans Verkuil
2019-05-12 6:00 ` [RFC PATCH V2 2/4] media: platform: Add Mediatek sensor interface driver KConfig Louis Kuo
2019-05-12 6:00 ` [RFC PATCH V2 3/4] dt-bindings: mt8183: Add sensor interface dt-bindings Louis Kuo
2019-05-14 20:22 ` Rob Herring
2019-05-12 6:00 ` [RFC PATCH V2 4/4] dts: arm64: mt8183: Add sensor interface nodes Louis Kuo
2019-05-14 20:19 ` Drew Davenport [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=20190514201952.GA107202@chromium.org \
--to=ddavenport@chromium.org \
--cc=Jerry-ch.Chen@mediatek.com \
--cc=Rynn.Wu@mediatek.com \
--cc=Sean.Cheng@mediatek.com \
--cc=christie.yu@mediatek.com \
--cc=devicetree@vger.kernel.org \
--cc=frederic.chen@mediatek.com \
--cc=hans.verkuil@cisco.com \
--cc=holmes.chiou@mediatek.com \
--cc=jungo.lin@mediatek.com \
--cc=keiichiw@chromium.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=louis.kuo@mediatek.com \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=sj.huang@mediatek.com \
--cc=srv_heupstream@mediatek.com \
--cc=tfiga@chromium.org \
--cc=yuzhao@chromium.org \
--cc=zwisler@chromium.org \
/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).