[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