summaryrefslogtreecommitdiff
path: root/src/superio/smsc/lpc47n217/superio.c
diff options
context:
space:
mode:
authorRudolf Marek <r.marek@assembler.cz>2011-02-19 14:51:31 +0000
committerRudolf Marek <r.marek@assembler.cz>2011-02-19 14:51:31 +0000
commit0c8e664713d4dc726bedb5ba0b2e356eed9ae14c (patch)
treea81752632b470dfcb252e15865c3f2aa7de9bfe0 /src/superio/smsc/lpc47n217/superio.c
parentb8cdd9b4506309ced1529862d7598131a5cb4ea7 (diff)
downloadcoreboot-0c8e664713d4dc726bedb5ba0b2e356eed9ae14c.tar.xz
It turns out that the code which enables specific LDN is somewhat buggy.
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. Signed-off-by: Rudolf Marek <r.marek@assembler.cz> Acked-by: Peter Stuge <peter@stuge.se> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6373 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'src/superio/smsc/lpc47n217/superio.c')
-rw-r--r--src/superio/smsc/lpc47n217/superio.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/superio/smsc/lpc47n217/superio.c b/src/superio/smsc/lpc47n217/superio.c
index 4cc580643f..b1455330d0 100644
--- a/src/superio/smsc/lpc47n217/superio.c
+++ b/src/superio/smsc/lpc47n217/superio.c
@@ -119,7 +119,7 @@ static void lpc47n217_pnp_enable_resources(device_t dev)
static void lpc47n217_pnp_enable(device_t dev)
{
pnp_enter_conf_state(dev);
- lpc47n217_pnp_set_enable(dev, (dev->enabled) ? 1 : 0);
+ lpc47n217_pnp_set_enable(dev, !!dev->enabled);
pnp_exit_conf_state(dev);
}