On 11.07.2008 21:51, ron minnich wrote:
> On Fri, Jul 11, 2008 at 12:46 PM, Uwe Hermann <[EMAIL PROTECTED]> wrote:
>
>> On Fri, Jul 11, 2008 at 06:50:27PM +0200, Carl-Daniel Hailfinger wrote:
>>
>>>> + [EMAIL PROTECTED],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.
>
Yes, the MSM800SEV has IDE, so my line removal request was mistaken.
Your original patch is
Acked-by: Carl-Daniel Hailfinger <[EMAIL PROTECTED]>
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
--
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
