From mboxrd@z Thu Jan 1 00:00:00 1970 From: matthias.bgg@gmail.com (Matthias Brugger) Date: Fri, 24 Jul 2015 14:56:47 +0200 Subject: [PATCH v6 1/3] dt-bindings: pwm: add MediaTek display PWM bindings In-Reply-To: References: <1437380237-28961-1-git-send-email-yh.huang@mediatek.com> <8769951.Y5Edyg2WDJ@ubix> Message-ID: <10346793.XDV1BVipfn@ubix> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Friday, July 24, 2015 05:00:13 PM Daniel Kurtz wrote: > On Fri, Jul 24, 2015 at 4:40 PM, Matthias Brugger > > 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 > >> --- > >> > >> .../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,-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!