diff options
Diffstat (limited to 'drivers/usb/misc')
| -rw-r--r-- | drivers/usb/misc/idmouse.c | 3 | ||||
| -rw-r--r-- | drivers/usb/misc/iowarrior.c | 21 | ||||
| -rw-r--r-- | drivers/usb/misc/lvstest.c | 4 | ||||
| -rw-r--r-- | drivers/usb/misc/usb251xb.c | 59 | ||||
| -rw-r--r-- | drivers/usb/misc/uss720.c | 5 | 
5 files changed, 51 insertions, 41 deletions
diff --git a/drivers/usb/misc/idmouse.c b/drivers/usb/misc/idmouse.c index 8b9fd7534f69..502bfe30a077 100644 --- a/drivers/usb/misc/idmouse.c +++ b/drivers/usb/misc/idmouse.c @@ -347,6 +347,9 @@ static int idmouse_probe(struct usb_interface *interface,  	if (iface_desc->desc.bInterfaceClass != 0x0A)  		return -ENODEV; +	if (iface_desc->desc.bNumEndpoints < 1) +		return -ENODEV; +  	/* allocate memory for our device state and initialize it */  	dev = kzalloc(sizeof(*dev), GFP_KERNEL);  	if (dev == NULL) diff --git a/drivers/usb/misc/iowarrior.c b/drivers/usb/misc/iowarrior.c index 095778ff984d..37c63cb39714 100644 --- a/drivers/usb/misc/iowarrior.c +++ b/drivers/usb/misc/iowarrior.c @@ -781,12 +781,6 @@ static int iowarrior_probe(struct usb_interface *interface,  	iface_desc = interface->cur_altsetting;  	dev->product_id = le16_to_cpu(udev->descriptor.idProduct); -	if (iface_desc->desc.bNumEndpoints < 1) { -		dev_err(&interface->dev, "Invalid number of endpoints\n"); -		retval = -EINVAL; -		goto error; -	} -  	/* set up the endpoint information */  	for (i = 0; i < iface_desc->desc.bNumEndpoints; ++i) {  		endpoint = &iface_desc->endpoint[i].desc; @@ -797,6 +791,21 @@ static int iowarrior_probe(struct usb_interface *interface,  			/* this one will match for the IOWarrior56 only */  			dev->int_out_endpoint = endpoint;  	} + +	if (!dev->int_in_endpoint) { +		dev_err(&interface->dev, "no interrupt-in endpoint found\n"); +		retval = -ENODEV; +		goto error; +	} + +	if (dev->product_id == USB_DEVICE_ID_CODEMERCS_IOW56) { +		if (!dev->int_out_endpoint) { +			dev_err(&interface->dev, "no interrupt-out endpoint found\n"); +			retval = -ENODEV; +			goto error; +		} +	} +  	/* we have to check the report_size often, so remember it in the endianness suitable for our machine */  	dev->report_size = usb_endpoint_maxp(dev->int_in_endpoint);  	if ((dev->interface->cur_altsetting->desc.bInterfaceNumber == 0) && diff --git a/drivers/usb/misc/lvstest.c b/drivers/usb/misc/lvstest.c index 77176511658f..d3d124753266 100644 --- a/drivers/usb/misc/lvstest.c +++ b/drivers/usb/misc/lvstest.c @@ -366,6 +366,10 @@ static int lvs_rh_probe(struct usb_interface *intf,  	hdev = interface_to_usbdev(intf);  	desc = intf->cur_altsetting; + +	if (desc->desc.bNumEndpoints < 1) +		return -ENODEV; +  	endpoint = &desc->endpoint[0].desc;  	/* valid only for SS root hub */ diff --git a/drivers/usb/misc/usb251xb.c b/drivers/usb/misc/usb251xb.c index 4e18600dc9b4..91f66d68bcb7 100644 --- a/drivers/usb/misc/usb251xb.c +++ b/drivers/usb/misc/usb251xb.c @@ -375,18 +375,24 @@ static int usb251xb_get_ofdata(struct usb251xb *hub,  	if (of_get_property(np, "dynamic-power-switching", NULL))  		hub->conf_data2 |= BIT(7); -	if (of_get_property(np, "oc-delay-100us", NULL)) { -		hub->conf_data2 &= ~BIT(5); -		hub->conf_data2 &= ~BIT(4); -	} else if (of_get_property(np, "oc-delay-4ms", NULL)) { -		hub->conf_data2 &= ~BIT(5); -		hub->conf_data2 |= BIT(4); -	} else if (of_get_property(np, "oc-delay-8ms", NULL)) { -		hub->conf_data2 |= BIT(5); -		hub->conf_data2 &= ~BIT(4); -	} else if (of_get_property(np, "oc-delay-16ms", NULL)) { -		hub->conf_data2 |= BIT(5); -		hub->conf_data2 |= BIT(4); +	if (!of_property_read_u32(np, "oc-delay-us", property_u32)) { +		if (*property_u32 == 100) { +			/* 100 us*/ +			hub->conf_data2 &= ~BIT(5); +			hub->conf_data2 &= ~BIT(4); +		} else if (*property_u32 == 4000) { +			/* 4 ms */ +			hub->conf_data2 &= ~BIT(5); +			hub->conf_data2 |= BIT(4); +		} else if (*property_u32 == 16000) { +			/* 16 ms */ +			hub->conf_data2 |= BIT(5); +			hub->conf_data2 |= BIT(4); +		} else { +			/* 8 ms (DEFAULT) */ +			hub->conf_data2 |= BIT(5); +			hub->conf_data2 &= ~BIT(4); +		}  	}  	if (of_get_property(np, "compound-device", NULL)) @@ -432,30 +438,9 @@ static int usb251xb_get_ofdata(struct usb251xb *hub,  		}  	} -	hub->max_power_sp = USB251XB_DEF_MAX_POWER_SELF; -	if (!of_property_read_u32(np, "max-sp-power", property_u32)) -		hub->max_power_sp = min_t(u8, be32_to_cpu(*property_u32) / 2, -					  250); - -	hub->max_power_bp = USB251XB_DEF_MAX_POWER_BUS; -	if (!of_property_read_u32(np, "max-bp-power", property_u32)) -		hub->max_power_bp = min_t(u8, be32_to_cpu(*property_u32) / 2, -					  250); - -	hub->max_current_sp = USB251XB_DEF_MAX_CURRENT_SELF; -	if (!of_property_read_u32(np, "max-sp-current", property_u32)) -		hub->max_current_sp = min_t(u8, be32_to_cpu(*property_u32) / 2, -					    250); - -	hub->max_current_bp = USB251XB_DEF_MAX_CURRENT_BUS; -	if (!of_property_read_u32(np, "max-bp-current", property_u32)) -		hub->max_current_bp = min_t(u8, be32_to_cpu(*property_u32) / 2, -					    250); -  	hub->power_on_time = USB251XB_DEF_POWER_ON_TIME; -	if (!of_property_read_u32(np, "power-on-time", property_u32)) -		hub->power_on_time = min_t(u8, be32_to_cpu(*property_u32) / 2, -					   255); +	if (!of_property_read_u32(np, "power-on-time-ms", property_u32)) +		hub->power_on_time = min_t(u8, *property_u32 / 2, 255);  	if (of_property_read_u16_array(np, "language-id", &hub->lang_id, 1))  		hub->lang_id = USB251XB_DEF_LANGUAGE_ID; @@ -492,6 +477,10 @@ static int usb251xb_get_ofdata(struct usb251xb *hub,  	/* The following parameters are currently not exposed to devicetree, but  	 * may be as soon as needed.  	 */ +	hub->max_power_sp = USB251XB_DEF_MAX_POWER_SELF; +	hub->max_power_bp = USB251XB_DEF_MAX_POWER_BUS; +	hub->max_current_sp = USB251XB_DEF_MAX_CURRENT_SELF; +	hub->max_current_bp = USB251XB_DEF_MAX_CURRENT_BUS;  	hub->bat_charge_en = USB251XB_DEF_BATTERY_CHARGING_ENABLE;  	hub->boost_up = USB251XB_DEF_BOOST_UP;  	hub->boost_x = USB251XB_DEF_BOOST_X; diff --git a/drivers/usb/misc/uss720.c b/drivers/usb/misc/uss720.c index e45a3a680db8..07014cad6dbe 100644 --- a/drivers/usb/misc/uss720.c +++ b/drivers/usb/misc/uss720.c @@ -709,6 +709,11 @@ static int uss720_probe(struct usb_interface *intf,  	interface = intf->cur_altsetting; +	if (interface->desc.bNumEndpoints < 3) { +		usb_put_dev(usbdev); +		return -ENODEV; +	} +  	/*  	 * Allocate parport interface   	 */  | 
