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

From: Bryan Brattlof
Date: Mon Mar 13 2023 - 15:01:13 EST


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 <devarsht@xxxxxx>
> ---
> 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 <bb@xxxxxx>

Thanks for getting this fixed up
~Bryan