From: "Zhi Mao (毛智)" <zhi.mao@mediatek.com>
To: "robh@kernel.org" <robh@kernel.org>,
"mchehab@kernel.org" <mchehab@kernel.org>,
"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
"angelogioacchino.delregno@collabora.com"
<angelogioacchino.delregno@collabora.com>,
"conor+dt@kernel.org" <conor+dt@kernel.org>
Cc: "heiko@sntech.de" <heiko@sntech.de>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"laurent.pinchart+renesas@ideasonboard.com"
<laurent.pinchart+renesas@ideasonboard.com>,
"yunkec@chromium.org" <yunkec@chromium.org>,
"linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
"hdegoede@redhat.com" <hdegoede@redhat.com>,
"bingbu.cao@intel.com" <bingbu.cao@intel.com>,
"paul.elder@ideasonboard.com" <paul.elder@ideasonboard.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
"Yaya Chang (張雅清)" <Yaya.Chang@mediatek.com>,
"Shengnan Wang (王圣男)" <shengnan.wang@mediatek.com>,
"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
"alain.volmat@foss.st.com" <alain.volmat@foss.st.com>,
"sakari.ailus@linux.intel.com" <sakari.ailus@linux.intel.com>,
"tomi.valkeinen@ideasonboard.com"
<tomi.valkeinen@ideasonboard.com>,
"10572168@qq.com" <10572168@qq.com>,
"hverkuil-cisco@xs4all.nl" <hverkuil-cisco@xs4all.nl>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
"mehdi.djait@bootlin.com" <mehdi.djait@bootlin.com>
Subject: Re: [PATCH v3 2/3] media: i2c: Add GT97xx VCM driver
Date: Wed, 12 Jun 2024 11:13:40 +0000 [thread overview]
Message-ID: <18d2c28fc8b47889689a1506957ea2a308c80fa2.camel@mediatek.com> (raw)
In-Reply-To: <7c71534f-9815-4ea3-969f-c04d249d35d2@collabora.com>
Hi Angelo,
Thanks for your review.
On Wed, 2024-06-12 at 09:07 +0200, AngeloGioacchino Del Regno wrote:
> Il 12/06/24 03:20, Zhi Mao ha scritto:
> > Add a V4L2 sub-device driver for Giantec GT97xx VCM.
> >
> > Signed-off-by: Zhi Mao <zhi.mao@mediatek.com>
>
> Hello Zhi,
>
> I fail to see why would you need to upstream this new driver instead
> of
> simply adding the IC_INFO_REG to the already existing (and more
> featureful)
> dw9768 driver, which also seems to support the Giantec GT9769 VCM.
>
Our project uses Giantec VCM hardware.
For detailed vendor information, please visit: (
https://en.giantec-semi.com/yqmd/164).
The VCM datasheet we are referencing is provided by Giantec.
Currently, the relationship between Giantec VCM and Dongwoon VCM is
unclear, but Dongwoon seems to be another manufacturer of VCM
hardware.
From the perspective of software driver development and maintenance, it
makes sense for each vendor's hardware should have its own software
driver.
So, I upstream a new VCM driver for Giantec.
> Cheers,
> Angelo
>
> >
>
>
next prev parent reply other threads:[~2024-06-12 11:13 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-12 1:20 [PATCH v3 0/3] media: i2c: Add support for GT97xx VCM Zhi Mao
2024-06-12 1:20 ` [PATCH v3 1/3] media: dt-bindings: i2c: add Giantec " Zhi Mao
2024-06-12 7:08 ` AngeloGioacchino Del Regno
2024-06-12 1:20 ` [PATCH v3 2/3] media: i2c: Add GT97xx VCM driver Zhi Mao
2024-06-12 7:07 ` AngeloGioacchino Del Regno
2024-06-12 11:13 ` Zhi Mao (毛智) [this message]
2024-06-13 0:05 ` Kieran Bingham
2024-06-13 19:51 ` Andy Shevchenko
2024-06-21 17:36 ` sakari.ailus
2024-06-12 1:20 ` [PATCH v3 3/3] MAINTAINERS: Add entry for " Zhi Mao
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=18d2c28fc8b47889689a1506957ea2a308c80fa2.camel@mediatek.com \
--to=zhi.mao@mediatek.com \
--cc=10572168@qq.com \
--cc=Yaya.Chang@mediatek.com \
--cc=alain.volmat@foss.st.com \
--cc=andy.shevchenko@gmail.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=bingbu.cao@intel.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=hdegoede@redhat.com \
--cc=heiko@sntech.de \
--cc=hverkuil-cisco@xs4all.nl \
--cc=krzk+dt@kernel.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=mehdi.djait@bootlin.com \
--cc=p.zabel@pengutronix.de \
--cc=paul.elder@ideasonboard.com \
--cc=robh@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=shengnan.wang@mediatek.com \
--cc=tomi.valkeinen@ideasonboard.com \
--cc=yunkec@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).