From 51f60a89b75e5b71c747e15b8247abae798244ec Mon Sep 17 00:00:00 2001 From: Ismael Luceno Cortes Date: Mon, 1 Apr 2019 16:09:13 +0000 Subject: usb: Make portspeed return a read-only string Current code is plain wrong, and there's no need to have a mutable string, so fix function type and remove the intermediate variable. Signed-off-by: Ismael Luceno --- cmd/usb.c | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) (limited to 'cmd/usb.c') diff --git a/cmd/usb.c b/cmd/usb.c index 0ccb1b5148..dd9ac0bc97 100644 --- a/cmd/usb.c +++ b/cmd/usb.c @@ -316,26 +316,18 @@ static struct usb_device *usb_find_device(int devnum) return NULL; } -static inline char *portspeed(int speed) +static inline const char *portspeed(int speed) { - char *speed_str; - switch (speed) { case USB_SPEED_SUPER: - speed_str = "5 Gb/s"; - break; + return "5 Gb/s"; case USB_SPEED_HIGH: - speed_str = "480 Mb/s"; - break; + return "480 Mb/s"; case USB_SPEED_LOW: - speed_str = "1.5 Mb/s"; - break; + return "1.5 Mb/s"; default: - speed_str = "12 Mb/s"; - break; + return "12 Mb/s"; } - - return speed_str; } /* shows the device tree recursively */ -- cgit v1.2.3