[LinuxBIOS] [PATCH] Reimplementation/fixing of CS5530/CS5530A southbridge code

[Jonathan Sturges]

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/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Config.lb
Type: application/octet-stream
Size: 4691 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070829/6dc6642d/attachment.obj>