[LinuxBIOS] Sun Ultra40 patches
Roman Kononov
kononov195-lbl at yahoo.com
Fri Feb 2 21:10:13 CET 2007
On 02/02/2007 11:58 AM, Uwe Hermann wrote:
> On Fri, Feb 02, 2007 at 05:50:04PM +0100, Stefan Reinauer wrote:
>> Could you please rediff with -w ? Something terribly broke the indenting.
>
> I guess this was intended. It looks like Roman fixed the coding style(?)
> E.g. replaced spaces with tabs in a number of places...
>
> Such patches should be submitted separately from "real" code changes,
> though, to make it easier to review the patches.
Rediffing with -w will not help. Fixing coding style first will not help
either. This is not as simple as a patch. It is a major re-writing.
The current code in the repository is half-finished and works only
by an accident. Reviewers should apply the patch and try to understand
what is going on there.
Regards, Roman
More information about the coreboot
mailing list