2023-05-03 15:00:56

by Kumar, Udit

Subject: [PATCH 1/1] arm64: dts: ti: k3-j721s2: Add reserved status in msmc

mark atf, l3-cache and tifs node as reserved.

Signed-off-by: Udit Kumar <[email protected]>
---
arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 3 +++
1 file changed, 3 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
index 2dd7865f7654..791993060f44 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
@@ -14,14 +14,17 @@ msmc_ram: sram@70000000 {
ranges = <0x0 0x0 0x70000000 0x400000>;

atf-sram@0 {
+ status = "reserved";
reg = <0x0 0x20000>;
};

tifs-sram@1f0000 {
+ status = "reserved";
reg = <0x1f0000 0x10000>;
};

l3cache-sram@200000 {
+ status = "reserved";
reg = <0x200000 0x200000>;
};
};
--
2.34.1

2023-05-03 15:03:07

by Nishanth Menon

Subject: Re: [PATCH 1/1] arm64: dts: ti: k3-j721s2: Add reserved status in msmc

On 20:17-20230503, Udit Kumar wrote:
> Mark atf, l3-cache and tifs node as reserved.

why? (I am not reading the cover-letter for a 1 patch)
>
> Signed-off-by: Udit Kumar <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> index 2dd7865f7654..791993060f44 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
> @@ -14,14 +14,17 @@ msmc_ram: sram@70000000 {
> ranges = <0x0 0x0 0x70000000 0x400000>;
>
> atf-sram@0 {
> + status = "reserved";
> reg = <0x0 0x20000>;
> };
>
> tifs-sram@1f0000 {
> + status = "reserved";
> reg = <0x1f0000 0x10000>;
> };
>
> l3cache-sram@200000 {
> + status = "reserved";
> reg = <0x200000 0x200000>;
> };
> };
> --
> 2.34.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2023-05-03 16:33:54

by Krzysztof Kozlowski

Subject: Re: [PATCH 1/1] arm64: dts: ti: k3-j721s2: Add reserved status in msmc

On 03/05/2023 16:51, Nishanth Menon wrote:
> On 20:17-20230503, Udit Kumar wrote:
>> Mark atf, l3-cache and tifs node as reserved.
>
> why? (I am not reading the cover-letter for a 1 patch)

And you should not have to. :) The commit msg should explain why it is
useful.

Best regards,
Krzysztof

2023-05-05 05:28:44

by Kumar, Udit

Subject: Re: [PATCH 1/1] arm64: dts: ti: k3-j721s2: Add reserved status in msmc

On 5/3/2023 8:21 PM, Nishanth Menon wrote:
> On 20:17-20230503, Udit Kumar wrote:
>> Mark atf, l3-cache and tifs node as reserved.
> why? (I am not reading the cover-letter for a 1 patch)


My bad , I should have description in patch itself.

Thanks


> [...]
>> 2.34.1
>>