[coreboot] patch: two bugs in the cs5536 ide code

Mart Raudsepp mart.raudsepp at artecdesign.ee
Wed May 7 18:20:38 CEST 2008


> On Wed, May 7, 2008 at 1:04 AM, Mart Raudsepp
>> Just removing enable_ide as in southbridge/cs5536 would be better in my
>> opinion. DBE62 uses NAND not any IDE, so I was actually intending to
>> send a patch that sets enable_ide to 0, but now I saw this.
>
> why is that an issue? The default value is 0.
>
> ron

DBE62's own dts currently sets it to 1. Carl-Daniel's patch (original at
least) moves that to a separate block - I'm just saying that instead it
should be dropped, as it should be really 0, so the default will kick in
and no unwanted IDE setup would happen.


Mart Raudsepp





More information about the coreboot mailing list