mbox series

[v2,00/11] arm64: Realtek RTD1295 reset controllers

Message ID 20191023101317.26656-1-afaerber@suse.de
Headers show
Series arm64: Realtek RTD1295 reset controllers | expand

Message

Andreas Färber Oct. 23, 2019, 10:13 a.m. UTC
Hello,

This series adds reset controllers for the Realtek RTD1295 and RTD1195 SoCs.

v2 adopts reset-simple driver and DesignWare bindings as simplification
and covers RTD1195, too.

Note that reset-simple driver would allow to cover RTD1195's reset1-3 in one
DT node, but it only maps the first resource, so RTD1295's reset4 would need
to remain separate due to a gap in between. I've therefore left them all as
separate nodes for now.

Also note that my initial 32-bit arm patch already selects RESET_CONTROLLER,
to avoid needing a separate patch here to add that one line as done for arm64.

If I can take the bindings patches through the Realtek tree then I can squash
the two final DT patches depending on them into the patches added the resets,
otherwise they need to go into v5.6 or be merged via a topic branch.

More experimental patches at:
https://github.com/afaerber/linux/commits/rtd1295-next

Have a lot of fun!

Cheers,
Andreas

v1 -> v2:
* Drop custom reset driver
* Drop "realtek,rtd1295-reset" binding
* Reordered to not depend on irqchip or clk patches
* Extended with RTD1195 patches

Cc: Philipp Zabel <p.zabel@pengutronix.de>
Cc: devicetree@vger.kernel.org

Andreas Färber (11):
  dt-bindings: reset: Add Realtek RTD1295
  dt-bindings: reset: Add Realtek RTD1195
  reset: simple: Keep alphabetical order
  reset: simple: Add Realtek RTD1195/RTD1295
  arm64: realtek: Select reset controller
  arm64: dts: realtek: Add RTD129x reset controller nodes
  arm64: dts: realtek: Add RTD129x UART resets
  ARM: dts: rtd1195: Add reset nodes
  ARM: dts: rtd1195: Add UART resets
  arm64: dts: realtek: Adopt RTD129x reset constants
  ARM: dts: rtd1195: Adopt reset constants

 arch/arm/boot/dts/rtd1195.dtsi              |  27 +++++++
 arch/arm64/Kconfig.platforms                |   1 +
 arch/arm64/boot/dts/realtek/rtd129x.dtsi    |  34 +++++++++
 drivers/reset/Kconfig                       |   5 +-
 include/dt-bindings/reset/realtek,rtd1195.h |  74 +++++++++++++++++++
 include/dt-bindings/reset/realtek,rtd1295.h | 111 ++++++++++++++++++++++++++++
 6 files changed, 250 insertions(+), 2 deletions(-)
 create mode 100644 include/dt-bindings/reset/realtek,rtd1195.h
 create mode 100644 include/dt-bindings/reset/realtek,rtd1295.h

-- 
2.16.4

Comments

Rob Herring (Arm) Oct. 26, 2019, 12:16 a.m. UTC | #1
On Wed, 23 Oct 2019 12:13:08 +0200, =?UTF-8?q?Andreas=20F=C3=A4rber?= wrote:
> Add a header with symbolic reset indices for Realtek RTD1195 SoC.

> Naming was derived from BSP register description headers.

> 

> Signed-off-by: Andreas Färber <afaerber@suse.de>

> ---

>  v2: New

>  

>  include/dt-bindings/reset/realtek,rtd1195.h | 74 +++++++++++++++++++++++++++++

>  1 file changed, 74 insertions(+)

>  create mode 100644 include/dt-bindings/reset/realtek,rtd1195.h

> 


Reviewed-by: Rob Herring <robh@kernel.org>
Andreas Färber Oct. 29, 2019, 5:25 a.m. UTC | #2
Am 23.10.19 um 12:13 schrieb Andreas Färber:
> Add a header with symbolic reset indices for Realtek RTD1295 SoC.

> Naming was derived from reset-names in an OEM's Device Tree.

> 

> Acked-by: Rob Herring <robh@kernel.org>

> [AF: Dropped RTD1295 specific binding definition, updated SPDX]

> Signed-off-by: Andreas Färber <afaerber@suse.de>

> ---

>  v1 -> v2:

>  * Dropped textual binding with new compatible

>  * Updated SPDX-License-Identifier location

>  * Updated to SPDX 2.0

>  * Changed from MIT to BSD (Rob)

>  

>  include/dt-bindings/reset/realtek,rtd1295.h | 111 ++++++++++++++++++++++++++++

>  1 file changed, 111 insertions(+)

>  create mode 100644 include/dt-bindings/reset/realtek,rtd1295.h


Added Philipp's Acked-by and applied to linux-realtek.git v5.5/dt64:

https://git.kernel.org/pub/scm/linux/kernel/git/afaerber/linux-realtek.git/log/?h=v5.5/dt64

Thanks,
Andreas

-- 
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)
Andreas Färber Oct. 29, 2019, 5:28 a.m. UTC | #3
Am 23.10.19 um 12:13 schrieb Andreas Färber:
> Associate the UART nodes with the corresponding reset controller bits.

> 

> Signed-off-by: Andreas Färber <afaerber@suse.de>

> ---

>  v1 -> v2:

>  * Rebased, moved from rtd1295.dtsi to rtd129x.dtsi

>  

>  arch/arm64/boot/dts/realtek/rtd129x.dtsi | 3 +++

>  1 file changed, 3 insertions(+)


Squashed the symbolic names and applied to linux-realtek.git v5.5/dt64:

https://git.kernel.org/pub/scm/linux/kernel/git/afaerber/linux-realtek.git/log/?h=v5.5/dt64

Thanks,
Andreas

-- 
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)
Andreas Färber Oct. 29, 2019, 5:29 a.m. UTC | #4
Am 23.10.19 um 12:13 schrieb Andreas Färber:
> Replace reset controller indices with constants.

> 

> Signed-off-by: Andreas Färber <afaerber@suse.de>

> ---

>  v1 -> v2: Unchanged

>  

>  arch/arm64/boot/dts/realtek/rtd129x.dtsi | 7 ++++---

>  1 file changed, 4 insertions(+), 3 deletions(-)


Squashed into preceding arm64 patch and applied to linux-realtek.git
v5.5/dt64:

https://git.kernel.org/pub/scm/linux/kernel/git/afaerber/linux-realtek.git/log/?h=v5.5/dt64

Thanks,
Andreas

-- 
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)