diff options
| author | hselasky <hselasky@FreeBSD.org> | 2012-04-02 10:50:42 +0000 |
|---|---|---|
| committer | hselasky <hselasky@FreeBSD.org> | 2012-04-02 10:50:42 +0000 |
| commit | 126953ccbe190fe51094d1ccefda3226d1e5140c (patch) | |
| tree | 70a9d586e75b3d8e087069f07020564df44b0b99 /sys/dev/usb/serial/usb_serial.c | |
| parent | 56216c3bc99342baf89a814301e85dffe7a62a9b (diff) | |
| download | FreeBSD-src-126953ccbe190fe51094d1ccefda3226d1e5140c.zip FreeBSD-src-126953ccbe190fe51094d1ccefda3226d1e5140c.tar.gz | |
Fix compiler warnings, mostly signed issues,
when USB modules are compiled with WARNS=9.
MFC after: 1 weeks
Diffstat (limited to 'sys/dev/usb/serial/usb_serial.c')
| -rw-r--r-- | sys/dev/usb/serial/usb_serial.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/sys/dev/usb/serial/usb_serial.c b/sys/dev/usb/serial/usb_serial.c index 08ce2ca..a2c1f97 100644 --- a/sys/dev/usb/serial/usb_serial.c +++ b/sys/dev/usb/serial/usb_serial.c @@ -236,14 +236,14 @@ ucom_unit_free(int unit) */ int ucom_attach(struct ucom_super_softc *ssc, struct ucom_softc *sc, - uint32_t subunits, void *parent, + int subunits, void *parent, const struct ucom_callback *callback, struct mtx *mtx) { - uint32_t subunit; + int subunit; int error = 0; if ((sc == NULL) || - (subunits == 0) || + (subunits <= 0) || (callback == NULL)) { return (EINVAL); } @@ -293,7 +293,7 @@ ucom_attach(struct ucom_super_softc *ssc, struct ucom_softc *sc, void ucom_detach(struct ucom_super_softc *ssc, struct ucom_softc *sc) { - uint32_t subunit; + int subunit; if (ssc->sc_subunits == 0) return; /* not initialized */ @@ -1089,11 +1089,6 @@ ucom_param(struct tty *tp, struct termios *t) DPRINTF("sc = %p\n", sc); /* Check requested parameters. */ - if (t->c_ospeed < 0) { - DPRINTF("negative ospeed\n"); - error = EINVAL; - goto done; - } if (t->c_ispeed && (t->c_ispeed != t->c_ospeed)) { DPRINTF("mismatch ispeed and ospeed\n"); error = EINVAL; |
