[PATCH] USB: Changes to core/config.c (1 of 9)

This first patch makes only trivial changes.  Excess whitespace at the end
of lines is removed and return codes are altered from -1 to symbolic error
values.  The only significant difference is that in one spot a valid
return is changed to an error return, when a descriptor does not have the
correct type.

 - Remove excess whitespace at the ends of lines.
 - Change return codes to symbolic values.
 - Return an error when an invalid endpoint descriptor is found.

drivers/usb/core/config.c

@@ -25,13 +25,13 @@ static int usb_parse_endpoint(struct usb_host_endpoint *endpoint, unsigned char
 	/*  check JIC */
 	if (header->bLength > size) {
 		err("ran out of descriptors parsing");
-		return -1;
+		return -EINVAL;
 	}
 
 	if (header->bDescriptorType != USB_DT_ENDPOINT) {
 		warn("unexpected descriptor 0x%X, expecting endpoint, 0x%X",
 			header->bDescriptorType, USB_DT_ENDPOINT);
-		return parsed;
+		return -EINVAL;
 	}
 
 	if (header->bLength == USB_DT_ENDPOINT_AUDIO_SIZE)
@@ -54,7 +54,7 @@ static int usb_parse_endpoint(struct usb_host_endpoint *endpoint, unsigned char
 
 		if (header->bLength < 2) {
 			err("invalid descriptor length of %d", header->bLength);
-			return -1;
+			return -EINVAL;
 		}
 
 		/* If we find another "proper" descriptor then we're done  */
@@ -145,7 +145,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 
 	if (!interface->altsetting) {
 		err("couldn't kmalloc interface->altsetting");
-		return -1;
+		return -ENOMEM;
 	}
 
 	while (size > 0) {
@@ -160,13 +160,13 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 			if (interface->max_altsetting > USB_MAXALTSETTING) {
 				warn("too many alternate settings (incr %d max %d)\n",
 					USB_ALTSETTINGALLOC, USB_MAXALTSETTING);
-				return -1;
+				return -EINVAL;
 			}
 
 			ptr = kmalloc(sizeof(*ptr) * interface->max_altsetting, GFP_KERNEL);
 			if (ptr == NULL) {
 				err("couldn't kmalloc interface->altsetting");
-				return -1;
+				return -ENOMEM;
 			}
 			memcpy(ptr, interface->altsetting, sizeof(*interface->altsetting) * oldmas);
 			kfree(interface->altsetting);
@@ -195,7 +195,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 
 			if (header->bLength < 2) {
 				err("invalid descriptor length of %d", header->bLength);
-				return -1;
+				return -EINVAL;
 			}
 
 			/* If we find another "proper" descriptor then we're done  */
@@ -224,7 +224,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 			if (!ifp->extra) {
 				err("couldn't allocate memory for interface extra descriptors");
 				ifp->extralen = 0;
-				return -1;
+				return -ENOMEM;
 			}
 			memcpy(ifp->extra, begin, len);
 			ifp->extralen = len;
@@ -239,7 +239,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 
 		if (ifp->desc.bNumEndpoints > USB_MAXENDPOINTS) {
 			warn("too many endpoints");
-			return -1;
+			return -EINVAL;
 		}
 
 		ifp->endpoint = (struct usb_host_endpoint *)
@@ -247,7 +247,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 			sizeof(struct usb_host_endpoint), GFP_KERNEL);
 		if (!ifp->endpoint) {
 			err("out of memory");
-			return -1;	
+			return -ENOMEM;
 		}
 
 		memset(ifp->endpoint, 0, ifp->desc.bNumEndpoints *
@@ -258,7 +258,7 @@ static int usb_parse_interface(struct usb_interface *interface, unsigned char *b
 
 			if (header->bLength > size) {
 				err("ran out of descriptors parsing");
-				return -1;
+				return -EINVAL;
 			}
 
 			retval = usb_parse_endpoint(ifp->endpoint + i, buffer, size);
@@ -329,7 +329,7 @@ int usb_parse_configuration(struct usb_host_config *config, char *buffer)
 
 			if ((header->bLength > size) || (header->bLength < 2)) {
 				err("invalid descriptor length of %d", header->bLength);
-				return -1;
+				return -EINVAL;
 			}
 
 			/* If we find another "proper" descriptor then we're done  */
@@ -359,7 +359,7 @@ int usb_parse_configuration(struct usb_host_config *config, char *buffer)
 				if (!config->extra) {
 					err("couldn't allocate memory for config extra descriptors");
 					config->extralen = 0;
-					return -1;
+					return -ENOMEM;
 				}
 
 				memcpy(config->extra, begin, len);