[coreboot] [patch][v2] AMD Fam10 memory controller updates

David Hendricks dhendrix at google.com
Tue Apr 8 01:43:44 CEST 2008


Zuh? Whitespace issues are generally so insignificant that if it doesn't get
fixed when the code is fixed then the whitespace issues might never get
revisited. They're also tedious to keep track of. And for a large patches
the follow-up whitespace code review can fall out of sync with the actual
code review and make resolving conflicts between the original patch and
follow-up whitespace patch prone to error. And in case we have to cherrypick
changes to rollback it doesn't make sense to have to worry about a separate
patch for whitespace fixes.

Just my $0.02.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080407/bf00d50d/attachment.html>


More information about the coreboot mailing list