mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: i.MX8MP: remove now unneeded DSP reserved mem override
@ 2024-11-19  8:45 Ahmad Fatoum
  2024-11-25  8:45 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2024-11-19  8:45 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

The upstream device tree described a dsp_reserved reserved memory entry
at address 0x9240_0000, which could collide with barebox malloc area in
some configurations.

This node has now been disabled upstream in commit 010dc015b811
("arm64: dts: imx8mp: Disable dsp reserved memory by default"), so we
can simplify our DT a bit by dropping the override.

Device trees that make use of the dsp node will now need to override
it on a barebox board DT level.

Cc: Lucas Stach <l.stach@pengutronix.de>
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 arch/arm/dts/imx8mp.dtsi | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/arch/arm/dts/imx8mp.dtsi b/arch/arm/dts/imx8mp.dtsi
index 4d1f1bf58827..09bd4b2c0356 100644
--- a/arch/arm/dts/imx8mp.dtsi
+++ b/arch/arm/dts/imx8mp.dtsi
@@ -23,15 +23,8 @@ chosen {
 	};
 };
 
-/*
- * The DSP reserved memory will collide with the Barebox malloc area for some
- * DRAM sizes, even though the DSP itself is disabled in most configurations.
- */
-/delete-node/ &dsp_reserved;
 &dsp {
 	barebox,feature-gates = <&feat IMX8M_FEAT_DSP>;
-	/delete-property/ memory-region;
-	status = "disabled";
 };
 
 &edacmc {
-- 
2.39.5




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

* Re: [PATCH] ARM: dts: i.MX8MP: remove now unneeded DSP reserved mem override
  2024-11-19  8:45 [PATCH] ARM: dts: i.MX8MP: remove now unneeded DSP reserved mem override Ahmad Fatoum
@ 2024-11-25  8:45 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2024-11-25  8:45 UTC (permalink / raw)
  To: barebox, Ahmad Fatoum


On Tue, 19 Nov 2024 09:45:55 +0100, Ahmad Fatoum wrote:
> The upstream device tree described a dsp_reserved reserved memory entry
> at address 0x9240_0000, which could collide with barebox malloc area in
> some configurations.
> 
> This node has now been disabled upstream in commit 010dc015b811
> ("arm64: dts: imx8mp: Disable dsp reserved memory by default"), so we
> can simplify our DT a bit by dropping the override.
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: i.MX8MP: remove now unneeded DSP reserved mem override
      https://git.pengutronix.de/cgit/barebox/commit/?id=d6e22a12115d (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>




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

end of thread, other threads:[~2024-11-25  8:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-19  8:45 [PATCH] ARM: dts: i.MX8MP: remove now unneeded DSP reserved mem override Ahmad Fatoum
2024-11-25  8:45 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox