On 11.07.2008 15:56, Uwe Hermann wrote:
> See patch.
>
> Someone with the board should check if '15,2' is really correct, and
> whether it makes sense to enable IDE or not for this board.
>
> Thanks, Uwe.
>   

> Fix a build-error because of a missing 'enable_ide'.
>   

In theory, this should not cause a build error. Then again, our DTS
concept still needs some tweaks.

> Signed-off-by: Uwe Hermann <[EMAIL PROTECTED]>
>
> Index: mainboard/adl/msm800sev/dts
> ===================================================================
> --- mainboard/adl/msm800sev/dts       (Revision 694)
> +++ mainboard/adl/msm800sev/dts       (Arbeitskopie)
> @@ -37,6 +37,10 @@
>               [EMAIL PROTECTED],1 {
>                       /config/("southbridge/amd/cs5536/dts");
>               };
> +             [EMAIL PROTECTED],2 {
> +                     /config/("southbridge/amd/cs5536/ide");
> +                     enable_ide = "1";
>   

What about removing the line above? In theory, stuff should still compile.

> +             };
>               [EMAIL PROTECTED] {
>                       /config/("superio/winbond/w83627hf/dts");
>                       com1enable = "1";

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/


-- 
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to