From: Rob Herring <robh@kernel.org>
To: Billy Tsai <billy_tsai@aspeedtech.com>
Cc: lee.jones@linaro.org, joel@jms.id.au, andrew@aj.id.au,
thierry.reding@gmail.com, u.kleine-koenig@pengutronix.de,
p.zabel@pengutronix.de, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org,
linux-pwm@vger.kernel.org, BMC-SW@aspeedtech.com
Subject: Re: [v3 0/2] Support pwm driver for aspeed ast26xx
Date: Thu, 6 May 2021 20:06:01 -0500 [thread overview]
Message-ID: <20210507010601.GA1110621@robh.at.kernel.org> (raw)
In-Reply-To: <20210506061050.8001-1-billy_tsai@aspeedtech.com>
On Thu, May 06, 2021 at 02:10:48PM +0800, Billy Tsai wrote:
> The legacy driver of aspeed pwm is binding with tach controller and it
> doesn't follow the pwm framworks usage. In addition, the pwm register
> usage of the 6th generation of ast26xx has drastic change. So these
> patch serials add the new aspeed pwm driver to fix up the problem above.
>
> Changes since v2:
> - Remove the tach node, #address-cells and #size-cells from pwm-tach.yaml
So you don't need tach functions, they don't need it to be described in
DT any more, or you plan to add them back later? The last one is not
acceptable.
> - Add clocks and reset properties to pwm-tach.yaml
> - Kconfig/Makfile sorted alphabetically
> - pwm-aspeed-g6.c suggested by Uwe Kleine-König
> - Add more hardware descriptions at top of the driver.
> - Remove unused api request and free
> - Move the initialize settings of all pwm channel to probe.
> - Change the method of getting the approximate period.
> - Read the hardware register values to fill the state for .get_state()
>
> Changes since v1:
> - Fix the dt_binding_check fail suggested by Rob Herring
> - Add depends to PWM_ASPEED_G6 configure suggested by Uwe Kleine-Konig
> - pwm-aspeed-g6.c suggested by Uwe Kleine-König
> - Fix license header
> - Use bitfiled.h macro to define register fields
> - Implement .remove device function
> - Implement .get_state pwm api
>
>
> Billy Tsai (2):
> dt-bindings: Add bindings for aspeed pwm-tach and pwm.
> pwm: Add Aspeed ast2600 PWM support
>
> .../bindings/mfd/aspeed,ast2600-pwm-tach.yaml | 55 +++
> .../bindings/pwm/aspeed,ast2600-pwm.yaml | 41 ++
> drivers/pwm/Kconfig | 8 +
> drivers/pwm/Makefile | 1 +
> drivers/pwm/pwm-aspeed-g6.c | 368 ++++++++++++++++++
> 5 files changed, 473 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/aspeed,ast2600-pwm-tach.yaml
> create mode 100644 Documentation/devicetree/bindings/pwm/aspeed,ast2600-pwm.yaml
> create mode 100644 drivers/pwm/pwm-aspeed-g6.c
>
> --
> 2.25.1
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2021-05-07 1:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-06 6:10 [v3 0/2] Support pwm driver for aspeed ast26xx Billy Tsai
2021-05-06 6:10 ` [v3 1/2] dt-bindings: Add bindings for aspeed pwm-tach and pwm Billy Tsai
2021-05-07 1:08 ` Rob Herring
2021-05-06 6:10 ` [v3 2/2] pwm: Add Aspeed ast2600 PWM support Billy Tsai
2021-05-07 1:06 ` Rob Herring [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=20210507010601.GA1110621@robh.at.kernel.org \
--to=robh@kernel.org \
--cc=BMC-SW@aspeedtech.com \
--cc=andrew@aj.id.au \
--cc=billy_tsai@aspeedtech.com \
--cc=devicetree@vger.kernel.org \
--cc=joel@jms.id.au \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=thierry.reding@gmail.com \
--cc=u.kleine-koenig@pengutronix.de \
/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;
as well as URLs for NNTP newsgroup(s).