Re: [PATCH] riscv: Rename Andes PLIC to PLICSW

2022-10-28 Thread Rick Chen
> From: Peter Yu-Chien Lin(林宇謙) 
> Sent: Tuesday, October 25, 2022 11:04 PM
> To: u-boot@lists.denx.de
> Cc: Leo Yu-Chi Liang(梁育齊) ; Rick Jian-Zhi Chen(陳建志) 
> ; Peter Yu-Chien Lin(林宇謙) 
> Subject: [PATCH] riscv: Rename Andes PLIC to PLICSW
>
> As PLICSW is used to trigger the software interrupt, we should rename Andes 
> PLIC configuration and file name to reflect the usage. This patch also 
> updates PLMT and PLICSW compatible strings to be consistent with OpenSBI fdt 
> driver.
>
> Signed-off-by: Yu Chien Peter Lin 
> ---
>  arch/riscv/Kconfig|  6 ++---
>  arch/riscv/cpu/ax25/Kconfig   |  2 +-
>  arch/riscv/dts/ae350-u-boot.dtsi  |  2 +-
>  arch/riscv/dts/ae350_32.dts   |  6 ++---
>  arch/riscv/dts/ae350_64.dts   |  6 ++---
>  arch/riscv/include/asm/global_data.h  |  4 +--
>  arch/riscv/include/asm/syscon.h   |  2 +-
>  arch/riscv/lib/Makefile   |  2 +-
>  .../lib/{andes_plic.c => andes_plicsw.c}  | 26 +--
>  9 files changed, 28 insertions(+), 28 deletions(-)  rename 
> arch/riscv/lib/{andes_plic.c => andes_plicsw.c} (76%)

Reviewed-by: Rick Chen 


[PATCH] riscv: Rename Andes PLIC to PLICSW

2022-10-25 Thread Yu Chien Peter Lin
As PLICSW is used to trigger the software interrupt, we should rename
Andes PLIC configuration and file name to reflect the usage. This patch
also updates PLMT and PLICSW compatible strings to be consistent with
OpenSBI fdt driver.

Signed-off-by: Yu Chien Peter Lin 
---
 arch/riscv/Kconfig|  6 ++---
 arch/riscv/cpu/ax25/Kconfig   |  2 +-
 arch/riscv/dts/ae350-u-boot.dtsi  |  2 +-
 arch/riscv/dts/ae350_32.dts   |  6 ++---
 arch/riscv/dts/ae350_64.dts   |  6 ++---
 arch/riscv/include/asm/global_data.h  |  4 +--
 arch/riscv/include/asm/syscon.h   |  2 +-
 arch/riscv/lib/Makefile   |  2 +-
 .../lib/{andes_plic.c => andes_plicsw.c}  | 26 +--
 9 files changed, 28 insertions(+), 28 deletions(-)
 rename arch/riscv/lib/{andes_plic.c => andes_plicsw.c} (76%)

diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index 8f9578171d..4d64e9be3f 100644
--- a/arch/riscv/Kconfig
+++ b/arch/riscv/Kconfig
@@ -199,7 +199,7 @@ config SIFIVE_CACHE
help
  This enables the operations to configure SiFive cache
 
-config ANDES_PLIC
+config ANDES_PLICSW
bool
depends on RISCV_MMODE || SPL_RISCV_MMODE
select REGMAP
@@ -207,8 +207,8 @@ config ANDES_PLIC
select SPL_REGMAP if SPL
select SPL_SYSCON if SPL
help
- The Andes PLIC block holds memory-mapped claim and pending registers
- associated with software interrupt.
+ The Andes PLICSW block holds memory-mapped claim and pending
+ registers associated with software interrupt.
 
 config SMP
bool "Symmetric Multi-Processing"
diff --git a/arch/riscv/cpu/ax25/Kconfig b/arch/riscv/cpu/ax25/Kconfig
index 941d963ece..4a7295d30c 100644
--- a/arch/riscv/cpu/ax25/Kconfig
+++ b/arch/riscv/cpu/ax25/Kconfig
@@ -4,7 +4,7 @@ config RISCV_NDS
imply CPU
imply CPU_RISCV
imply RISCV_TIMER if (RISCV_SMODE || SPL_RISCV_SMODE)
-   imply ANDES_PLIC if (RISCV_MMODE || SPL_RISCV_MMODE)
+   imply ANDES_PLICSW if (RISCV_MMODE || SPL_RISCV_MMODE)
imply ANDES_PLMT_TIMER if (RISCV_MMODE || SPL_RISCV_MMODE)
imply SPL_CPU
imply SPL_OPENSBI
diff --git a/arch/riscv/dts/ae350-u-boot.dtsi b/arch/riscv/dts/ae350-u-boot.dtsi
index 0d4201cfae..7011f59831 100644
--- a/arch/riscv/dts/ae350-u-boot.dtsi
+++ b/arch/riscv/dts/ae350-u-boot.dtsi
@@ -36,7 +36,7 @@
soc {
u-boot,dm-spl;
 
-   plic1: interrupt-controller@e640 {
+   plicsw: interrupt-controller@e640 {
u-boot,dm-spl;
};
 
diff --git a/arch/riscv/dts/ae350_32.dts b/arch/riscv/dts/ae350_32.dts
index 083f676333..96ef8bd8dd 100644
--- a/arch/riscv/dts/ae350_32.dts
+++ b/arch/riscv/dts/ae350_32.dts
@@ -146,8 +146,8 @@
&CPU3_intc 11 &CPU3_intc 9>;
};
 
-   plic1: interrupt-controller@e640 {
-   compatible = "riscv,plic1";
+   plicsw: interrupt-controller@e640 {
+   compatible = "andestech,plicsw";
#interrupt-cells = <1>;
interrupt-controller;
reg = <0xe640 0x40>;
@@ -159,7 +159,7 @@
};
 
plmt0@e600 {
-   compatible = "riscv,plmt0";
+   compatible = "andestech,plmt0";
interrupts-extended = <&CPU0_intc 7
&CPU1_intc 7
&CPU2_intc 7
diff --git a/arch/riscv/dts/ae350_64.dts b/arch/riscv/dts/ae350_64.dts
index 74cff9122d..cddbaec98a 100644
--- a/arch/riscv/dts/ae350_64.dts
+++ b/arch/riscv/dts/ae350_64.dts
@@ -146,8 +146,8 @@
&CPU3_intc 11 &CPU3_intc 9>;
};
 
-   plic1: interrupt-controller@e640 {
-   compatible = "riscv,plic1";
+   plicsw: interrupt-controller@e640 {
+   compatible = "andestech,plicsw";
#interrupt-cells = <2>;
interrupt-controller;
reg = <0x0 0xe640 0x0 0x40>;
@@ -159,7 +159,7 @@
};
 
plmt0@e600 {
-   compatible = "riscv,plmt0";
+   compatible = "andestech,plmt0";
interrupts-extended = <&CPU0_intc 7
&CPU1_intc 7
&CPU2_intc 7
diff --git a/arch/riscv/include/asm/global_data.h 
b/arch/riscv/include/asm/global_data.h
index 858594a191..6fdc86dd8b 100644
--- a/arch/riscv/include/asm/global_data.h
+++ b/arch/riscv/include/asm/global_data.h
@@ -21,8 +21,8 @@ struct arch_global_data {
 #if CONFIG_IS_ENABLED(SIFIVE_CLINT)
void __iomem *clint;/* clint base a