From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: p.zabel@pengutronix.de
Cc: mturquette@baylibre.com, sboyd@kernel.org, afaerber@suse.de,
robh+dt@kernel.org, linux-clk@vger.kernel.org,
liuwei@actions-semi.com, mp-cs@actions-semi.com,
96boards@ucrobotics.com, devicetree@vger.kernel.org,
daniel.thompson@linaro.org, amit.kucheria@linaro.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, hzhang@ucrobotics.com,
bdong@ucrobotics.com, manivannanece23@gmail.com,
thomas.liau@actions-semi.com, jeff.chen@actions-semi.com,
pn@denx.de, edgar.righi@lsitec.org.br, sravanhome@gmail.com
Subject: Re: [PATCH v3 0/9] Add Reset Controller support for Actions Semi Owl SoCs
Date: Thu, 30 Aug 2018 08:49:33 +0530 [thread overview]
Message-ID: <20180830031933.GD8565@Mani-XPS-13-9360> (raw)
In-Reply-To: <20180810095113.25292-1-manivannan.sadhasivam@linaro.org>
On Fri, Aug 10, 2018 at 03:21:04PM +0530, Manivannan Sadhasivam wrote:
> This patchset adds Reset Controller (RMU) support for Actions Semi
> Owl SoCs, S900 and S700. For the Owl SoCs, RMU has been integrated into
> the clock subsystem in hardware. Hence, in software we integrate RMU
> support into common clock driver inorder to maintain compatibility. The
> syscon approach is not taken into account based on the review by Rob.
>
> This patch series depends on the recently posted S700 clk series:
> "[PATCH v7 0/5] Add clock driver for Actions S700 SoC". For the S700 clk
> series, driver and bindings patches are applied through the clk tree.
> But the DTS patches are not yet picked up by the platform maintainer,
> Andreas.
>
> Hence, Andreas is expected to pick the DTS patches in this series once
> reviewed by the maintainers along with S700 clk DTS patches.
>
> Because of the absence of the S500 SoC clk support, the reset controller
> registration code is added to both S700 and S900 SoC clk drivers for now.
> But once S500 clk support is added, the reset controller registration part
> will be moved to Owl SoCs common clk code.
>
Hi Pavel,
Just a ping on this patchset!
Thanks,
Mani
> Thanks,
> Mani
>
> Changes in v3:
>
> * Reverted back the syscon approach based on the review by Rob Herring
> * Dropped the MAINTAINERS patch
> * Converted reset_map struct definitions to const based on the review by
> Philipp
> * Fixed S700 header in DT spotted by Parthiban
> * Added Reviewed-by tags from Rob for 3 bindings patches
>
> Changes in v2:
>
> * Converted the CMU and RMU drivers to syscon for a more cleaner
> approach
> * Declared the owl_reset_map structs to const
> * Used regmap_update_bits instead of a combined regmap_read and write
> * Removed unused headers in RMU drivers
> * Added MAINTAINERS entry for the RMU driver and bindings
>
> Manivannan Sadhasivam (9):
> clk: actions: Cache regmap info in private clock descriptor
> dt-bindings: clock: Add reset controller bindings for Actions Semi Owl
> SoCs
> dt-bindings: reset: Add binding constants for Actions Semi S700 RMU
> dt-bindings: reset: Add binding constants for Actions Semi S900 RMU
> arm64: dts: actions: Add Reset Controller support for S700 SoC
> arm64: dts: actions: Add Reset Controller support for S900 SoC
> clk: actions: Add Actions Semi Owl SoCs Reset Management Unit support
> clk: actions: Add Actions Semi S700 SoC Reset Management Unit support
> clk: actions: Add Actions Semi S900 SoC Reset Management Unit support
>
> .../bindings/clock/actions,owl-cmu.txt | 2 +
> arch/arm64/boot/dts/actions/s700.dtsi | 2 +
> arch/arm64/boot/dts/actions/s900.dtsi | 2 +
> drivers/clk/actions/Kconfig | 1 +
> drivers/clk/actions/Makefile | 1 +
> drivers/clk/actions/owl-common.c | 3 +-
> drivers/clk/actions/owl-common.h | 5 +-
> drivers/clk/actions/owl-reset.c | 66 ++++++++++++++
> drivers/clk/actions/owl-reset.h | 31 +++++++
> drivers/clk/actions/owl-s700.c | 55 +++++++++++-
> drivers/clk/actions/owl-s900.c | 86 ++++++++++++++++++-
> .../dt-bindings/reset/actions,s700-reset.h | 34 ++++++++
> .../dt-bindings/reset/actions,s900-reset.h | 65 ++++++++++++++
> 13 files changed, 347 insertions(+), 6 deletions(-)
> create mode 100644 drivers/clk/actions/owl-reset.c
> create mode 100644 drivers/clk/actions/owl-reset.h
> create mode 100644 include/dt-bindings/reset/actions,s700-reset.h
> create mode 100644 include/dt-bindings/reset/actions,s900-reset.h
>
> --
> 2.17.1
>
prev parent reply other threads:[~2018-08-30 3:19 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-10 9:51 [PATCH v3 0/9] Add Reset Controller support for Actions Semi Owl SoCs Manivannan Sadhasivam
2018-08-10 9:51 ` [PATCH v3 1/9] clk: actions: Cache regmap info in private clock descriptor Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 2/9] dt-bindings: clock: Add reset controller bindings for Actions Semi Owl SoCs Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 3/9] dt-bindings: reset: Add binding constants for Actions Semi S700 RMU Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 4/9] dt-bindings: reset: Add binding constants for Actions Semi S900 RMU Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 5/9] arm64: dts: actions: Add Reset Controller support for S700 SoC Manivannan Sadhasivam
2018-08-10 9:51 ` [PATCH v3 6/9] arm64: dts: actions: Add Reset Controller support for S900 SoC Manivannan Sadhasivam
2018-10-16 21:42 ` Stephen Boyd
2018-10-17 2:56 ` Manivannan Sadhasivam
2018-08-10 9:51 ` [PATCH v3 7/9] clk: actions: Add Actions Semi Owl SoCs Reset Management Unit support Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 8/9] clk: actions: Add Actions Semi S700 SoC " Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-10 9:51 ` [PATCH v3 9/9] clk: actions: Add Actions Semi S900 " Manivannan Sadhasivam
2018-10-17 6:02 ` Stephen Boyd
2018-08-30 3:19 ` Manivannan Sadhasivam [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=20180830031933.GD8565@Mani-XPS-13-9360 \
--to=manivannan.sadhasivam@linaro.org \
--cc=96boards@ucrobotics.com \
--cc=afaerber@suse.de \
--cc=amit.kucheria@linaro.org \
--cc=bdong@ucrobotics.com \
--cc=daniel.thompson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=edgar.righi@lsitec.org.br \
--cc=hzhang@ucrobotics.com \
--cc=jeff.chen@actions-semi.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=liuwei@actions-semi.com \
--cc=manivannanece23@gmail.com \
--cc=mp-cs@actions-semi.com \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=pn@denx.de \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=sravanhome@gmail.com \
--cc=thomas.liau@actions-semi.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 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).