2020-01-16 07:41:34

by Horia Geanta

Subject: [PATCH v2 0/3] clk: imx8mn: add snvs clock

v2: add commmit message for trivial patch 1/3

This patch set adds the clock for snvs module on imx8mn.
DT bindings, clk driver are updated accordingly.
DT for imx8mn (snvs-rtc-lp node) is also updated.

Horia Geantă (3):
dt-bindings: clock: imx8mn: add SNVS clock
clk: imx8mn: add SNVS clock to clock tree
arm64: dts: imx8mn: add clock for snvs rtc node

arch/arm64/boot/dts/freescale/imx8mn.dtsi | 1 +
drivers/clk/imx/clk-imx8mn.c | 1 +
include/dt-bindings/clock/imx8mn-clock.h | 4 +++-
3 files changed, 5 insertions(+), 1 deletion(-)

--
2.17.1

2020-01-16 08:41:55

by Horia Geanta

Subject: [PATCH v2 1/3] dt-bindings: clock: imx8mn: add SNVS clock

Add macro for the SNVS clock of the i.MX8MN.

Signed-off-by: Horia Geantă <[email protected]>
---
include/dt-bindings/clock/imx8mn-clock.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/include/dt-bindings/clock/imx8mn-clock.h b/include/dt-bindings/clock/imx8mn-clock.h
index 0f2b8423ce1d..43883fe65db5 100644
--- a/include/dt-bindings/clock/imx8mn-clock.h
+++ b/include/dt-bindings/clock/imx8mn-clock.h
@@ -228,6 +228,8 @@
#define IMX8MN_SYS_PLL2_333M_CG 209
#define IMX8MN_SYS_PLL2_500M_CG 210

-#define IMX8MN_CLK_END 211
+#define IMX8MN_CLK_SNVS_ROOT 211
+
+#define IMX8MN_CLK_END 212

#endif
--
2.17.1

2020-01-16 08:41:57

by Horia Geanta

Subject: [PATCH v2 3/3] arm64: dts: imx8mn: add clock for snvs rtc node

Initial commit adding imx8mn support:
6c3debcbae47 ("arm64: dts: freescale: Add i.MX8MN dtsi support")
added the "clock-names" property for the snvs rtc node,
however it missed adding the clock.

Signed-off-by: Horia Geantă <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index a44b5438e842..d23347aaedda 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -358,6 +358,7 @@
offset = <0x34>;
interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&clk IMX8MN_CLK_SNVS_ROOT>;
clock-names = "snvs-rtc";
};

--
2.17.1

2020-01-22 01:50:39

by Rob Herring (Arm)

Subject: Re: [PATCH v2 1/3] dt-bindings: clock: imx8mn: add SNVS clock

On Thu, 16 Jan 2020 09:37:16 +0200, =?UTF-8?q?Horia=20Geant=C4=83?= wrote:
> Add macro for the SNVS clock of the i.MX8MN.
>
> Signed-off-by: Horia Geantă <[email protected]>
> ---
> include/dt-bindings/clock/imx8mn-clock.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

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

2020-02-12 21:08:27

by Horia Geanta

Subject: Re: [PATCH v2 0/3] clk: imx8mn: add snvs clock

On 1/16/2020 9:37 AM, Horia Geant? wrote:
> v2: add commmit message for trivial patch 1/3
>
> This patch set adds the clock for snvs module on imx8mn.
> DT bindings, clk driver are updated accordingly.
> DT for imx8mn (snvs-rtc-lp node) is also updated.
>
> Horia Geant? (3):
> dt-bindings: clock: imx8mn: add SNVS clock
> clk: imx8mn: add SNVS clock to clock tree
> arm64: dts: imx8mn: add clock for snvs rtc node
>
> arch/arm64/boot/dts/freescale/imx8mn.dtsi | 1 +
> drivers/clk/imx/clk-imx8mn.c | 1 +
> include/dt-bindings/clock/imx8mn-clock.h | 4 +++-
> 3 files changed, 5 insertions(+), 1 deletion(-)
>
Kind reminder.

Patch set is here:
https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=229047&archive=both&state=*
Let me know if I should resend it.

Thanks,
Horia

2020-02-13 03:27:21

by Shawn Guo

Subject: Re: [PATCH v2 0/3] clk: imx8mn: add snvs clock

On Thu, Jan 16, 2020 at 09:37:15AM +0200, Horia Geantă wrote:
> v2: add commmit message for trivial patch 1/3
>
> This patch set adds the clock for snvs module on imx8mn.
> DT bindings, clk driver are updated accordingly.
> DT for imx8mn (snvs-rtc-lp node) is also updated.
>
> Horia Geantă (3):
> dt-bindings: clock: imx8mn: add SNVS clock
> clk: imx8mn: add SNVS clock to clock tree
> arm64: dts: imx8mn: add clock for snvs rtc node

Applied all, thanks.