2023-02-10 18:03:42

by Devarsh Thakkar

Subject: [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB

All revisions of AM62A7-SK board have 4GB LPDDR4 Micron
MT53E2G32D4DE-046 AUT:B memory. Commit 38c4a08c820c ("arm64: dts: ti:
Add support for AM62A7-SK") enabled just 2GB due to a schematics error
in early revision of the board. Fix it by enabling full 4GB available on
the platform.

Design docs: Link: https://www.ti.com/lit/zip/spruj16

Fixes: 38c4a08c820c ("arm64: dts: ti: Add support for AM62A7-SK")

Signed-off-by: Devarsh Thakkar <[email protected]>
---
Logs:
https://gist.github.com/devarsht/e85b6af89c01ddadb3a62f3e5f196af8

Changelog:
V2: Update commit message with mention of schematics error
---
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index 5c9012141ee2..f6a67f072dca 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -27,8 +27,9 @@

memory@80000000 {
device_type = "memory";
- /* 2G RAM */
- reg = <0x00000000 0x80000000 0x00000000 0x80000000>;
+ /* 4G RAM */
+ reg = <0x00000000 0x80000000 0x00000000 0x80000000>,
+ <0x00000008 0x80000000 0x00000000 0x80000000>;
};

reserved-memory {
--
2.17.1

2023-03-13 19:01:13

by Bryan Brattlof

Subject: Re: [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB

Hi Devarsh!

On February 10, 2023 thus sayeth Devarsh Thakkar:
> All revisions of AM62A7-SK board have 4GB LPDDR4 Micron
> MT53E2G32D4DE-046 AUT:B memory. Commit 38c4a08c820c ("arm64: dts: ti:
> Add support for AM62A7-SK") enabled just 2GB due to a schematics error
> in early revision of the board. Fix it by enabling full 4GB available on
> the platform.
>
> Design docs: Link: https://www.ti.com/lit/zip/spruj16

Patch looks great! Though this link doesn't seem to be working. Should
we use: https://www.ti.com/lit/zip/sprr459 ?

>
> Fixes: 38c4a08c820c ("arm64: dts: ti: Add support for AM62A7-SK")
>
> Signed-off-by: Devarsh Thakkar <[email protected]>
> ---
> Logs:
> https://gist.github.com/devarsht/e85b6af89c01ddadb3a62f3e5f196af8
>
> Changelog:
> V2: Update commit message with mention of schematics error
> ---
> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>

A part from that

Reviewed-by: Bryan Brattlof <[email protected]>

Thanks for getting this fixed up
~Bryan