2018-11-25 16:23:34

by Hao Zhang

[permalink] [raw]
Subject: [PATCH v3 5/6] DEV: CLK: sunxi ccu: export clk_apb1 for sun8i-r40 soc pwm.

The clock source for sun8i R40 is from apb1, so export it for
dt parses.

Signed-off-by: Hao Zhang <[email protected]>
---
drivers/clk/sunxi-ng/ccu-sun8i-r40.h | 4 +++-
include/dt-bindings/clock/sun8i-r40-ccu.h | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-r40.h b/drivers/clk/sunxi-ng/ccu-sun8i-r40.h
index db2a124..181ab26 100644
--- a/drivers/clk/sunxi-ng/ccu-sun8i-r40.h
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-r40.h
@@ -51,7 +51,9 @@

#define CLK_AXI 25
#define CLK_AHB1 26
-#define CLK_APB1 27
+
+/* The APB1 clock is exported */
+
#define CLK_APB2 28

/* All the bus gates are exported */
diff --git a/include/dt-bindings/clock/sun8i-r40-ccu.h b/include/dt-bindings/clock/sun8i-r40-ccu.h
index f9e15a2..a2b8f06 100644
--- a/include/dt-bindings/clock/sun8i-r40-ccu.h
+++ b/include/dt-bindings/clock/sun8i-r40-ccu.h
@@ -49,6 +49,8 @@

#define CLK_CPU 24

+#define CLK_APB1 27
+
#define CLK_BUS_MIPI_DSI 29
#define CLK_BUS_CE 30
#define CLK_BUS_DMA 31
--
2.7.4



2018-11-26 20:51:54

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH v3 5/6] DEV: CLK: sunxi ccu: export clk_apb1 for sun8i-r40 soc pwm.

On Mon, Nov 26, 2018 at 12:22:03AM +0800, Hao Zhang wrote:
> +#define CLK_APB1 27

This is already used in patch 2, so the patches must be reordered.

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | http://www.pengutronix.de/ |

2018-12-07 23:26:50

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH v3 5/6] DEV: CLK: sunxi ccu: export clk_apb1 for sun8i-r40 soc pwm.

On Mon, 26 Nov 2018 00:22:03 +0800, Hao Zhang wrote:
> The clock source for sun8i R40 is from apb1, so export it for
> dt parses.
>
> Signed-off-by: Hao Zhang <[email protected]>
> ---
> drivers/clk/sunxi-ng/ccu-sun8i-r40.h | 4 +++-
> include/dt-bindings/clock/sun8i-r40-ccu.h | 2 ++
> 2 files changed, 5 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>