From: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
To: Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>
Cc: Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Maxime Ripard
<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
"linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
linux-arm-kernel
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
linux-kernel
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
linux-sunxi <linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>
Subject: Re: [PATCH 2/3] pwm: sunxi: Add A64 support
Date: Sat, 25 Feb 2017 17:23:09 +0800 [thread overview]
Message-ID: <870861488014589@web9m.yandex.ru> (raw)
In-Reply-To: <CAGb2v67OaUps1yffyZ8pBvMwbAU+6WC2pPkt25e5i+SYU1nrDw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
25.02.2017, 16:53, "Chen-Yu Tsai" <wens-jdAy2FN1RRM@public.gmane.org>:
> On Sat, Feb 25, 2017 at 3:42 PM, Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> wrote:
>> A64 PWM controller has same register layout as sun4i driver, so it works
>> by adding A64 specific data.
>>
>> Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
>> ---
>> drivers/pwm/pwm-sun4i.c | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/drivers/pwm/pwm-sun4i.c b/drivers/pwm/pwm-sun4i.c
>> index 1284ffa05921..ca3431e03fb8 100644
>> --- a/drivers/pwm/pwm-sun4i.c
>> +++ b/drivers/pwm/pwm-sun4i.c
>> @@ -290,6 +290,12 @@ static const struct sun4i_pwm_data sun4i_pwm_data_h3 = {
>> .npwm = 1,
>> };
>>
>> +static const struct sun4i_pwm_data sun4i_pwm_data_a64 = {
>> + .has_prescaler_bypass = true,
>> + .has_rdy = true,
>> + .npwm = 1,
>> +};
>> +
>
> This seems to be tha same as the H3. Any differences between the
> H3 and A64? Otherwise please just use that compatible.
H3 is also same as A13...
See http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486595.html
>
> ChenYu
>
>> static const struct of_device_id sun4i_pwm_dt_ids[] = {
>> {
>> .compatible = "allwinner,sun4i-a10-pwm",
>> @@ -307,6 +313,9 @@ static const struct of_device_id sun4i_pwm_dt_ids[] = {
>> .compatible = "allwinner,sun8i-h3-pwm",
>> .data = &sun4i_pwm_data_h3,
>> }, {
>> + .compatible = "allwinner,sun50i-a64-pwm",
>> + .data = &sun4i_pwm_data_a64,
>> + }, {
>> /* sentinel */
>> },
>> };
>> --
>> 2.11.1
>>
>> --
>> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
>> For more options, visit https://groups.google.com/d/optout.
--
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.
next prev parent reply other threads:[~2017-02-25 9:23 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-25 7:42 [PATCH 1/3] dt-bindings: add compatible for the PWM controller on Allwinner A64 Icenowy Zheng
[not found] ` <20170225074216.64868-1-icenowy-ymACFijhrKM@public.gmane.org>
2017-02-25 7:42 ` [PATCH 2/3] pwm: sunxi: Add A64 support Icenowy Zheng
[not found] ` <20170225074216.64868-2-icenowy-ymACFijhrKM@public.gmane.org>
2017-02-25 8:53 ` Chen-Yu Tsai
[not found] ` <CAGb2v67OaUps1yffyZ8pBvMwbAU+6WC2pPkt25e5i+SYU1nrDw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-02-25 9:23 ` Icenowy Zheng [this message]
[not found] ` <870861488014589-IyhJ2jtAr7tuio3avFS2gg@public.gmane.org>
2017-02-25 13:00 ` Siarhei Volkau
2017-02-27 7:44 ` Maxime Ripard
2017-02-25 7:42 ` [PATCH 3/3] arm64: dts: allwinner: add pwm device node for A64 Icenowy Zheng
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=870861488014589@web9m.yandex.ru \
--to=icenowy-ymacfijhrkm@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
--cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=wens-jdAy2FN1RRM@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox