[coreboot] [PATCH] Fix superio enable calls

Rudolf Marek r.marek at assembler.cz
Tue Jan 25 21:59:01 CET 2011


While hunting yet another bug, I noticed that part of my serial log is missing. 
Especially I missed the Allocating & Setting resources.

It turns out that the code which enables specific LDN is somewhat buggy:

  static void w83627dhg_pnp_enable(device_t dev)
-	if (!dev->enabled)
-		return;
-	pnp_set_enable(dev, 0);

So instead of enable the device the device gets disabled. However after some 
time the serial line gets back, most likely some "enable resources" might fix it.

I'm attaching patch which somewhat fixes the problem and changes the function to 
look same in all superio code. Some boards even did not convert the dev->enabled 
to 0,1 values.

Also makes me wonder some sio call pnp_enable directly, which has yet another 
semantics, it wont enable already enabled devices just disable those which 
declared not enabled. And no PnP enter conf magic is called (maybe this is the 

Signed-off-by: Rudolf Marek <r.marek at assembler.cz>

Not abuild tested, works on dhg. I need to know if this fix is OK this way.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_enable_calls.patch
Type: text/x-diff
Size: 9040 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20110125/d03198db/attachment.bin>

More information about the coreboot mailing list