Re: [coreboot] [PATCH] AMD CS5530(A) ROM enable

[Uwe Hermann]

On Tue, Oct 06, 2009 at 08:16:17PM +0200, Paul Menzel wrote:
> Dear Uwe,
>
> Am Dienstag, den 06.10.2009, 16:34 +0200 schrieb Uwe Hermann:
> > Index: src/southbridge/amd/cs5530/cs5530.h
> >
> > --- src/southbridge/amd/cs5530/cs5530.h (Revision 4726)
> > +++ src/southbridge/amd/cs5530/cs5530.h (Arbeitskopie)
> > @@ -27,7 +27,13 @@
>
> […]
> > +#define LOWER_ROM_ADDRESS_RANGE                (1 << 0)
> > +#define ROM_WRITE_ENABLE               (1 << 1)
> > +#define UPPER_ROM_ADDRESS_RANGE                (1 << 2)
> > +#define BIOS_ROM_POSITIVE_DECODE       (1 << 5)
>
> Is this spacing intentional?

Yes. It looks strange in the patch/email, but if you apply the patch and
look at the file in a text editor it'll be nicely aligned.

Uwe.