* [PATCH] riscv: dts: sophgo: add timer dt node for CV1800
@ 2023-12-28 13:06 AnnanLiu
2023-12-28 13:36 ` Conor Dooley
2023-12-28 13:47 ` Emil Renner Berthing
0 siblings, 2 replies; 3+ messages in thread
From: AnnanLiu @ 2023-12-28 13:06 UTC (permalink / raw)
To: chao.wei, unicorn_wang, robh+dt, krzysztof.kozlowski+dt,
conor+dt, paul.walmsley, palmer, aou
Cc: devicetree, linux-riscv, linux-kernel, AnnanLiu
Add the timer device tree node to CV1800 SoC.
This patch depends on the clk driver and reset driver.
Clk driver link:
https://lore.kernel.org/all/IA1PR20MB49539CDAD9A268CBF6CA184BBB9FA@IA1PR20MB4953.namprd20.prod.outlook.com/
Reset driver link:
https://lore.kernel.org/all/20231113005503.2423-1-jszhang@kernel.org/
Signed-off-by: AnnanLiu <annan.liu.xdu@outlook.com>
---
arch/riscv/boot/dts/sophgo/cv1800b.dtsi | 72 +++++++++++++++++++++++++
1 file changed, 72 insertions(+)
diff --git a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
index aec6401a467b..34a1647cc51b 100644
--- a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
+++ b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
@@ -113,6 +113,78 @@ plic: interrupt-controller@70000000 {
riscv,ndev = <101>;
};
+ timer0: timer@030a0000 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0000 0x14>;
+ interrupts = <79 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER0>;
+ status = "disabled";
+ };
+
+ timer1: timer@030a0014 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0014 0x14>;
+ interrupts = <80 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER1>;
+ status = "disabled";
+ };
+
+ timer2: timer@030a0028 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0028 0x14>;
+ interrupts = <81 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER2>;
+ status = "disabled";
+ };
+
+ timer3: timer@030a003c {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a003c 0x14>;
+ interrupts = <82 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER3>;
+ status = "disabled";
+ };
+
+ timer4: timer@030a0050 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0050 0x14>;
+ interrupts = <83 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER4>;
+ status = "disabled";
+ };
+
+ timer5: timer@30a0064 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0064 0x14>;
+ interrupts = <84 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER5>;
+ status = "disabled";
+ };
+
+ timer6: timer@030a0078 {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a0078 0x14>;
+ interrupts = <85 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER6>;
+ status = "disabled";
+ };
+
+ timer7: timer@030a008c {
+ compatible = "snps,dw-apb-timer";
+ reg = <0x030a008c 0x14>;
+ interrupts = <86 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&osc>;
+ resets = <&rst RST_TIMER7>;
+ status = "disabled";
+ };
+
clint: timer@74000000 {
compatible = "sophgo,cv1800b-clint", "thead,c900-clint";
reg = <0x74000000 0x10000>;
--
2.34.1
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] riscv: dts: sophgo: add timer dt node for CV1800
2023-12-28 13:06 [PATCH] riscv: dts: sophgo: add timer dt node for CV1800 AnnanLiu
@ 2023-12-28 13:36 ` Conor Dooley
2023-12-28 13:47 ` Emil Renner Berthing
1 sibling, 0 replies; 3+ messages in thread
From: Conor Dooley @ 2023-12-28 13:36 UTC (permalink / raw)
To: AnnanLiu
Cc: chao.wei, unicorn_wang, robh+dt, krzysztof.kozlowski+dt,
conor+dt, paul.walmsley, palmer, aou, devicetree, linux-riscv,
linux-kernel
[-- Attachment #1.1: Type: text/plain, Size: 3228 bytes --]
On Thu, Dec 28, 2023 at 09:06:54PM +0800, AnnanLiu wrote:
> Add the timer device tree node to CV1800 SoC.
> This patch depends on the clk driver and reset driver.
> Clk driver link:
> https://lore.kernel.org/all/IA1PR20MB49539CDAD9A268CBF6CA184BBB9FA@IA1PR20MB4953.namprd20.prod.outlook.com/
> Reset driver link:
> https://lore.kernel.org/all/20231113005503.2423-1-jszhang@kernel.org/
FYI, this stuff should be under the --- line.
If there's nothing else wrong with this commit, I can fix this while
applying.
Cheers,
Conor.
>
> Signed-off-by: AnnanLiu <annan.liu.xdu@outlook.com>
> ---
> arch/riscv/boot/dts/sophgo/cv1800b.dtsi | 72 +++++++++++++++++++++++++
> 1 file changed, 72 insertions(+)
>
> diff --git a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> index aec6401a467b..34a1647cc51b 100644
> --- a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> +++ b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> @@ -113,6 +113,78 @@ plic: interrupt-controller@70000000 {
> riscv,ndev = <101>;
> };
>
> + timer0: timer@030a0000 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0000 0x14>;
> + interrupts = <79 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER0>;
> + status = "disabled";
> + };
> +
> + timer1: timer@030a0014 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0014 0x14>;
> + interrupts = <80 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER1>;
> + status = "disabled";
> + };
> +
> + timer2: timer@030a0028 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0028 0x14>;
> + interrupts = <81 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER2>;
> + status = "disabled";
> + };
> +
> + timer3: timer@030a003c {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a003c 0x14>;
> + interrupts = <82 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER3>;
> + status = "disabled";
> + };
> +
> + timer4: timer@030a0050 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0050 0x14>;
> + interrupts = <83 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER4>;
> + status = "disabled";
> + };
> +
> + timer5: timer@30a0064 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0064 0x14>;
> + interrupts = <84 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER5>;
> + status = "disabled";
> + };
> +
> + timer6: timer@030a0078 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0078 0x14>;
> + interrupts = <85 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER6>;
> + status = "disabled";
> + };
> +
> + timer7: timer@030a008c {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a008c 0x14>;
> + interrupts = <86 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER7>;
> + status = "disabled";
> + };
> +
> clint: timer@74000000 {
> compatible = "sophgo,cv1800b-clint", "thead,c900-clint";
> reg = <0x74000000 0x10000>;
> --
> 2.34.1
>
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
[-- Attachment #2: Type: text/plain, Size: 161 bytes --]
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] riscv: dts: sophgo: add timer dt node for CV1800
2023-12-28 13:06 [PATCH] riscv: dts: sophgo: add timer dt node for CV1800 AnnanLiu
2023-12-28 13:36 ` Conor Dooley
@ 2023-12-28 13:47 ` Emil Renner Berthing
1 sibling, 0 replies; 3+ messages in thread
From: Emil Renner Berthing @ 2023-12-28 13:47 UTC (permalink / raw)
To: AnnanLiu, chao.wei, unicorn_wang, robh+dt,
krzysztof.kozlowski+dt, conor+dt, paul.walmsley, palmer, aou
Cc: devicetree, linux-riscv, linux-kernel
AnnanLiu wrote:
> Add the timer device tree node to CV1800 SoC.
> This patch depends on the clk driver and reset driver.
> Clk driver link:
> https://lore.kernel.org/all/IA1PR20MB49539CDAD9A268CBF6CA184BBB9FA@IA1PR20MB4953.namprd20.prod.outlook.com/
> Reset driver link:
> https://lore.kernel.org/all/20231113005503.2423-1-jszhang@kernel.org/
>
> Signed-off-by: AnnanLiu <annan.liu.xdu@outlook.com>
> ---
> arch/riscv/boot/dts/sophgo/cv1800b.dtsi | 72 +++++++++++++++++++++++++
> 1 file changed, 72 insertions(+)
>
> diff --git a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> index aec6401a467b..34a1647cc51b 100644
> --- a/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> +++ b/arch/riscv/boot/dts/sophgo/cv1800b.dtsi
> @@ -113,6 +113,78 @@ plic: interrupt-controller@70000000 {
> riscv,ndev = <101>;
> };
>
> + timer0: timer@030a0000 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0000 0x14>;
> + interrupts = <79 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER0>;
> + status = "disabled";
> + };
Why do all these timers need to be disabled? Usually peripherals like DMA and
internal temperature sensors etc. that don't need any support outside the chip
can just be left on.
> +
> + timer1: timer@030a0014 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0014 0x14>;
> + interrupts = <80 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER1>;
> + status = "disabled";
> + };
> +
> + timer2: timer@030a0028 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0028 0x14>;
> + interrupts = <81 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER2>;
> + status = "disabled";
> + };
> +
> + timer3: timer@030a003c {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a003c 0x14>;
> + interrupts = <82 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER3>;
> + status = "disabled";
> + };
> +
> + timer4: timer@030a0050 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0050 0x14>;
> + interrupts = <83 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER4>;
> + status = "disabled";
> + };
> +
> + timer5: timer@30a0064 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0064 0x14>;
> + interrupts = <84 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER5>;
> + status = "disabled";
> + };
> +
> + timer6: timer@030a0078 {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a0078 0x14>;
> + interrupts = <85 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER6>;
> + status = "disabled";
> + };
> +
> + timer7: timer@030a008c {
> + compatible = "snps,dw-apb-timer";
> + reg = <0x030a008c 0x14>;
> + interrupts = <86 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&osc>;
> + resets = <&rst RST_TIMER7>;
> + status = "disabled";
> + };
> +
> clint: timer@74000000 {
> compatible = "sophgo,cv1800b-clint", "thead,c900-clint";
> reg = <0x74000000 0x10000>;
> --
> 2.34.1
>
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-12-28 13:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-28 13:06 [PATCH] riscv: dts: sophgo: add timer dt node for CV1800 AnnanLiu
2023-12-28 13:36 ` Conor Dooley
2023-12-28 13:47 ` Emil Renner Berthing
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).