[coreboot] [patch][v2] AMD Fam10 memory controller updates
Uwe Hermann
uwe at hermann-uwe.de
Tue Apr 8 02:48:42 CEST 2008
Hi,
(sorry, can't resist to join the thread :)
On Mon, Apr 07, 2008 at 05:10:34PM -0700, Stefan Reinauer wrote:
> While in theory I agree with Peter that separating whitespace is a lot
> easier to read,
Yes.
> I don't believe this is really a practical approach.
Yes and no. I personally don't mind one or two whitespace fixes in
a patch which actually does functional changes, as long as there are
no _massive_ whitespace changes mixed into the patch. Those should
definately be separated in an extra patch.
That's not the case in Marc's patch, though.
> chaff without being all too confused. If it happens to be viable, yes,
> please try to make it like that, but NACKing a patch for that reason is
> unacceptable.
While I'm probably the whitespace fanatic #1 around here I have to agree
with this ;) Yes, whitespace fixes are important, and yes they should be
in separate patches if there are many of them, but that's not a reason
to NACK a patch, IMO...
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
More information about the coreboot
mailing list