[coreboot] [PATCH] VIA C7 CAR doesn't configure the stack pointer

Myles Watson mylesgw at gmail.com
Mon Apr 27 20:18:28 CEST 2009


> the attached patch sets up the stack pointer in the c7 CAR-post routine,
> to
> make function calls work in the small area after CAR and before
> coreboot_ram.
> For sufficient high loglevels, there are two printk_*s in generic CAR code
> that made the system reboot. (with my preceding patch, it crashes for all
> loglevel configurations and an issue like this one would have been found
> directly)
> 
> 
> Signed-off-by: Patrick Georgi <patrick.georgi at coresystems.de>

The stack pointer is a magic value to me.  Could you add a comment stating
why you chose 0x400000?  Maybe that will help someone decide whether it
needs to change later for suspend-to-RAM.

Acked-by: Myles Watson <mylesgw at gmail.com>

Thanks,
Myles





More information about the coreboot mailing list