All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB
@ 2023-02-10 18:03 ` Devarsh Thakkar
  0 siblings, 0 replies; 4+ messages in thread
From: Devarsh Thakkar @ 2023-02-10 18:03 UTC (permalink / raw)
  To: devicetree, linux-kernel, linux-arm-kernel,
	krzysztof.kozlowski+dt, robh+dt, kristo, nm, vigneshr
  Cc: hnagalla, praneeth, a-bhatia1, j-luthra, devarsht

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 <devarsht@ti.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(-)

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB
@ 2023-02-10 18:03 ` Devarsh Thakkar
  0 siblings, 0 replies; 4+ messages in thread
From: Devarsh Thakkar @ 2023-02-10 18:03 UTC (permalink / raw)
  To: devicetree, linux-kernel, linux-arm-kernel,
	krzysztof.kozlowski+dt, robh+dt, kristo, nm, vigneshr
  Cc: hnagalla, praneeth, a-bhatia1, j-luthra, devarsht

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 <devarsht@ti.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(-)

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


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB
  2023-02-10 18:03 ` Devarsh Thakkar
@ 2023-03-13 19:00   ` Bryan Brattlof
  -1 siblings, 0 replies; 4+ messages in thread
From: Bryan Brattlof @ 2023-03-13 19:00 UTC (permalink / raw)
  To: Devarsh Thakkar
  Cc: devicetree, linux-kernel, linux-arm-kernel,
	krzysztof.kozlowski+dt, robh+dt, kristo, nm, vigneshr, hnagalla,
	praneeth, a-bhatia1, j-luthra

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@ti.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@ti.com>

Thanks for getting this fixed up
~Bryan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB
@ 2023-03-13 19:00   ` Bryan Brattlof
  0 siblings, 0 replies; 4+ messages in thread
From: Bryan Brattlof @ 2023-03-13 19:00 UTC (permalink / raw)
  To: Devarsh Thakkar
  Cc: devicetree, linux-kernel, linux-arm-kernel,
	krzysztof.kozlowski+dt, robh+dt, kristo, nm, vigneshr, hnagalla,
	praneeth, a-bhatia1, j-luthra

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@ti.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@ti.com>

Thanks for getting this fixed up
~Bryan

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-03-13 19:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-10 18:03 [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Fix DDR size to full 4GB Devarsh Thakkar
2023-02-10 18:03 ` Devarsh Thakkar
2023-03-13 19:00 ` Bryan Brattlof
2023-03-13 19:00   ` Bryan Brattlof

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.