From: Yingjoe Chen <yingjoe.chen@mediatek.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: YH Huang <yh.huang@mediatek.com>,
Mark Rutland <mark.rutland@arm.com>, <linux-pwm@vger.kernel.org>,
<srv_heupstream@mediatek.com>, Pawel Moll <pawel.moll@arm.com>,
"open list:OPEN FIRMWARE AND..." <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>,
"Thierry Reding" <thierry.reding@gmail.com>,
Sascha Hauer <kernel@pengutronix.de>,
Matthias Brugger <matthias.bgg@gmail.com>,
<linux-mediatek@lists.infradead.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/2] pwm: add Mediatek display PWM driver support
Date: Tue, 26 May 2015 16:29:03 +0800 [thread overview]
Message-ID: <1432628943.11070.2.camel@mtksdaap41> (raw)
In-Reply-To: <20150526060533.GW6325@pengutronix.de>
On Tue, 2015-05-26 at 08:05 +0200, Sascha Hauer wrote:
> On Thu, May 21, 2015 at 04:22:31PM +0800, YH Huang wrote:
> > On Mon, 2015-05-18 at 11:42 +0800, Daniel Kurtz wrote:
<...>
> > > On Mon, May 11, 2015 at 5:26 PM, YH Huang <yh.huang@mediatek.com> wrote:
> > > > +
> > > > +static const struct of_device_id mtk_disp_pwm_of_match[] = {
> > > > + { .compatible = "mediatek,mt6595-disp-pwm" },
> > >
> > > Does this driver support the PWM in mt8173?
> > > If so, don't you need this, too:
> > >
> > > { .compatible = "mediatek,mt8173-disp-pwm" },
> >
> > Yes, it supports mt8173.
> > I will add it.
>
> If both yre compatible you don't need to add it since you have a
> "mediatek,mt6595-disp-pwm" in the mt8173 device tree.
Hi Sascha,
We got a similar discussion about this with Mark Brown for spi driver,
please see:
http://lists.infradead.org/pipermail/linux-mediatek/2015-May/000673.html
Joe.C
prev parent reply other threads:[~2015-05-26 8:31 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-11 9:26 [PATCH 0/2] Add Mediatek display PWM driver YH Huang
2015-05-11 9:26 ` [PATCH 1/2] dt-bindings: pwm: add Mediatek display PWM bindings YH Huang
2015-05-11 9:26 ` [PATCH 2/2] pwm: add Mediatek display PWM driver support YH Huang
2015-05-12 12:37 ` Matthias Brugger
2015-05-12 12:44 ` Matthias Brugger
2015-05-14 14:45 ` YH Huang
2015-05-14 14:39 ` YH Huang
2015-05-12 13:00 ` Sascha Hauer
2015-05-14 14:52 ` YH Huang
2015-05-12 13:32 ` Thierry Reding
2015-05-12 14:34 ` Matthias Brugger
2015-05-14 15:39 ` YH Huang
2015-05-14 15:35 ` YH Huang
2015-05-18 3:42 ` Daniel Kurtz
2015-05-21 8:22 ` YH Huang
2015-05-26 6:05 ` Sascha Hauer
2015-05-26 8:29 ` Yingjoe Chen [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=1432628943.11070.2.camel@mtksdaap41 \
--to=yingjoe.chen@mediatek.com \
--cc=devicetree@vger.kernel.org \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pwm@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=srv_heupstream@mediatek.com \
--cc=thierry.reding@gmail.com \
--cc=yh.huang@mediatek.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