[coreboot] [PATCH]Remove fallback/normal decision from mainboards' romstage.c

Peter Stuge peter at stuge.se
Thu Mar 18 17:36:43 CET 2010


Patrick Georgi wrote:
> >> +		/* sb600_lpc_port80(); */
> >> +		sb600_pci_port80();
> > 
> > This is a recurring pattern with both sb600 and sb700. Is there a
> > reason to duplicate that call in a comment or could the comment just
> > be removed?
> 
> This was a mechanical refactoring without looking in-depth at such
> issues. (all that code existed before)

Nod, yeah, I noticed it was there before.


> There might be ways to refactor such code, but that requires a more
> global view.

Fair enough.


> >> --- src/mainboard/amd/mahogany/cache_as_ram_auto.c	(revision 5256)
> > Where did this code go?
> 
> src/mainboard/amd/mahogany/romstage.c already exists and is nearly
> identical, except for an additional bugfix.

Brilliant.

Acked-by: Peter Stuge <peter at stuge.se>




More information about the coreboot mailing list