From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: Thu, 4 Jul 2019 11:01:48 +0000 (+0200)
Subject: Revert "usb:gadget Simplify usb_decode_get_set_descriptor function."
X-Git-Tag: v5.15~5975^2~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=40abbef1c9af849fcdd23a7061b3f9a29067067b;p=platform%2Fkernel%2Flinux-starfive.git

Revert "usb:gadget Simplify usb_decode_get_set_descriptor function."

This reverts commit c2af6b07803ebd099950cd608f404a7bff9037b2.

It's broken.

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Felipe Balbi <balbi@kernel.org>
Cc: Pawel Laszczak <pawell@cadence.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/usb/gadget/debug.c b/drivers/usb/gadget/debug.c
index 92a986a..60a9f70a 100644
--- a/drivers/usb/gadget/debug.c
+++ b/drivers/usb/gadget/debug.c
@@ -105,65 +105,62 @@ static void usb_decode_get_set_descriptor(__u8 bRequestType, __u8 bRequest,
 					  __u16 wValue, __u16 wIndex,
 					  __u16 wLength, char *str, size_t size)
 {
-	char *s;
-
-	switch (wValue >> 8) {
-	case USB_DT_DEVICE:
-		s = "Device";
-		break;
-	case USB_DT_CONFIG:
-		s = "Configuration";
-		break;
-	case USB_DT_STRING:
-		s = "String";
-		break;
-	case USB_DT_INTERFACE:
-		s = "Interface";
-		break;
-	case USB_DT_ENDPOINT:
-		s = "Endpoint";
-		break;
-	case USB_DT_DEVICE_QUALIFIER:
-		s = "Device Qualifier";
-		break;
-	case USB_DT_OTHER_SPEED_CONFIG:
-		s = "Other Speed Config";
-		break;
-	case USB_DT_INTERFACE_POWER:
-		s = "Interface Power";
-		break;
-	case USB_DT_OTG:
-		s = "OTG";
-		break;
-	case USB_DT_DEBUG:
-		s = "Debug";
-		break;
-	case USB_DT_INTERFACE_ASSOCIATION:
-		s = "Interface Association";
-		break;
-	case USB_DT_BOS:
-		s = "BOS";
-		break;
-	case USB_DT_DEVICE_CAPABILITY:
-		s = "Device Capability";
-		break;
-	case USB_DT_PIPE_USAGE:
-		s = "Pipe Usage";
-		break;
-	case USB_DT_SS_ENDPOINT_COMP:
-		s = "SS Endpoint Companion";
-		break;
-	case USB_DT_SSP_ISOC_ENDPOINT_COMP:
-		s = "SSP Isochronous Endpoint Companion";
-		break;
-	default:
-		s = "UNKNOWN";
-		break;
-	}
-
 	snprintf(str, size, "%s %s Descriptor(Index = %d, Length = %d)",
-		bRequest == USB_REQ_GET_DESCRIPTOR ? "Get" : "Set",
-		s, wValue & 0xff, wLength);
+		 bRequest == USB_REQ_GET_DESCRIPTOR ? "Get" : "Set",
+		 ({ char *s;
+			switch (wValue >> 8) {
+			case USB_DT_DEVICE:
+				s = "Device";
+				break;
+			case USB_DT_CONFIG:
+				s = "Configuration";
+				break;
+			case USB_DT_STRING:
+				s = "String";
+				break;
+			case USB_DT_INTERFACE:
+				s = "Interface";
+				break;
+			case USB_DT_ENDPOINT:
+				s = "Endpoint";
+				break;
+			case USB_DT_DEVICE_QUALIFIER:
+				s = "Device Qualifier";
+				break;
+			case USB_DT_OTHER_SPEED_CONFIG:
+				s = "Other Speed Config";
+				break;
+			case USB_DT_INTERFACE_POWER:
+				s = "Interface Power";
+				break;
+			case USB_DT_OTG:
+				s = "OTG";
+				break;
+			case USB_DT_DEBUG:
+				s = "Debug";
+				break;
+			case USB_DT_INTERFACE_ASSOCIATION:
+				s = "Interface Association";
+				break;
+			case USB_DT_BOS:
+				s = "BOS";
+				break;
+			case USB_DT_DEVICE_CAPABILITY:
+				s = "Device Capability";
+				break;
+			case USB_DT_PIPE_USAGE:
+				s = "Pipe Usage";
+				break;
+			case USB_DT_SS_ENDPOINT_COMP:
+				s = "SS Endpoint Companion";
+				break;
+			case USB_DT_SSP_ISOC_ENDPOINT_COMP:
+				s = "SSP Isochronous Endpoint Companion";
+				break;
+			default:
+				s = "UNKNOWN";
+				break;
+			} s; }), wValue & 0xff, wLength);
 }
 
 static void usb_decode_get_configuration(__u16 wLength, char *str, size_t size)