From: YH Huang <yh.huang@mediatek.com>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: "Matthias Brugger" <matthias.bgg@gmail.com>,
"Mark Rutland" <mark.rutland@arm.com>,
"linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
srv_heupstream <srv_heupstream@mediatek.com>,
"Pawel Moll" <pawel.moll@arm.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Rob Herring" <robh+dt@kernel.org>,
"linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
"Sascha Hauer" <kernel@pengutronix.de>,
"Yingjoe Chen (陳英洲)" <Yingjoe.Chen@mediatek.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v7 0/3] Add MediaTek display PWM driver
Date: Tue, 15 Sep 2015 11:52:53 +0800 [thread overview]
Message-ID: <1442289173.11445.5.camel@mtksdaap41> (raw)
In-Reply-To: <1439882875-19596-1-git-send-email-yh.huang@mediatek.com>
On Tue, 2015-08-18 at 15:27 +0800, YH Huang wrote:
> This patch series add the use of display PWM driver, documentation
> and device tree for Mediatek SoCs. The driver is used to support
> the backlight of the panel. This is based on v4.2-rc1.
>
> The clock definitions (CLK_MM_DISP_PWM*) are added by James Liao's patch:
> clk: mediatek: Add subsystem clocks of MT8173
>
> Change in v7:
> 1. Add the dependency HAS_IOMEM in Kconfig.
> 2. Revise misprints and coding style for readability.
>
> Change in v6:
> 1. Enable clocks in the pwm_enable function.
> 2. Remove suspend/resume code since pwm-backlight driver has done the
> same things.
> 3. Revise some code to make it easier to read.
>
> Change in v5:
> 1. Configure PWM output via pinctrl.
> 2. Fix the parameter name in dtsi.
>
> Change in v4:
> 1. Codebase is on v4.2-rc1.
> 2. Add the PWM node in dtsi.
> 3. Change the dependency in Kconfig.
> 4. Rewrite some code for readability.
>
> Change in v3:
> 1. Add suspend/resume function.
> 2. Fix the formula for high_width calculation.
> 3. Rewrite some code to make it easier to read.
> 4. Add more information in the commit message.
>
> Change in v2:
> 1. Rewrite descriptions for driver in kconfig to make it much clear.
> 2. Rename the driver from "pwm-mediatek-disp" to "pwm-mtk-disp".
> 3. Disable clocks in the error path.
> 4. Change the shift values in decimal instead of hex.
> 5. Rename and add some variables or function name for consistency and
> readability.
> 6. Drop unnecessary parentheses, spaces, variables and add newlines to
> make the code easiler to read.
>
> YH Huang (3):
> dt-bindings: pwm: add MediaTek display PWM bindings
> pwm: add MediaTek display PWM driver support
> arm64: dts: mt8173: add MT8173 display PWM driver support node
>
> .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 42 ++++
> arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 13 ++
> arch/arm64/boot/dts/mediatek/mt8173.dtsi | 22 ++
> drivers/pwm/Kconfig | 11 +
> drivers/pwm/Makefile | 1 +
> drivers/pwm/pwm-mtk-disp.c | 232 ++++++++++++++++++++
> 6 files changed, 321 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> create mode 100644 drivers/pwm/pwm-mtk-disp.c
>
> --
> 1.7.9.5
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Hi Thierry,
Will you please pick up these patches in your pwm tree?
Should I rebase to 4.3-rc1?
Regards,
YH Huang
WARNING: multiple messages have this Message-ID (diff)
From: yh.huang@mediatek.com (YH Huang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 0/3] Add MediaTek display PWM driver
Date: Tue, 15 Sep 2015 11:52:53 +0800 [thread overview]
Message-ID: <1442289173.11445.5.camel@mtksdaap41> (raw)
In-Reply-To: <1439882875-19596-1-git-send-email-yh.huang@mediatek.com>
On Tue, 2015-08-18 at 15:27 +0800, YH Huang wrote:
> This patch series add the use of display PWM driver, documentation
> and device tree for Mediatek SoCs. The driver is used to support
> the backlight of the panel. This is based on v4.2-rc1.
>
> The clock definitions (CLK_MM_DISP_PWM*) are added by James Liao's patch:
> clk: mediatek: Add subsystem clocks of MT8173
>
> Change in v7:
> 1. Add the dependency HAS_IOMEM in Kconfig.
> 2. Revise misprints and coding style for readability.
>
> Change in v6:
> 1. Enable clocks in the pwm_enable function.
> 2. Remove suspend/resume code since pwm-backlight driver has done the
> same things.
> 3. Revise some code to make it easier to read.
>
> Change in v5:
> 1. Configure PWM output via pinctrl.
> 2. Fix the parameter name in dtsi.
>
> Change in v4:
> 1. Codebase is on v4.2-rc1.
> 2. Add the PWM node in dtsi.
> 3. Change the dependency in Kconfig.
> 4. Rewrite some code for readability.
>
> Change in v3:
> 1. Add suspend/resume function.
> 2. Fix the formula for high_width calculation.
> 3. Rewrite some code to make it easier to read.
> 4. Add more information in the commit message.
>
> Change in v2:
> 1. Rewrite descriptions for driver in kconfig to make it much clear.
> 2. Rename the driver from "pwm-mediatek-disp" to "pwm-mtk-disp".
> 3. Disable clocks in the error path.
> 4. Change the shift values in decimal instead of hex.
> 5. Rename and add some variables or function name for consistency and
> readability.
> 6. Drop unnecessary parentheses, spaces, variables and add newlines to
> make the code easiler to read.
>
> YH Huang (3):
> dt-bindings: pwm: add MediaTek display PWM bindings
> pwm: add MediaTek display PWM driver support
> arm64: dts: mt8173: add MT8173 display PWM driver support node
>
> .../devicetree/bindings/pwm/pwm-mtk-disp.txt | 42 ++++
> arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 13 ++
> arch/arm64/boot/dts/mediatek/mt8173.dtsi | 22 ++
> drivers/pwm/Kconfig | 11 +
> drivers/pwm/Makefile | 1 +
> drivers/pwm/pwm-mtk-disp.c | 232 ++++++++++++++++++++
> 6 files changed, 321 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt
> create mode 100644 drivers/pwm/pwm-mtk-disp.c
>
> --
> 1.7.9.5
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Hi Thierry,
Will you please pick up these patches in your pwm tree?
Should I rebase to 4.3-rc1?
Regards,
YH Huang
next prev parent reply other threads:[~2015-09-15 3:52 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-18 7:27 [PATCH v7 0/3] Add MediaTek display PWM driver YH Huang
2015-08-18 7:27 ` YH Huang
2015-08-18 7:27 ` YH Huang
[not found] ` <1439882875-19596-1-git-send-email-yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-08-18 7:27 ` [PATCH v7 1/3] dt-bindings: pwm: add MediaTek display PWM bindings YH Huang
2015-08-18 7:27 ` YH Huang
2015-08-18 7:27 ` YH Huang
[not found] ` <1439882875-19596-2-git-send-email-yh.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-10-05 15:58 ` Thierry Reding
2015-10-05 15:58 ` Thierry Reding
2015-10-05 15:58 ` Thierry Reding
2015-08-18 7:27 ` [PATCH v7 2/3] pwm: add MediaTek display PWM driver support YH Huang
2015-08-18 7:27 ` YH Huang
2015-08-18 7:27 ` YH Huang
2015-10-05 15:59 ` Thierry Reding
2015-10-05 15:59 ` Thierry Reding
2015-08-18 7:27 ` [PATCH v7 3/3] arm64: dts: mt8173: add MT8173 display PWM driver support node YH Huang
2015-08-18 7:27 ` YH Huang
2015-08-18 7:27 ` YH Huang
2015-09-15 3:52 ` YH Huang [this message]
2015-09-15 3:52 ` [PATCH v7 0/3] Add MediaTek display PWM driver YH Huang
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=1442289173.11445.5.camel@mtksdaap41 \
--to=yh.huang@mediatek.com \
--cc=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=srv_heupstream@mediatek.com \
--cc=thierry.reding@gmail.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 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.