diff options
| author | Mark Kettenis <kettenis@cvs.openbsd.org> | 2006-01-12 22:25:47 +0000 |
|---|---|---|
| committer | Mark Kettenis <kettenis@cvs.openbsd.org> | 2006-01-12 22:25:47 +0000 |
| commit | 4ae88a5bf1ec8006777c1e861df6f208de5b715b (patch) | |
| tree | 5a8e42856fef1699f5df7aad0ef78e099741c452 /sys/dev/pci/viapm.c | |
| parent | 8b17857c6ead71a1ad4bac43af6fa5c6e9ff6521 (diff) | |
Don't start new command if bus is busy and kill command on timeout.
Diffstat (limited to 'sys/dev/pci/viapm.c')
| -rw-r--r-- | sys/dev/pci/viapm.c | 34 |
1 files changed, 27 insertions, 7 deletions
diff --git a/sys/dev/pci/viapm.c b/sys/dev/pci/viapm.c index 80fb6f86eb4..1b0d103c120 100644 --- a/sys/dev/pci/viapm.c +++ b/sys/dev/pci/viapm.c @@ -1,4 +1,4 @@ -/* $OpenBSD: viapm.c,v 1.4 2006/01/01 20:52:27 deraadt Exp $ */ +/* $OpenBSD: viapm.c,v 1.5 2006/01/12 22:25:46 kettenis Exp $ */ /* * Copyright (c) 2005 Mark Kettenis <kettenis@openbsd.org> @@ -252,6 +252,13 @@ viapm_i2c_exec(void *cookie, i2c_op_t op, i2c_addr_t addr, cmdlen, len, flags, bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS), VIAPM_SMB_HS_BITS)); + /* Check if there's a transfer already running */ + st = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS); + DPRINTF(("%s: exec: st 0x%b\n", sc->sc_dev.dv_xname, st, + VIAPM_SMB_HS_BITS)); + if (st & VIAPM_SMB_HS_BUSY) + return (1); + if (cold || sc->sc_poll) flags |= I2C_F_POLL; @@ -312,22 +319,35 @@ viapm_i2c_exec(void *cookie, i2c_op_t op, i2c_addr_t addr, break; DELAY(VIAPM_DELAY); } - if (st & VIAPM_SMB_HS_BUSY) { - printf("%s: timeout, status 0x%b\n", - sc->sc_dev.dv_xname, st, VIAPM_SMB_HS_BITS); - return (1); - } + if (st & VIAPM_SMB_HS_BUSY) + goto timeout; viapm_intr(sc); } else { /* Wait for interrupt */ if (tsleep(sc, PRIBIO, "iicexec", VIAPM_TIMEOUT * hz)) - return (1); + goto timeout; } if (sc->sc_i2c_xfer.error) return (1); return (0); + +timeout: + /* + * Transfer timeout. Kill the transaction and clear status bits. + */ + printf("%s: timeout, status 0x%b\n", sc->sc_dev.dv_xname, st, + VIAPM_SMB_HS_BITS); + bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HC, + VIAPM_SMB_HC_KILL); + DELAY(VIAPM_DELAY); + st = bus_space_read_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS); + if ((st & VIAPM_SMB_HS_FAILED) == 0) + printf("%s: transaction abort failed, status 0x%b\n", + sc->sc_dev.dv_xname, st, VIAPM_SMB_HS_BITS); + bus_space_write_1(sc->sc_iot, sc->sc_ioh, VIAPM_SMB_HS, st); + return (1); } int |