u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE
@ 2021-11-14 12:09 Mark Kettenis
  2021-11-25  0:12 ` Simon Glass
  2022-01-07 17:03 ` Tom Rini
  0 siblings, 2 replies; 3+ messages in thread
From: Mark Kettenis @ 2021-11-14 12:09 UTC (permalink / raw)
  To: u-boot; +Cc: Mark Kettenis

The memory layout is taken from the device tree passed to us by
m1n1, so there is no need to define this.

Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
---
 include/configs/apple.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/configs/apple.h b/include/configs/apple.h
index b1f6043174..112724883b 100644
--- a/include/configs/apple.h
+++ b/include/configs/apple.h
@@ -3,8 +3,6 @@
 
 #include <linux/sizes.h>
 
-#define CONFIG_SYS_SDRAM_BASE	0x880000000
-
 #define CONFIG_LNX_KRNL_IMG_TEXT_OFFSET_BASE	CONFIG_SYS_TEXT_BASE
 
 /* Environment */
-- 
2.33.1


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

* Re: [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE
  2021-11-14 12:09 [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE Mark Kettenis
@ 2021-11-25  0:12 ` Simon Glass
  2022-01-07 17:03 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Simon Glass @ 2021-11-25  0:12 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: u-boot

On Sun, 14 Nov 2021 at 05:10, Mark Kettenis <kettenis@openbsd.org> wrote:
>
> The memory layout is taken from the device tree passed to us by
> m1n1, so there is no need to define this.
>
> Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
> ---
>  include/configs/apple.h | 2 --
>  1 file changed, 2 deletions(-)
>

Reviewed-by: Simon Glass <sjg@chromium.org>
Tested-on: Apple M1 Macbook
Tested-by: Simon Glass <sjg@chromium.org>

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

* Re: [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE
  2021-11-14 12:09 [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE Mark Kettenis
  2021-11-25  0:12 ` Simon Glass
@ 2022-01-07 17:03 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2022-01-07 17:03 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: u-boot

[-- Attachment #1: Type: text/plain, Size: 376 bytes --]

On Sun, Nov 14, 2021 at 01:09:54PM +0100, Mark Kettenis wrote:

> The memory layout is taken from the device tree passed to us by
> m1n1, so there is no need to define this.
> 
> Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Tested-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/next, thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

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

end of thread, other threads:[~2022-01-07 17:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-14 12:09 [PATCH] arm: apple: Remove CONFIG_SYS_SDRAM_BASE Mark Kettenis
2021-11-25  0:12 ` Simon Glass
2022-01-07 17:03 ` Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).