diff mbox series

arm64: dts: sprd: Add clock properties for serial devices

Message ID 1246f7a9ce912458ea3b889b0c0e392897a664c8.1554879978.git.baolin.wang@linaro.org
State Accepted
Commit 15d574fbd3f8ec7705896ed14b74eae482cadd4e
Headers show
Series arm64: dts: sprd: Add clock properties for serial devices | expand

Commit Message

(Exiting) Baolin Wang April 10, 2019, 7:22 a.m. UTC
We've introduced power management logics for the Spreadtrum serial
controller by commit 062ec2774c8a ("serial: sprd: Add power management
for the Spreadtrum serial controller"), thus add related clock properties
to support this feature.

Signed-off-by: Baolin Wang <baolin.wang@linaro.org>

---
 arch/arm64/boot/dts/sprd/whale2.dtsi |   16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

-- 
1.7.9.5

Comments

(Exiting) Baolin Wang May 6, 2019, 6:36 a.m. UTC | #1
Hi Olof,

On Wed, 10 Apr 2019 at 15:23, Baolin Wang <baolin.wang@linaro.org> wrote:
>

> We've introduced power management logics for the Spreadtrum serial

> controller by commit 062ec2774c8a ("serial: sprd: Add power management

> for the Spreadtrum serial controller"), thus add related clock properties

> to support this feature.

>

> Signed-off-by: Baolin Wang <baolin.wang@linaro.org>

> ---


Could you take this patch through the arm-soc tree if no objections
from you? Thanks.

>  arch/arm64/boot/dts/sprd/whale2.dtsi |   16 ++++++++++++----

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

>

> diff --git a/arch/arm64/boot/dts/sprd/whale2.dtsi b/arch/arm64/boot/dts/sprd/whale2.dtsi

> index 34b6ca0..b5c5dce 100644

> --- a/arch/arm64/boot/dts/sprd/whale2.dtsi

> +++ b/arch/arm64/boot/dts/sprd/whale2.dtsi

> @@ -75,7 +75,9 @@

>                                              "sprd,sc9836-uart";

>                                 reg = <0x0 0x100>;

>                                 interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;

> -                               clocks = <&ext_26m>;

> +                               clock-names = "enable", "uart", "source";

> +                               clocks = <&apapb_gate CLK_UART0_EB>,

> +                                      <&ap_clk CLK_UART0>, <&ext_26m>;

>                                 status = "disabled";

>                         };

>

> @@ -84,7 +86,9 @@

>                                              "sprd,sc9836-uart";

>                                 reg = <0x100000 0x100>;

>                                 interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;

> -                               clocks = <&ext_26m>;

> +                               clock-names = "enable", "uart", "source";

> +                               clocks = <&apapb_gate CLK_UART1_EB>,

> +                                      <&ap_clk CLK_UART1>, <&ext_26m>;

>                                 status = "disabled";

>                         };

>

> @@ -93,7 +97,9 @@

>                                              "sprd,sc9836-uart";

>                                 reg = <0x200000 0x100>;

>                                 interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>;

> -                               clocks = <&ext_26m>;

> +                               clock-names = "enable", "uart", "source";

> +                               clocks = <&apapb_gate CLK_UART2_EB>,

> +                                      <&ap_clk CLK_UART2>, <&ext_26m>;

>                                 status = "disabled";

>                         };

>

> @@ -102,7 +108,9 @@

>                                              "sprd,sc9836-uart";

>                                 reg = <0x300000 0x100>;

>                                 interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;

> -                               clocks = <&ext_26m>;

> +                               clock-names = "enable", "uart", "source";

> +                               clocks = <&apapb_gate CLK_UART3_EB>,

> +                                      <&ap_clk CLK_UART3>, <&ext_26m>;

>                                 status = "disabled";

>                         };

>                 };

> --

> 1.7.9.5

>



-- 
Baolin Wang
Best Regards
Olof Johansson May 16, 2019, 9:43 p.m. UTC | #2
On Wed, Apr 10, 2019 at 03:22:50PM +0800, Baolin Wang wrote:
> We've introduced power management logics for the Spreadtrum serial

> controller by commit 062ec2774c8a ("serial: sprd: Add power management

> for the Spreadtrum serial controller"), thus add related clock properties

> to support this feature.

> 

> Signed-off-by: Baolin Wang <baolin.wang@linaro.org>


Applied to arm/late.


-Olof
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/sprd/whale2.dtsi b/arch/arm64/boot/dts/sprd/whale2.dtsi
index 34b6ca0..b5c5dce 100644
--- a/arch/arm64/boot/dts/sprd/whale2.dtsi
+++ b/arch/arm64/boot/dts/sprd/whale2.dtsi
@@ -75,7 +75,9 @@ 
 					     "sprd,sc9836-uart";
 				reg = <0x0 0x100>;
 				interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
-				clocks = <&ext_26m>;
+				clock-names = "enable", "uart", "source";
+				clocks = <&apapb_gate CLK_UART0_EB>,
+				       <&ap_clk CLK_UART0>, <&ext_26m>;
 				status = "disabled";
 			};
 
@@ -84,7 +86,9 @@ 
 					     "sprd,sc9836-uart";
 				reg = <0x100000 0x100>;
 				interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
-				clocks = <&ext_26m>;
+				clock-names = "enable", "uart", "source";
+				clocks = <&apapb_gate CLK_UART1_EB>,
+				       <&ap_clk CLK_UART1>, <&ext_26m>;
 				status = "disabled";
 			};
 
@@ -93,7 +97,9 @@ 
 					     "sprd,sc9836-uart";
 				reg = <0x200000 0x100>;
 				interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>;
-				clocks = <&ext_26m>;
+				clock-names = "enable", "uart", "source";
+				clocks = <&apapb_gate CLK_UART2_EB>,
+				       <&ap_clk CLK_UART2>, <&ext_26m>;
 				status = "disabled";
 			};
 
@@ -102,7 +108,9 @@ 
 					     "sprd,sc9836-uart";
 				reg = <0x300000 0x100>;
 				interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
-				clocks = <&ext_26m>;
+				clock-names = "enable", "uart", "source";
+				clocks = <&apapb_gate CLK_UART3_EB>,
+				       <&ap_clk CLK_UART3>, <&ext_26m>;
 				status = "disabled";
 			};
 		};