diff options
author | thompsa <thompsa@FreeBSD.org> | 2009-06-23 02:19:59 +0000 |
---|---|---|
committer | thompsa <thompsa@FreeBSD.org> | 2009-06-23 02:19:59 +0000 |
commit | 74c6c20b93f0513a453c6544f8b98ad55cbdc741 (patch) | |
tree | ee5c9acfe716a9835136731811e9e1a68751e278 /sys/dev/usb/serial/uvisor.c | |
parent | 52922c0742bc8545734b2903d6407b99664e84ed (diff) | |
download | FreeBSD-src-74c6c20b93f0513a453c6544f8b98ad55cbdc741.zip FreeBSD-src-74c6c20b93f0513a453c6544f8b98ad55cbdc741.tar.gz |
- Make struct usb_xfer opaque so that drivers can not access the internals
- Reduce the number of headers needed for a usb driver, the common case is just usb.h and usbdi.h
Diffstat (limited to 'sys/dev/usb/serial/uvisor.c')
-rw-r--r-- | sys/dev/usb/serial/uvisor.c | 69 |
1 files changed, 44 insertions, 25 deletions
diff --git a/sys/dev/usb/serial/uvisor.c b/sys/dev/usb/serial/uvisor.c index 1e39947..c7175e3 100644 --- a/sys/dev/usb/serial/uvisor.c +++ b/sys/dev/usb/serial/uvisor.c @@ -54,22 +54,34 @@ * Handspring Visor (Palmpilot compatible PDA) driver */ -#include "usbdevs.h" +#include <sys/stdint.h> +#include <sys/stddef.h> +#include <sys/param.h> +#include <sys/queue.h> +#include <sys/types.h> +#include <sys/systm.h> +#include <sys/kernel.h> +#include <sys/bus.h> +#include <sys/linker_set.h> +#include <sys/module.h> +#include <sys/lock.h> +#include <sys/mutex.h> +#include <sys/condvar.h> +#include <sys/sysctl.h> +#include <sys/sx.h> +#include <sys/unistd.h> +#include <sys/callout.h> +#include <sys/malloc.h> +#include <sys/priv.h> + #include <dev/usb/usb.h> -#include <dev/usb/usb_mfunc.h> -#include <dev/usb/usb_error.h> -#include <dev/usb/usb_cdc.h> -#include <dev/usb/usb_ioctl.h> +#include <dev/usb/usbdi.h> +#include <dev/usb/usbdi_util.h> +#include "usbdevs.h" #define USB_DEBUG_VAR uvisor_debug - -#include <dev/usb/usb_core.h> #include <dev/usb/usb_debug.h> #include <dev/usb/usb_process.h> -#include <dev/usb/usb_request.h> -#include <dev/usb/usb_lookup.h> -#include <dev/usb/usb_util.h> -#include <dev/usb/usb_busdma.h> #include <dev/usb/serial/usb_serial.h> @@ -324,8 +336,8 @@ uvisor_attach(device_t dev) } /* clear stall at first run */ mtx_lock(&sc->sc_mtx); - usbd_transfer_set_stall(sc->sc_xfer[UVISOR_BULK_DT_WR]); - usbd_transfer_set_stall(sc->sc_xfer[UVISOR_BULK_DT_RD]); + usbd_xfer_set_stall(sc->sc_xfer[UVISOR_BULK_DT_WR]); + usbd_xfer_set_stall(sc->sc_xfer[UVISOR_BULK_DT_RD]); mtx_unlock(&sc->sc_mtx); error = ucom_attach(&sc->sc_super_ucom, &sc->sc_ucom, 1, sc, @@ -562,27 +574,29 @@ uvisor_stop_write(struct ucom_softc *ucom) } static void -uvisor_write_callback(struct usb_xfer *xfer) +uvisor_write_callback(struct usb_xfer *xfer, usb_error_t error) { - struct uvisor_softc *sc = xfer->priv_sc; + struct uvisor_softc *sc = usbd_xfer_softc(xfer); + struct usb_page_cache *pc; uint32_t actlen; switch (USB_GET_STATE(xfer)) { case USB_ST_SETUP: case USB_ST_TRANSFERRED: tr_setup: - if (ucom_get_data(&sc->sc_ucom, xfer->frbuffers, 0, + pc = usbd_xfer_get_frame(xfer, 0); + if (ucom_get_data(&sc->sc_ucom, pc, 0, UVISOR_BUFSIZE, &actlen)) { - xfer->frlengths[0] = actlen; + usbd_xfer_set_frame_len(xfer, 0, actlen); usbd_transfer_submit(xfer); } return; default: /* Error */ - if (xfer->error != USB_ERR_CANCELLED) { + if (error != USB_ERR_CANCELLED) { /* try to clear stall first */ - xfer->flags.stall_pipe = 1; + usbd_xfer_set_stall(xfer); goto tr_setup; } return; @@ -590,24 +604,29 @@ tr_setup: } static void -uvisor_read_callback(struct usb_xfer *xfer) +uvisor_read_callback(struct usb_xfer *xfer, usb_error_t error) { - struct uvisor_softc *sc = xfer->priv_sc; + struct uvisor_softc *sc = usbd_xfer_softc(xfer); + struct usb_page_cache *pc; + int actlen; + + usbd_xfer_status(xfer, &actlen, NULL, NULL, NULL); switch (USB_GET_STATE(xfer)) { case USB_ST_TRANSFERRED: - ucom_put_data(&sc->sc_ucom, xfer->frbuffers, 0, xfer->actlen); + pc = usbd_xfer_get_frame(xfer, 0); + ucom_put_data(&sc->sc_ucom, pc, 0, actlen); case USB_ST_SETUP: tr_setup: - xfer->frlengths[0] = xfer->max_data_length; + usbd_xfer_set_frame_len(xfer, 0, usbd_xfer_max_len(xfer)); usbd_transfer_submit(xfer); return; default: /* Error */ - if (xfer->error != USB_ERR_CANCELLED) { + if (error != USB_ERR_CANCELLED) { /* try to clear stall first */ - xfer->flags.stall_pipe = 1; + usbd_xfer_set_stall(xfer); goto tr_setup; } return; |