Skip to contentSkip to conversation

integratorap: remove hardcoded 32MB memory cmdline

PATCH New Linus Walleij 3 participants 5 messages Jul 14, 2011 – Jul 15, 2011

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 integratorap: remove hardcoded 32MB memory cmdline

The default configuration for the Integrator AP forces memory to be 32 MB on the command line to the kernel, while we have perfect information and detection of the actual memory size in the ATAGs. Delete the confusion.

---
 include/configs/integratorap.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
include/configs/integratorap.h +1 -1
index d724c91..7b8aea6 100644
@@ -94,7 +94,7 @@
 
 
 #define CONFIG_BOOTDELAY	2
-#define CONFIG_BOOTARGS		"root=/dev/mtdblock0 mem=32M console=ttyAM0 console=tty"
+#define CONFIG_BOOTARGS		"root=/dev/mtdblock0 console=ttyAM0 console=tty"
 #define CONFIG_BOOTCOMMAND	""
 
 /*
Signature · 1 line
1.7.6
#1 lore raw

PATCH 1/1 integratorap: remove hardcoded 32MB memory cmdline

Dear Linus Walleij,

Quoted 15 lines · In message <[email protected]> you…

The default configuration for the Integrator AP forces memory to be 32 MB on the command line to the kernel, while we have perfect information and detection of the actual memory size in the ATAGs. Delete the confusion.

---
 include/configs/integratorap.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
include/configs/integratorap.h +0 -0
index d724c91..7b8aea6 100644

include/configs/integratorap.h line 98

Quoted 6 lines
include/configs/integratorap.h +1 -1
@@ -94,7 +94,7 @@
 
 
 #define CONFIG_BOOTDELAY	2
-#define CONFIG_BOOTARGS		"root=/dev/mtdblock0 mem=32M console=ttyAM0 console=tty"
+#define CONFIG_BOOTARGS		"root=/dev/mtdblock0 console=ttyAM0 console=tty"
 #define CONFIG_BOOTCOMMAND	""

Please make sure to Cc: the board maintainer!

Best regards,

Wolfgang Denk

Signature · 4 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
To be awake is to be alive.        - Henry David Thoreau, in "Walden"
#2 lore raw

PATCH 1/1 integratorap: remove hardcoded 32MB memory cmdline

Quoted 4 lines · Wolfgang Denk

include/configs/integratorap.h

Quoted 2 lines
include/configs/integratorap.h +0 -0

Please make sure to Cc: the board maintainer!

Aw yes, that's one thing. Peter, are you still looking after the Integrator AP board support in U-boot?

I mailed with Philippe who said ARM dropped support for the Integrator board, and given the state of the code in U-boot (doesn't compile, doesn't work) I might be the only one actually looking after it in practice.

I'm happy to take over maintenance of it for a while atleast, if I have yours and/or Philippe's ACK.

Thanks,
Linus Walleij

#3 lore raw

PATCH 1/1 integratorap: remove hardcoded 32MB memory cmdline

Quoted 9 lines · Linus Walleij
Quoted 4 lines · Wolfgang Denk

include/configs/integratorap.h

Quoted 2 lines
include/configs/integratorap.h +0 -0

Please make sure to Cc: the board maintainer!

Aw yes, that's one thing. Peter, are you still looking after the Integrator AP board support in U-boot?

I got this as autoreply from another conversation, so obviously Peter isn't working with this anymore, actually if I recall correctly, he has retired from ARM.

I will attempt to find his private email address but I will propose a patch taking over as Integrator maintainer since I have the board and I suspect noone else i likely to be working with it.

--------------------------------
From: [email protected]
Sender: StaffLeavers at arm.com
Reply-To: StaffLeavers at arm.com
To: Linus Walleij <[email protected]>
Subject: Re: Re: ARM DMA controllers
X-OriginalArrivalTime: 14 Jul 2011 20:25:56.0600 (UTC) FILETIME=[3C7EAF80:01CC4264]

peter.pearse no longer works for ARM.

Your email will be forwarded to their line manager.

Please do not reply to this email.
If you need more information, please email real-postmaster at arm.com

Thank you.

-------------------------------

Thanks,
Linus Walleij

#4 lore raw

PATCH 1/1 integratorap: remove hardcoded 32MB memory cmdline

Hi Linus, Wolfgang,

Quoted 1 line · Linus Walleij
Quoted 1 line

On Thu, Jul 14, 2011 at 8:03 PM, Linus Walleij <[email protected]>

Quoted 8 lines · wrote:
Quoted 8 lines
Quoted 8 lines · Wolfgang Denk
Quoted 5 lines

include/configs/integratorap.h

Quoted 3 lines
include/configs/integratorap.h +0 -0

Please make sure to Cc: the board maintainer!

Aw yes, that's one thing. Peter, are you still looking after the

Integrator

Quoted 3 lines
Quoted 3 lines
Quoted 1 line

AP board support in U-boot?

I got this as autoreply from another conversation, so obviously Peter

isn't

Quoted 1 line
Quoted 1 line

working with this anymore, actually if I recall correctly, he has retired

from

Quoted 1 line
Quoted 1 line

ARM.

This is correct, Peter retired a couple of months ago.

Quoted 3 lines
Quoted 3 lines

I will attempt to find his private email address but I will propose a patch taking over as Integrator maintainer since I have the board and I suspect noone else i likely to be working with it.

This is probably best for this. Many thanks.

Regards,
Philippe