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.
On Wed, Aug 29, 2007 at 08:16:52PM -0700, Jonathan Sturges wrote:
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?
No, whitespace should not matter there.
Anyway, FWIW, I attached the Config.lb I'm working from. Any hints appreciated.
Please retry with the latest svn revision, I just committed a rewrite of the CS5530 code. I verified that it works on the ASI MB-5BLMP.
Uwe.
-
Jonathan Sturges -
Uwe Hermann