Patchwork ppc4xx-Walnut: fixes early boot messages

login
register
mail settings
Submitter Niklaus Giger
Date Oct. 2, 2008, 5:36 a.m.
Message ID <200810020736.40736.niklaus.giger@member.fsf.org>
Download mbox | patch
Permalink /patch/2334/
State Rejected, archived
Delegated to: Josh Boyer
Headers show

Comments

Niklaus Giger - Oct. 2, 2008, 5:36 a.m.
Inherits the baudrate from U-boot to print things like:
Using Walnut machine description                                          
Linux version <..>
console [udbg0] enabled                                                                                       

Signed-off-by: Niklaus Giger <niklaus.giger@member.fsf.org>
---
 arch/powerpc/boot/dts/walnut.dts |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
David Gibson - Oct. 2, 2008, 6:39 a.m.
On Thu, Oct 02, 2008 at 07:36:40AM +0200, Niklaus Giger wrote:
> Inherits the baudrate from U-boot to print things like:
> Using Walnut machine description                                          
> Linux version <..>
> console [udbg0] enabled                                                                                       
Uh.. but what if you're booting from cuboot or openbios, rather than a
device-tree-aware uboot.
Josh Boyer - Oct. 2, 2008, 12:09 p.m.
On Thu, Oct 02, 2008 at 04:39:54PM +1000, David Gibson wrote:
>On Thu, Oct 02, 2008 at 07:36:40AM +0200, Niklaus Giger wrote:
>> Inherits the baudrate from U-boot to print things like:
>> Using Walnut machine description                                          
>> Linux version <..>
>> console [udbg0] enabled                                                                                       
>Uh.. but what if you're booting from cuboot or openbios, rather than a
>device-tree-aware uboot.

Right.  I don't think we want this particular change.  U-Boot can always
update it anyway.

josh
Niklaus Giger - Oct. 2, 2008, 6:51 p.m.
Am Donnerstag 02 Oktober 2008 08.39:54 schrieb David Gibson:
> On Thu, Oct 02, 2008 at 07:36:40AM +0200, Niklaus Giger wrote:
> > Inherits the baudrate from U-boot to print things like:
> > Using Walnut machine description
> > Linux version <..>
> > console [udbg0] enabled
>
> Uh.. but what if you're booting from cuboot or openbios, rather than a
> device-tree-aware uboot.
Sorry. I forgot that you can have other boot-eprom beside U-Boot. I will try 
to fix that in my u-boot configuration.

Best regards

Niklaus

Patch

diff --git a/arch/powerpc/boot/dts/walnut.dts 
b/arch/powerpc/boot/dts/walnut.dts
index 4a9f726..6c93f72 100644
--- a/arch/powerpc/boot/dts/walnut.dts
+++ b/arch/powerpc/boot/dts/walnut.dts
@@ -98,7 +98,7 @@ 
 				reg = <0xef600300 0x00000008>;
 				virtual-reg = <0xef600300>;
 				clock-frequency = <0>; /* Filled in by zImage */
-				current-speed = <9600>;
+				current-speed = <0>;	/* Filled in by U-Boot */
 				interrupt-parent = <&UIC0>;
 				interrupts = <0x0 0x4>;
 			};