[coreboot] [commit] r5713 - in trunk/src/mainboard/gigabyte: . ma785gmt

Peter Stuge peter at stuge.se
Wed Aug 18 18:18:34 CEST 2010

repository service wrote:
> +++ trunk/src/mainboard/gigabyte/ma785gmt/romstage.c	Tue Aug 17 13:08:31 2010	(r5713)
> @@ -137,6 +132,7 @@
>  	sb700_lpc_init();
>  	it8718f_enable_serial(0, CONFIG_TTYS0_BASE);
> +	it8718f_disable_reboot();
>  	uart_init();
>  	console_init();
>  	printk(BIOS_DEBUG, "\n");

I know this was from a different commit, but I think maybe this
_disable_reboot function should be moved out from the southbridge
code, since it is not really specific for the it8718f southbridge.

It is critical to add new code in the most appropriate place. It can
be difficult to find sometimes, and may require some more changes
around the coreboot code.


More information about the coreboot mailing list