netdev - Re: [PATCH net-next 10/10] arm64: dts: mt7986: move dlm in a dedicated node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <68a51c57-ba63-94c3-3ca3-f7d4ab4984ae@gmail.com>
Date:   Fri, 31 Mar 2023 12:56:32 +0200
From:   Matthias Brugger <matthias.bgg@...il.com>
To:     Lorenzo Bianconi <lorenzo@...nel.org>, netdev@...r.kernel.org
Cc:     davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
        pabeni@...hat.com, linux-mediatek@...ts.infradead.org,
        nbd@....name, john@...ozen.org, sean.wang@...iatek.com,
        Mark-MC.Lee@...iatek.com, lorenzo.bianconi@...hat.com,
        daniel@...rotopia.org, krzysztof.kozlowski+dt@...aro.org,
        robh+dt@...nel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH net-next 10/10] arm64: dts: mt7986: move dlm in a
 dedicated node



On 20/03/2023 17:58, Lorenzo Bianconi wrote:
> Since the dlm memory region is not part of the RAM SoC, move dlm in a
> deidicated syscon node.
> This patch helps to keep backward-compatibility with older version of
> uboot codebase where we have a limit of 8 reserved-memory dts child
> nodes.
> 
> Signed-off-by: Lorenzo Bianconi <lorenzo@...nel.org>

Acked-by: Matthias Brugger <matthias.bgg@...il.com>

> ---
>   arch/arm64/boot/dts/mediatek/mt7986a.dtsi | 30 ++++++++++++-----------
>   1 file changed, 16 insertions(+), 14 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt7986a.dtsi b/arch/arm64/boot/dts/mediatek/mt7986a.dtsi
> index a0d96d232ee5..0ae6aa59d3c6 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7986a.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt7986a.dtsi
> @@ -93,16 +93,6 @@ wo_data: wo-data@...80000 {
>   			reg = <0 0x4fd80000 0 0x240000>;
>   			no-map;
>   		};
> -
> -		wo_dlm0: wo-dlm@...e8000 {
> -			reg = <0 0x151e8000 0 0x2000>;
> -			no-map;
> -		};
> -
> -		wo_dlm1: wo-dlm@...f8000 {
> -			reg = <0 0x151f8000 0 0x2000>;
> -			no-map;
> -		};
>   	};
>   
>   	timer {
> @@ -444,10 +434,11 @@ wed0: wed@...10000 {
>   			reg = <0 0x15010000 0 0x1000>;
>   			interrupt-parent = <&gic>;
>   			interrupts = <GIC_SPI 205 IRQ_TYPE_LEVEL_HIGH>;
> -			memory-region = <&wo_emi0>, <&wo_dlm0>, <&wo_data>;
> -			memory-region-names = "wo-emi", "wo-dlm", "wo-data";
> +			memory-region = <&wo_emi0>, <&wo_data>;
> +			memory-region-names = "wo-emi", "wo-data";
>   			mediatek,wo-ccif = <&wo_ccif0>;
>   			mediatek,wo-ilm = <&wo_ilm0>;
> +			mediatek,wo-dlm = <&wo_dlm0>;
>   			mediatek,wo-cpuboot = <&wo_cpuboot>;
>   		};
>   
> @@ -457,10 +448,11 @@ wed1: wed@...11000 {
>   			reg = <0 0x15011000 0 0x1000>;
>   			interrupt-parent = <&gic>;
>   			interrupts = <GIC_SPI 206 IRQ_TYPE_LEVEL_HIGH>;
> -			memory-region = <&wo_emi1>, <&wo_dlm1>, <&wo_data>;
> -			memory-region-names = "wo-emi", "wo-dlm", "wo-data";
> +			memory-region = <&wo_emi1>, <&wo_data>;
> +			memory-region-names = "wo-emi", "wo-data";
>   			mediatek,wo-ccif = <&wo_ccif1>;
>   			mediatek,wo-ilm = <&wo_ilm1>;
> +			mediatek,wo-dlm = <&wo_dlm1>;
>   			mediatek,wo-cpuboot = <&wo_cpuboot>;
>   		};
>   
> @@ -488,6 +480,16 @@ wo_ilm1: syscon@...f0000 {
>   			reg = <0 0x151f0000 0 0x8000>;
>   		};
>   
> +		wo_dlm0: syscon@...e8000 {
> +			compatible = "mediatek,mt7986-wo-dlm", "syscon";
> +			reg = <0 0x151e8000 0 0x2000>;
> +		};
> +
> +		wo_dlm1: syscon@...f8000 {
> +			compatible = "mediatek,mt7986-wo-dlm", "syscon";
> +			reg = <0 0x151f8000 0 0x2000>;
> +		};
> +
>   		wo_cpuboot: syscon@...94000 {
>   			compatible = "mediatek,mt7986-wo-cpuboot", "syscon";
>   			reg = <0 0x15194000 0 0x1000>;