[coreboot] [PATCH 3/5] artecgroup/dbe61: Gather RAM initialization function calls to one helper function.
Peter Stuge
peter at stuge.se
Thu Nov 13 04:22:05 CET 2008
Carl-Daniel Hailfinger wrote:
> >> we want to keep Geode targets consistent.
> >
> > Says who?
>
> I say so and my clone agrees. ;-)
Well speak for yourself. :) I sure don't want to keep Geode targets
consistent at all costs. I don't think that will be useful.
> > If a board is unique and we can't abstract in general code then
> > mainboard/ code is fine.
>
> If the abstraction works for all the other boards, there is no
> reason to have different codebases except to confuse people who
> want to port a new similar board.
>
> We have lots of places in v2 where people made some change to one
> specific board, but the change would have applied to lots of
> boards.
In that case, please apply the change to more places. Better yet,
move the common code out of board specific code. I promise to review
and ack swiftly.
> Later on, nobody could recall offhand why the files were different.
Do you see this being a problem in this dual fake SPD and RAM chip
case anytime soon?
> To be honest, the amount of code duplication we have in v2 with
> little arbitrary changes sprinkled all over the map is one of the
> biggest reasons why I try to avoid v2 wherever possible.
Cool. Feel free to help improve this in v3 by sending fixes when you
identify possible improvements.
//Peter
More information about the coreboot
mailing list