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

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: <20230313190048.w74clxxsdqxzuhor@bryanbrattlof.com>
Date:   Mon, 13 Mar 2023 14:00:48 -0500
From:   Bryan Brattlof <bb@...com>
To:     Devarsh Thakkar <devarsht@...com>
CC:     <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <krzysztof.kozlowski+dt@...aro.org>, <robh+dt@...nel.org>,
        <kristo@...nel.org>, <nm@...com>, <vigneshr@...com>,
        <hnagalla@...com>, <praneeth@...com>, <a-bhatia1@...com>,
        <j-luthra@...com>
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 <devarsht@...com>
> ---
> 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@...com>

Thanks for getting this fixed up
~Bryan