[v1] arm64: dts: nuvoton: Add EDAC controller

[PATCH] arm64: dts: nuvoton: Add EDAC controller

Posted by William A. Kennington III 8 months, 1 week ago

We have the driver support but need a common node for all the 8xx
platforms that contain this device.

Signed-off-by: William A. Kennington III <william@wkennington.com>
---
 arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
index 4da62308b274..ccebcb11c05e 100644
--- a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
+++ b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
@@ -56,6 +56,13 @@ clk: rstc: reset-controller@f0801000 {
 			#clock-cells = <1>;
 		};
 
+		mc: memory-controller@f0824000 {
+			compatible = "nuvoton,npcm845-memory-controller";
+			reg = <0x0 0xf0824000 0x0 0x2000>;
+			interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
 		apb {
 			#address-cells = <1>;
 			#size-cells = <1>;
-- 
2.49.0.604.gff1f9ca942-goog

Re: [PATCH] arm64: dts: nuvoton: Add EDAC controller

Posted by Andrew Jeffery 7 months, 2 weeks ago

On Tue, 2025-04-15 at 16:34 -0700, William A. Kennington III wrote:
> We have the driver support but need a common node for all the 8xx
> platforms that contain this device.
> 
> Signed-off-by: William A. Kennington III <william@wkennington.com>
> ---
>  arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> index 4da62308b274..ccebcb11c05e 100644
> --- a/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> +++ b/arch/arm64/boot/dts/nuvoton/nuvoton-common-npcm8xx.dtsi
> @@ -56,6 +56,13 @@ clk: rstc: reset-controller@f0801000 {
>                         #clock-cells = <1>;
>                 };
>  
> +               mc: memory-controller@f0824000 {
> +                       compatible = "nuvoton,npcm845-memory-controller";
> +                       reg = <0x0 0xf0824000 0x0 0x2000>;
> +                       interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>;
> +                       status = "disabled";
> +               };
> +

The patch fails to apply due to fuzz, do you mind rebasing it on
nuvoton/arm64/dt from [1]?

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/arj/bmc.git

Possibly it's the result of applying other patches of yours as they
were sent independently. It might be best to send them as a series even
if they are conceptually independent.

Andrew