From: Matthias Brugger <matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Daniel Kurtz <djkurtz-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: YH Huang <yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
"open list:OPEN FIRMWARE AND..."
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
srv_heupstream
<srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
Yingjoe Chen
<yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
Subject: Re: [PATCH v6 1/3] dt-bindings: pwm: add MediaTek display PWM bindings
Date: Fri, 24 Jul 2015 14:56:47 +0200 [thread overview]
Message-ID: <10346793.XDV1BVipfn@ubix> (raw)
In-Reply-To: <CAGS+omAzhfCAoGArK8XUMhNFuqiErPcvGh0q=+Se1dHeBnJyUg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On Friday, July 24, 2015 05:00:13 PM Daniel Kurtz wrote:
> On Fri, Jul 24, 2015 at 4:40 PM, Matthias Brugger
>
> <matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> > On Monday, July 20, 2015 04:17:15 PM YH Huang wrote:
> >> Document the device-tree binding of MediatTek display PWM.
> >> The PWM has one channel to control the backlight brightness for display.
> >> It supports MT8173 and MT6595.
> >>
> >> Signed-off-by: YH Huang <yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
> >> ---
> >>
> >> .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 42
> >>
> >> ++++++++++++++++++++++ 1 file changed, 42 insertions(+)
> >>
> >> create mode 100644
> >> Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >>
> >> diff --git a/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt new file mode
> >> 100644
> >> index 0000000..f8f59ba
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> @@ -0,0 +1,42 @@
> >> +MediaTek display PWM controller
> >> +
> >> +Required properties:
> >> + - compatible: should be "mediatek,<name>-disp-pwm":
> >> + - "mediatek,mt8173-disp-pwm": found on mt8173 SoC.
> >> + - "mediatek,mt6595-disp-pwm": found on mt6595 SoC.
> >
> > I had another look on the mt6589 datasheet and for me it doesn't look like
> > as if this drivers is compatible to mt6589.
>
> Matthias - the compatible is "mt6595", not mt6589 :-).
> Which datasheet did you check?
>
In further versions the driver stated it support mt6589, so I didn' realize
that it is mt6595 now. Logically I checked mt6589 datasheet... :(
Sorry for the noise!
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: matthias.bgg@gmail.com (Matthias Brugger)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 1/3] dt-bindings: pwm: add MediaTek display PWM bindings
Date: Fri, 24 Jul 2015 14:56:47 +0200 [thread overview]
Message-ID: <10346793.XDV1BVipfn@ubix> (raw)
In-Reply-To: <CAGS+omAzhfCAoGArK8XUMhNFuqiErPcvGh0q=+Se1dHeBnJyUg@mail.gmail.com>
On Friday, July 24, 2015 05:00:13 PM Daniel Kurtz wrote:
> On Fri, Jul 24, 2015 at 4:40 PM, Matthias Brugger
>
> <matthias.bgg@gmail.com> wrote:
> > On Monday, July 20, 2015 04:17:15 PM YH Huang wrote:
> >> Document the device-tree binding of MediatTek display PWM.
> >> The PWM has one channel to control the backlight brightness for display.
> >> It supports MT8173 and MT6595.
> >>
> >> Signed-off-by: YH Huang <yh.huang@mediatek.com>
> >> ---
> >>
> >> .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 42
> >>
> >> ++++++++++++++++++++++ 1 file changed, 42 insertions(+)
> >>
> >> create mode 100644
> >> Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >>
> >> diff --git a/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt new file mode
> >> 100644
> >> index 0000000..f8f59ba
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> @@ -0,0 +1,42 @@
> >> +MediaTek display PWM controller
> >> +
> >> +Required properties:
> >> + - compatible: should be "mediatek,<name>-disp-pwm":
> >> + - "mediatek,mt8173-disp-pwm": found on mt8173 SoC.
> >> + - "mediatek,mt6595-disp-pwm": found on mt6595 SoC.
> >
> > I had another look on the mt6589 datasheet and for me it doesn't look like
> > as if this drivers is compatible to mt6589.
>
> Matthias - the compatible is "mt6595", not mt6589 :-).
> Which datasheet did you check?
>
In further versions the driver stated it support mt6589, so I didn' realize
that it is mt6595 now. Logically I checked mt6589 datasheet... :(
Sorry for the noise!
WARNING: multiple messages have this Message-ID (diff)
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Daniel Kurtz <djkurtz@chromium.org>
Cc: YH Huang <yh.huang@mediatek.com>,
Mark Rutland <mark.rutland@arm.com>,
Thierry Reding <thierry.reding@gmail.com>,
Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
linux-pwm@vger.kernel.org,
"open list:OPEN FIRMWARE AND..." <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
srv_heupstream <srv_heupstream@mediatek.com>,
linux-mediatek@lists.infradead.org,
Sascha Hauer <kernel@pengutronix.de>,
Yingjoe Chen <yingjoe.chen@mediatek.com>
Subject: Re: [PATCH v6 1/3] dt-bindings: pwm: add MediaTek display PWM bindings
Date: Fri, 24 Jul 2015 14:56:47 +0200 [thread overview]
Message-ID: <10346793.XDV1BVipfn@ubix> (raw)
In-Reply-To: <CAGS+omAzhfCAoGArK8XUMhNFuqiErPcvGh0q=+Se1dHeBnJyUg@mail.gmail.com>
On Friday, July 24, 2015 05:00:13 PM Daniel Kurtz wrote:
> On Fri, Jul 24, 2015 at 4:40 PM, Matthias Brugger
>
> <matthias.bgg@gmail.com> wrote:
> > On Monday, July 20, 2015 04:17:15 PM YH Huang wrote:
> >> Document the device-tree binding of MediatTek display PWM.
> >> The PWM has one channel to control the backlight brightness for display.
> >> It supports MT8173 and MT6595.
> >>
> >> Signed-off-by: YH Huang <yh.huang@mediatek.com>
> >> ---
> >>
> >> .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 42
> >>
> >> ++++++++++++++++++++++ 1 file changed, 42 insertions(+)
> >>
> >> create mode 100644
> >> Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >>
> >> diff --git a/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt new file mode
> >> 100644
> >> index 0000000..f8f59ba
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> >> @@ -0,0 +1,42 @@
> >> +MediaTek display PWM controller
> >> +
> >> +Required properties:
> >> + - compatible: should be "mediatek,<name>-disp-pwm":
> >> + - "mediatek,mt8173-disp-pwm": found on mt8173 SoC.
> >> + - "mediatek,mt6595-disp-pwm": found on mt6595 SoC.
> >
> > I had another look on the mt6589 datasheet and for me it doesn't look like
> > as if this drivers is compatible to mt6589.
>
> Matthias - the compatible is "mt6595", not mt6589 :-).
> Which datasheet did you check?
>
In further versions the driver stated it support mt6589, so I didn' realize
that it is mt6595 now. Logically I checked mt6589 datasheet... :(
Sorry for the noise!
next prev parent reply other threads:[~2015-07-24 12:56 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-20 8:17 [PATCH v6 0/3] Add MediaTek display PWM driver YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 8:17 ` [PATCH v6 1/3] dt-bindings: pwm: add MediaTek display PWM bindings YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 8:17 ` YH Huang
[not found] ` <1437380237-28961-2-git-send-email-yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-07-24 8:40 ` Matthias Brugger
2015-07-24 8:40 ` Matthias Brugger
2015-07-24 8:40 ` Matthias Brugger
2015-07-24 9:00 ` Daniel Kurtz
2015-07-24 9:00 ` Daniel Kurtz
[not found] ` <CAGS+omAzhfCAoGArK8XUMhNFuqiErPcvGh0q=+Se1dHeBnJyUg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-07-24 12:56 ` Matthias Brugger [this message]
2015-07-24 12:56 ` Matthias Brugger
2015-07-24 12:56 ` Matthias Brugger
2015-08-17 13:23 ` Thierry Reding
2015-08-17 13:23 ` Thierry Reding
[not found] ` <20150817132358.GN8453-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-08-18 2:23 ` YH Huang
2015-08-18 2:23 ` YH Huang
2015-08-18 2:23 ` YH Huang
[not found] ` <1437380237-28961-1-git-send-email-yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-07-20 8:17 ` [PATCH v6 2/3] pwm: add MediaTek display PWM driver support YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 8:17 ` YH Huang
2015-08-17 13:23 ` Thierry Reding
2015-08-17 13:23 ` Thierry Reding
[not found] ` <20150817132259.GM8453-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-08-18 2:47 ` YH Huang
2015-08-18 2:47 ` YH Huang
2015-08-18 2:47 ` YH Huang
2015-07-20 8:17 ` [PATCH v6 3/3] arm64: dts: mt8173: add MT8173 display PWM driver support node YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 8:17 ` YH Huang
2015-07-20 12:22 ` Daniel Kurtz
2015-07-20 12:22 ` Daniel Kurtz
2015-07-24 8:42 ` [PATCH v6 0/3] Add MediaTek display PWM driver Matthias Brugger
2015-07-24 8:42 ` Matthias Brugger
2015-07-24 9:10 ` YH Huang
2015-07-24 9:10 ` YH Huang
2015-07-24 9:10 ` YH Huang
2015-07-29 3:01 ` YH Huang
2015-07-29 3:01 ` YH Huang
2015-07-29 3:01 ` YH Huang
2015-08-03 6:21 ` YH Huang
2015-08-03 6:21 ` YH Huang
2015-08-03 6:21 ` YH Huang
2015-08-11 3:38 ` Daniel Kurtz
2015-08-11 3:38 ` Daniel Kurtz
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=10346793.XDV1BVipfn@ubix \
--to=matthias.bgg-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=djkurtz-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.