On Wed, Aug 15, 2007 at 11:02:29PM -0400, Jonathan Sturges wrote:
> Thanks for the updated 5530 code!
> However, I'm having some difficulty enabling the IDE controllers when using
> this patch. As far as I can tell, configuration parsing doesn't appear to
> work for new ideX_enable flags.
>
> The 'register "ide0_enable" = "1"' configuration option seemed to have no
> effect on my 5530 system. Even though both IDE controllers were enabled in
> Config.lb, when I'd test the resulting image, IDE0 would be disabled and
> IDE1 enabled. In the end, to make it work, I changed:
> "if (conf->ide0_enable) {"
> to:
> "if ( 1 ) {"
> ...in cs5530_ide.c to get IDE0 enabled. This forces it to work.The placement of the ide0_enable lines is important. I think they should _not_ be within the IDE section. Check src/mainboard/asi/mb_5blmp/Config.lb for an example which worked for me, and please report if that fixes the problem. Uwe. -- http://www.hermann-uwe.de | http://www.holsham-traders.de http://www.crazy-hacks.org | http://www.unmaintained-free-software.org Uwe, I'm still having problems. The ideX_enable register values are within the "chip southbridge/amd/cs5530" context in my Config.lb, which matches the ASI mb-5blmp you cited. This is grasping at straws, but my Config.lb inconsistently uses tabs and spaces... any issues there, beyond aesthetics? Anyway, FWIW, I attached the Config.lb I'm working from. Any hints appreciated. thanks, Jonathan ____________________________________________________________________________________ Pinpoint customers who are looking for what you sell. http://searchmarketing.yahoo.com/
Config.lb
Description: Binary data
-- linuxbios mailing list [email protected] http://www.linuxbios.org/mailman/listinfo/linuxbios
