[PATCH v4 8/8] arm: dts: k3-*-ddr: Add ss_cfg reg entry

Santhosh Kumar K s-k6 at ti.com
Thu Dec 12 15:40:40 CET 2024


Hi, Neha,

On 22/10/24 14:52, Neha Malcom Francis wrote:
> Hi Santhosh
> 
> On 21/10/24 10:10, Santhosh Kumar K wrote:
>> Add ss_cfg memory region which maps the DDRSS configuration region
>> for the memory controller node.
>>
>> Signed-off-by: Santhosh Kumar K <s-k6 at ti.com>
>> Signed-off-by: Neha Malcom Francis <n-francis at ti.com>
>> ---
>>   arch/arm/dts/k3-am62a-ddr.dtsi  |  7 ++++---
>>   arch/arm/dts/k3-j721s2-ddr.dtsi | 12 ++++++++----
>>   arch/arm/dts/k3-j784s4-ddr.dtsi | 24 ++++++++++++++++--------
>>   3 files changed, 28 insertions(+), 15 deletions(-)
>>
> 
> Thank you for this series. Maybe I missed something, but why is this DT 
> patch at the very end of the series? Doesn't it make sense to include it 
> before the patches triggering inline ECC i.e. patch 7/8?

True, will change the order in next version.

Regards,
Santhosh.

> 
> Sanity tested inline ECC on j784s4 and j721s2 and all looks good [1].
> 
> Tested-by: Neha Malcom Francis <n-francis at ti.com>
> 
> [1] https://gist.github.com/nehamalcom/66f1a2ef5288a94eebabe97708b5a277
> 


More information about the U-Boot mailing list