2005-12-21 15:49:51 +00:00
|
|
|
#include <sys/cdefs.h>
|
|
|
|
__FBSDID("$FreeBSD$");
|
|
|
|
|
|
|
|
#include <sys/param.h>
|
2007-01-11 19:56:24 +00:00
|
|
|
#include <sys/bus.h>
|
2005-12-21 15:49:51 +00:00
|
|
|
#include <sys/kernel.h>
|
2007-01-11 19:56:24 +00:00
|
|
|
#include <sys/lock.h>
|
2005-12-21 15:49:51 +00:00
|
|
|
#include <sys/module.h>
|
2007-01-11 19:56:24 +00:00
|
|
|
#include <sys/mutex.h>
|
|
|
|
#include <sys/systm.h>
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
#include <machine/bus.h>
|
|
|
|
#include <machine/resource.h>
|
|
|
|
#include <sys/rman.h>
|
|
|
|
|
|
|
|
#include <dev/pci/pcivar.h>
|
|
|
|
#include <dev/pci/pcireg.h>
|
|
|
|
|
|
|
|
#include <dev/smbus/smbconf.h>
|
|
|
|
#include "smbus_if.h"
|
|
|
|
|
|
|
|
#define AMDSMB_DEBUG(x) if (amdsmb_debug) (x)
|
|
|
|
|
|
|
|
#ifdef DEBUG
|
|
|
|
static int amdsmb_debug = 1;
|
|
|
|
#else
|
|
|
|
static int amdsmb_debug = 0;
|
|
|
|
#endif
|
|
|
|
|
|
|
|
#define AMDSMB_VENDORID_AMD 0x1022
|
|
|
|
#define AMDSMB_DEVICEID_AMD8111_SMB2 0x746a
|
|
|
|
|
|
|
|
/*
|
|
|
|
* ACPI 3.0, Chapter 12, Embedded Controller Interface.
|
|
|
|
*/
|
|
|
|
#define EC_DATA 0x00 /* data register */
|
|
|
|
#define EC_SC 0x04 /* status of controller */
|
|
|
|
#define EC_CMD 0x04 /* command register */
|
|
|
|
|
|
|
|
#define EC_SC_IBF 0x02 /* data ready for embedded controller */
|
|
|
|
#define EC_SC_OBF 0x01 /* data ready for host */
|
|
|
|
#define EC_CMD_WR 0x81 /* write EC */
|
|
|
|
#define EC_CMD_RD 0x80 /* read EC */
|
|
|
|
|
|
|
|
/*
|
|
|
|
* ACPI 3.0, Chapter 12, SMBus Host Controller Interface.
|
|
|
|
*/
|
|
|
|
#define SMB_PRTCL 0x00 /* protocol */
|
|
|
|
#define SMB_STS 0x01 /* status */
|
|
|
|
#define SMB_ADDR 0x02 /* address */
|
|
|
|
#define SMB_CMD 0x03 /* command */
|
|
|
|
#define SMB_DATA 0x04 /* 32 data registers */
|
|
|
|
#define SMB_BCNT 0x24 /* number of data bytes */
|
|
|
|
#define SMB_ALRM_A 0x25 /* alarm address */
|
|
|
|
#define SMB_ALRM_D 0x26 /* 2 bytes alarm data */
|
|
|
|
|
|
|
|
#define SMB_STS_DONE 0x80
|
|
|
|
#define SMB_STS_ALRM 0x40
|
|
|
|
#define SMB_STS_RES 0x20
|
|
|
|
#define SMB_STS_STATUS 0x1f
|
|
|
|
#define SMB_STS_OK 0x00 /* OK */
|
|
|
|
#define SMB_STS_UF 0x07 /* Unknown Failure */
|
|
|
|
#define SMB_STS_DANA 0x10 /* Device Address Not Acknowledged */
|
|
|
|
#define SMB_STS_DED 0x11 /* Device Error Detected */
|
|
|
|
#define SMB_STS_DCAD 0x12 /* Device Command Access Denied */
|
|
|
|
#define SMB_STS_UE 0x13 /* Unknown Error */
|
|
|
|
#define SMB_STS_DAD 0x17 /* Device Access Denied */
|
|
|
|
#define SMB_STS_T 0x18 /* Timeout */
|
|
|
|
#define SMB_STS_HUP 0x19 /* Host Unsupported Protocol */
|
|
|
|
#define SMB_STS_B 0x1a /* Busy */
|
|
|
|
#define SMB_STS_PEC 0x1f /* PEC (CRC-8) Error */
|
|
|
|
|
|
|
|
#define SMB_PRTCL_WRITE 0x00
|
|
|
|
#define SMB_PRTCL_READ 0x01
|
|
|
|
#define SMB_PRTCL_QUICK 0x02
|
|
|
|
#define SMB_PRTCL_BYTE 0x04
|
|
|
|
#define SMB_PRTCL_BYTE_DATA 0x06
|
|
|
|
#define SMB_PRTCL_WORD_DATA 0x08
|
|
|
|
#define SMB_PRTCL_BLOCK_DATA 0x0a
|
|
|
|
#define SMB_PRTCL_PROC_CALL 0x0c
|
|
|
|
#define SMB_PRTCL_BLOCK_PROC_CALL 0x0d
|
|
|
|
#define SMB_PRTCL_PEC 0x80
|
|
|
|
|
|
|
|
struct amdsmb_softc {
|
|
|
|
int rid;
|
|
|
|
struct resource *res;
|
|
|
|
bus_space_tag_t smbst;
|
|
|
|
bus_space_handle_t smbsh;
|
|
|
|
device_t smbus;
|
2007-01-11 19:56:24 +00:00
|
|
|
struct mtx lock;
|
2005-12-21 15:49:51 +00:00
|
|
|
};
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
#define AMDSMB_LOCK(amdsmb) mtx_lock(&(amdsmb)->lock)
|
|
|
|
#define AMDSMB_UNLOCK(amdsmb) mtx_unlock(&(amdsmb)->lock)
|
|
|
|
#define AMDSMB_LOCK_ASSERT(amdsmb) mtx_assert(&(amdsmb)->lock, MA_OWNED)
|
|
|
|
|
|
|
|
#define AMDSMB_ECINB(amdsmb, register) \
|
2005-12-21 15:49:51 +00:00
|
|
|
(bus_space_read_1(amdsmb->smbst, amdsmb->smbsh, register))
|
|
|
|
#define AMDSMB_ECOUTB(amdsmb, register, value) \
|
|
|
|
(bus_space_write_1(amdsmb->smbst, amdsmb->smbsh, register, value))
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
static int amdsmb_detach(device_t dev);
|
|
|
|
|
2005-12-21 15:49:51 +00:00
|
|
|
static int
|
|
|
|
amdsmb_probe(device_t dev)
|
|
|
|
{
|
|
|
|
u_int16_t vid;
|
|
|
|
u_int16_t did;
|
|
|
|
|
|
|
|
vid = pci_get_vendor(dev);
|
|
|
|
did = pci_get_device(dev);
|
|
|
|
|
|
|
|
if (vid == AMDSMB_VENDORID_AMD) {
|
|
|
|
switch(did) {
|
|
|
|
case AMDSMB_DEVICEID_AMD8111_SMB2:
|
|
|
|
device_set_desc(dev, "AMD-8111 SMBus 2.0 Controller");
|
|
|
|
return (BUS_PROBE_DEFAULT);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return (ENXIO);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_attach(device_t dev)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *amdsmb_sc = device_get_softc(dev);
|
|
|
|
|
|
|
|
/* Allocate I/O space */
|
|
|
|
amdsmb_sc->rid = PCIR_BAR(0);
|
|
|
|
|
|
|
|
amdsmb_sc->res = bus_alloc_resource_any(dev, SYS_RES_IOPORT,
|
|
|
|
&amdsmb_sc->rid, RF_ACTIVE);
|
|
|
|
|
|
|
|
if (amdsmb_sc->res == NULL) {
|
|
|
|
device_printf(dev, "could not map i/o space\n");
|
|
|
|
return (ENXIO);
|
|
|
|
}
|
|
|
|
|
|
|
|
amdsmb_sc->smbst = rman_get_bustag(amdsmb_sc->res);
|
|
|
|
amdsmb_sc->smbsh = rman_get_bushandle(amdsmb_sc->res);
|
2007-01-11 19:56:24 +00:00
|
|
|
mtx_init(&amdsmb_sc->lock, device_get_nameunit(dev), "amdsmb", MTX_DEF);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
/* Allocate a new smbus device */
|
|
|
|
amdsmb_sc->smbus = device_add_child(dev, "smbus", -1);
|
2007-01-11 19:56:24 +00:00
|
|
|
if (!amdsmb_sc->smbus) {
|
|
|
|
amdsmb_detach(dev);
|
2005-12-21 15:49:51 +00:00
|
|
|
return (EINVAL);
|
2007-01-11 19:56:24 +00:00
|
|
|
}
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
bus_generic_attach(dev);
|
|
|
|
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_detach(device_t dev)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *amdsmb_sc = device_get_softc(dev);
|
|
|
|
|
|
|
|
if (amdsmb_sc->smbus) {
|
|
|
|
device_delete_child(dev, amdsmb_sc->smbus);
|
|
|
|
amdsmb_sc->smbus = NULL;
|
|
|
|
}
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
mtx_destroy(&amdsmb_sc->lock);
|
2005-12-21 15:49:51 +00:00
|
|
|
if (amdsmb_sc->res)
|
|
|
|
bus_release_resource(dev, SYS_RES_IOPORT, amdsmb_sc->rid,
|
|
|
|
amdsmb_sc->res);
|
|
|
|
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
amdsmb_callback(device_t dev, int index, void *data)
|
2005-12-21 15:49:51 +00:00
|
|
|
{
|
|
|
|
int error = 0;
|
|
|
|
|
|
|
|
switch (index) {
|
|
|
|
case SMB_REQUEST_BUS:
|
|
|
|
case SMB_RELEASE_BUS:
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
error = EINVAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_ec_wait_write(struct amdsmb_softc *sc)
|
|
|
|
{
|
|
|
|
int timeout = 500;
|
|
|
|
|
|
|
|
while (timeout-- && AMDSMB_ECINB(sc, EC_SC) & EC_SC_IBF)
|
|
|
|
DELAY(1);
|
|
|
|
if (timeout == 0) {
|
|
|
|
device_printf(sc->smbus, "timeout waiting for IBF to clear\n");
|
|
|
|
return (1);
|
|
|
|
}
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_ec_wait_read(struct amdsmb_softc *sc)
|
|
|
|
{
|
|
|
|
int timeout = 500;
|
|
|
|
|
|
|
|
while (timeout-- && ~AMDSMB_ECINB(sc, EC_SC) & EC_SC_OBF)
|
|
|
|
DELAY(1);
|
|
|
|
if (timeout == 0) {
|
|
|
|
device_printf(sc->smbus, "timeout waiting for OBF to set\n");
|
|
|
|
return (1);
|
|
|
|
}
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_ec_read(struct amdsmb_softc *sc, u_char addr, u_char *data)
|
|
|
|
{
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK_ASSERT(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
if (amdsmb_ec_wait_write(sc))
|
|
|
|
return (1);
|
|
|
|
AMDSMB_ECOUTB(sc, EC_CMD, EC_CMD_RD);
|
|
|
|
|
|
|
|
if (amdsmb_ec_wait_write(sc))
|
|
|
|
return (1);
|
|
|
|
AMDSMB_ECOUTB(sc, EC_DATA, addr);
|
|
|
|
|
|
|
|
if (amdsmb_ec_wait_read(sc))
|
|
|
|
return (1);
|
|
|
|
*data = AMDSMB_ECINB(sc, EC_DATA);
|
|
|
|
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_ec_write(struct amdsmb_softc *sc, u_char addr, u_char data)
|
|
|
|
{
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK_ASSERT(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
if (amdsmb_ec_wait_write(sc))
|
|
|
|
return (1);
|
|
|
|
AMDSMB_ECOUTB(sc, EC_CMD, EC_CMD_WR);
|
|
|
|
|
|
|
|
if (amdsmb_ec_wait_write(sc))
|
|
|
|
return (1);
|
|
|
|
AMDSMB_ECOUTB(sc, EC_DATA, addr);
|
|
|
|
|
|
|
|
if (amdsmb_ec_wait_write(sc))
|
|
|
|
return (1);
|
|
|
|
AMDSMB_ECOUTB(sc, EC_DATA, data);
|
|
|
|
|
|
|
|
return (0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_wait(struct amdsmb_softc *sc)
|
|
|
|
{
|
|
|
|
u_char sts, temp;
|
|
|
|
int error, count;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK_ASSERT(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_read(sc, SMB_PRTCL, &temp);
|
|
|
|
if (temp != 0)
|
|
|
|
{
|
|
|
|
count = 10000;
|
|
|
|
do {
|
|
|
|
DELAY(500);
|
|
|
|
amdsmb_ec_read(sc, SMB_PRTCL, &temp);
|
|
|
|
} while (temp != 0 && count--);
|
|
|
|
if (count == 0)
|
|
|
|
return (SMB_ETIMEOUT);
|
|
|
|
}
|
|
|
|
|
|
|
|
amdsmb_ec_read(sc, SMB_STS, &sts);
|
|
|
|
sts &= SMB_STS_STATUS;
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: STS=0x%x\n", sts));
|
|
|
|
|
|
|
|
switch (sts) {
|
|
|
|
case SMB_STS_OK:
|
|
|
|
error = SMB_ENOERR;
|
|
|
|
break;
|
|
|
|
case SMB_STS_DANA:
|
|
|
|
error = SMB_ENOACK;
|
|
|
|
break;
|
|
|
|
case SMB_STS_B:
|
|
|
|
error = SMB_EBUSY;
|
|
|
|
break;
|
|
|
|
case SMB_STS_T:
|
|
|
|
error = SMB_ETIMEOUT;
|
|
|
|
break;
|
|
|
|
case SMB_STS_DCAD:
|
|
|
|
case SMB_STS_DAD:
|
|
|
|
case SMB_STS_HUP:
|
|
|
|
error = SMB_ENOTSUPP;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
error = SMB_EBUSERR;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_quick(device_t dev, u_char slave, int how)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
u_char protocol;
|
|
|
|
int error;
|
|
|
|
|
|
|
|
protocol = SMB_PRTCL_QUICK;
|
|
|
|
|
|
|
|
switch (how) {
|
|
|
|
case SMB_QWRITE:
|
|
|
|
protocol |= SMB_PRTCL_WRITE;
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: QWRITE to 0x%x", slave));
|
|
|
|
break;
|
|
|
|
case SMB_QREAD:
|
|
|
|
protocol |= SMB_PRTCL_READ;
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: QREAD to 0x%x", slave));
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
panic("%s: unknown QUICK command (%x)!", __func__, how);
|
|
|
|
}
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, protocol);
|
|
|
|
|
|
|
|
error = amdsmb_wait(sc);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf(", error=0x%x\n", error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_sendb(device_t dev, u_char slave, char byte)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, byte);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_WRITE | SMB_PRTCL_BYTE);
|
|
|
|
|
|
|
|
error = amdsmb_wait(sc);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: SENDB to 0x%x, byte=0x%x, error=0x%x\n",
|
|
|
|
slave, byte, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_recvb(device_t dev, u_char slave, char *byte)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_READ | SMB_PRTCL_BYTE);
|
|
|
|
|
|
|
|
if ((error = amdsmb_wait(sc)) == SMB_ENOERR)
|
|
|
|
amdsmb_ec_read(sc, SMB_DATA, byte);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: RECVB from 0x%x, byte=0x%x, error=0x%x\n",
|
|
|
|
slave, *byte, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_writeb(device_t dev, u_char slave, char cmd, char byte)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
|
|
|
amdsmb_ec_write(sc, SMB_DATA, byte);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_WRITE | SMB_PRTCL_BYTE_DATA);
|
|
|
|
|
|
|
|
error = amdsmb_wait(sc);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: WRITEB to 0x%x, cmd=0x%x, byte=0x%x, "
|
|
|
|
"error=0x%x\n", slave, cmd, byte, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_readb(device_t dev, u_char slave, char cmd, char *byte)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_READ | SMB_PRTCL_BYTE_DATA);
|
|
|
|
|
|
|
|
if ((error = amdsmb_wait(sc)) == SMB_ENOERR)
|
|
|
|
amdsmb_ec_read(sc, SMB_DATA, byte);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: READB from 0x%x, cmd=0x%x, byte=0x%x, "
|
|
|
|
"error=0x%x\n", slave, cmd, (unsigned char)*byte, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_writew(device_t dev, u_char slave, char cmd, short word)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
|
|
|
amdsmb_ec_write(sc, SMB_DATA, word);
|
|
|
|
amdsmb_ec_write(sc, SMB_DATA + 1, word >> 8);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_WRITE | SMB_PRTCL_WORD_DATA);
|
|
|
|
|
|
|
|
error = amdsmb_wait(sc);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: WRITEW to 0x%x, cmd=0x%x, word=0x%x, "
|
|
|
|
"error=0x%x\n", slave, cmd, word, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_readw(device_t dev, u_char slave, char cmd, short *word)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
|
|
|
u_char temp[2];
|
|
|
|
int error;
|
|
|
|
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_READ | SMB_PRTCL_WORD_DATA);
|
|
|
|
|
|
|
|
if ((error = amdsmb_wait(sc)) == SMB_ENOERR) {
|
|
|
|
amdsmb_ec_read(sc, SMB_DATA + 0, &temp[0]);
|
|
|
|
amdsmb_ec_read(sc, SMB_DATA + 1, &temp[1]);
|
|
|
|
*word = temp[0] | (temp[1] << 8);
|
|
|
|
}
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: READW from 0x%x, cmd=0x%x, word=0x%x, "
|
|
|
|
"error=0x%x\n", slave, cmd, (unsigned short)*word, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
amdsmb_bwrite(device_t dev, u_char slave, char cmd, u_char count, char *buf)
|
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
u_char i;
|
2005-12-21 15:49:51 +00:00
|
|
|
int error;
|
|
|
|
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
if (count < 1 || count > 32)
|
|
|
|
return (SMB_EINVAL);
|
2007-01-11 19:56:24 +00:00
|
|
|
|
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_BCNT, count);
|
|
|
|
for (i = 0; i < count; i++)
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_DATA + i, buf[i]);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_WRITE | SMB_PRTCL_BLOCK_DATA);
|
|
|
|
|
|
|
|
error = amdsmb_wait(sc);
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: WRITEBLK to 0x%x, count=0x%x, cmd=0x%x, "
|
|
|
|
"error=0x%x", slave, count, cmd, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
amdsmb_bread(device_t dev, u_char slave, char cmd, u_char *count, char *buf)
|
2005-12-21 15:49:51 +00:00
|
|
|
{
|
|
|
|
struct amdsmb_softc *sc = (struct amdsmb_softc *)device_get_softc(dev);
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
u_char data, len, i;
|
2005-12-21 15:49:51 +00:00
|
|
|
int error;
|
|
|
|
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
if (*count < 1 || *count > 32)
|
|
|
|
return (SMB_EINVAL);
|
2007-01-11 19:56:24 +00:00
|
|
|
|
|
|
|
AMDSMB_LOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
amdsmb_ec_write(sc, SMB_CMD, cmd);
|
|
|
|
amdsmb_ec_write(sc, SMB_ADDR, slave);
|
|
|
|
amdsmb_ec_write(sc, SMB_PRTCL, SMB_PRTCL_READ | SMB_PRTCL_BLOCK_DATA);
|
|
|
|
|
|
|
|
if ((error = amdsmb_wait(sc)) == SMB_ENOERR) {
|
|
|
|
amdsmb_ec_read(sc, SMB_BCNT, &len);
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
for (i = 0; i < len; i++) {
|
|
|
|
amdsmb_ec_read(sc, SMB_DATA + i, &data);
|
|
|
|
if (i < *count)
|
|
|
|
buf[i] = data;
|
|
|
|
}
|
|
|
|
*count = len;
|
2005-12-21 15:49:51 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
AMDSMB_DEBUG(printf("amdsmb: READBLK to 0x%x, count=0x%x, cmd=0x%x, "
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
"error=0x%x", slave, *count, cmd, error));
|
2007-01-11 19:56:24 +00:00
|
|
|
AMDSMB_UNLOCK(sc);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
return (error);
|
|
|
|
}
|
|
|
|
|
|
|
|
static device_method_t amdsmb_methods[] = {
|
|
|
|
/* Device interface */
|
|
|
|
DEVMETHOD(device_probe, amdsmb_probe),
|
|
|
|
DEVMETHOD(device_attach, amdsmb_attach),
|
|
|
|
DEVMETHOD(device_detach, amdsmb_detach),
|
|
|
|
|
|
|
|
/* SMBus interface */
|
|
|
|
DEVMETHOD(smbus_callback, amdsmb_callback),
|
|
|
|
DEVMETHOD(smbus_quick, amdsmb_quick),
|
|
|
|
DEVMETHOD(smbus_sendb, amdsmb_sendb),
|
|
|
|
DEVMETHOD(smbus_recvb, amdsmb_recvb),
|
|
|
|
DEVMETHOD(smbus_writeb, amdsmb_writeb),
|
|
|
|
DEVMETHOD(smbus_readb, amdsmb_readb),
|
|
|
|
DEVMETHOD(smbus_writew, amdsmb_writew),
|
|
|
|
DEVMETHOD(smbus_readw, amdsmb_readw),
|
|
|
|
DEVMETHOD(smbus_bwrite, amdsmb_bwrite),
|
|
|
|
DEVMETHOD(smbus_bread, amdsmb_bread),
|
|
|
|
|
|
|
|
{ 0, 0 }
|
|
|
|
};
|
|
|
|
|
|
|
|
static devclass_t amdsmb_devclass;
|
|
|
|
|
|
|
|
static driver_t amdsmb_driver = {
|
|
|
|
"amdsmb",
|
|
|
|
amdsmb_methods,
|
|
|
|
sizeof(struct amdsmb_softc),
|
|
|
|
};
|
|
|
|
|
|
|
|
DRIVER_MODULE(amdsmb, pci, amdsmb_driver, amdsmb_devclass, 0, 0);
|
Minor overhaul of SMBus support:
- Change smbus_callback() to pass a void * rather than caddr_t.
- Change smbus_bread() to pass a pointer to the count and have it be an
in/out parameter. The input is the size of the buffer (same as before),
but on return it will contain the actual amount of data read back from
the bus. Note that this value may be larger than the input value. It
is up to the caller to treat this as an error if desired.
- Change the SMB_BREAD ioctl to write out the updated struct smbcmd which
will contain the actual number of bytes read in the 'count' field. To
preserve the previous ABI, the old ioctl value is mapped to SMB_OLD_BREAD
which doesn't copy the updated smbcmd back out to userland. I doubt anyone
actually used the old BREAD anyway as it was rediculous to do a bulk-read
but not tell the using program how much data was actually read.
- Make the smbus driver and devclass public in the smbus module and
push all the DRIVER_MODULE()'s for attaching the smbus driver to
various foosmb drivers out into the foosmb modules. This makes all
the foosmb logic centralized and allows new foosmb modules to be
self-contained w/o having to hack smbus.c everytime a new smbus driver
is added.
- Add a new SMB_EINVAL error bit and use it in place of EINVAL to return
an error for bad arguments (such as invalid counts for bread and bwrite).
- Map SMB bus error bits to EIO in smbus_error().
- Make the smbus driver call bus_generic_probe() and require child drivers
such as smb(4) to create device_t's via identify routines. Previously,
smbus just created one anonymous device during attach, and if you had
multiple drivers that could attach it was just random chance as to which
driver got to probe for the sole device_t first.
- Add a mutex to the smbus(4) softc and use it in place of dummy splhigh()
to protect the 'owner' field and perform necessary synchronization for
smbus_request_bus() and smbus_release_bus().
- Change the bread() and bwrite() methods of alpm(4), amdpm(4), and
viapm(4) to only perform a single transaction and not try to use a
loop of multiple transactions for a large request. The framing and
commands to use for a large transaction depend on the upper-layer
protocol (such as SSIF for IPMI over SMBus) from what I can tell, and the
smb(4) driver never allowed bulk read/writes of more than 32-bytes
anyway. The other smb drivers only performed single transactions.
- Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
- Use SMB_xxx errors in viapm(4).
- Destroy ichsmb(4)'s mutex after bus_generic_detach() to avoid problems
from child devices making smb upcalls that would use the mutex during
their detach methods.
MFC after: 1 week
Reviewed by: jmg (mostly)
2006-09-11 20:52:41 +00:00
|
|
|
DRIVER_MODULE(smbus, amdsmb, smbus_driver, smbus_devclass, 0, 0);
|
2005-12-21 15:49:51 +00:00
|
|
|
|
|
|
|
MODULE_DEPEND(amdsmb, pci, 1, 1, 1);
|
|
|
|
MODULE_DEPEND(amdsmb, smbus, SMBUS_MINVER, SMBUS_PREFVER, SMBUS_MAXVER);
|
|
|
|
MODULE_VERSION(amdsmb, 1);
|