From 0c8e664713d4dc726bedb5ba0b2e356eed9ae14c Mon Sep 17 00:00:00 2001 From: Rudolf Marek Date: Sat, 19 Feb 2011 14:51:31 +0000 Subject: 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 Acked-by: Peter Stuge git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6373 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/superio/fintek/f71859/superio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/superio/fintek/f71859') diff --git a/src/superio/fintek/f71859/superio.c b/src/superio/fintek/f71859/superio.c index d7d71c6ec8..6c74d81516 100755 --- a/src/superio/fintek/f71859/superio.c +++ b/src/superio/fintek/f71859/superio.c @@ -74,7 +74,7 @@ static void f71859_pnp_enable(device_t dev) { pnp_enter_conf_state(dev); pnp_set_logical_device(dev); - (dev->enabled) ? pnp_set_enable(dev, 1) : pnp_set_enable(dev, 0); + pnp_set_enable(dev, !!dev->enabled); pnp_exit_conf_state(dev); } -- cgit v1.2.3