[coreboot] [PATCH] v3: Fix ADL MSM800SEV build

ron minnich rminnich at gmail.com
Fri Jul 11 21:51:27 CEST 2008


On Fri, Jul 11, 2008 at 12:46 PM, Uwe Hermann <uwe at hermann-uwe.de> wrote:
> On Fri, Jul 11, 2008 at 06:50:27PM +0200, Carl-Daniel Hailfinger wrote:
>> > +           pci at 15,2 {
>> > +                   /config/("southbridge/amd/cs5536/ide");
>> > +                   enable_ide = "1";
>> >
>>
>> What about removing the line above? In theory, stuff should still compile.
>
> Yep, compiles, no idea if it actually works (or how) on the board.
>
> New patch attached.

taking that out seems a mistake. Until we can test, leave too much
stuff in, don't take too much stuff out.

ron




More information about the coreboot mailing list