[PATCH v5 04/10] usb: common: add define of usb_speed_string()

Roger Quadros rogerq at ti.com
Mon Aug 17 09:54:33 CEST 2020



On 17/08/2020 10:51, Roger Quadros wrote:
> Hi,
> 
> On 17/08/2020 10:40, Chunfeng Yun wrote:
>> There is only declear of usb_speed_string(), but not define it.
> 
> s/declear/declaration
> 
> s/not define/no definition of/
> 
>>
>> Signed-off-by: Chunfeng Yun <chunfeng.yun at mediatek.com>
>> ---
>> v5: no changes
>>
>> v4: new patch
>> ---
>>   drivers/usb/common/common.c | 7 +++++++
>>   include/linux/usb/ch9.h     | 4 ----
>>   2 files changed, 7 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/usb/common/common.c b/drivers/usb/common/common.c
>> index 76f5a9c..5e5c3c3 100644
>> --- a/drivers/usb/common/common.c
>> +++ b/drivers/usb/common/common.c
>> @@ -49,6 +49,13 @@ static const char *const speed_names[] = {
>>       [USB_SPEED_SUPER_PLUS] = "super-speed-plus",
>>   };
>> +const char *usb_speed_string(enum usb_device_speed speed)
>> +{
>> +    if (speed < 0 || speed >= ARRAY_SIZE(speed_names))
>> +        speed = USB_SPEED_UNKNOWN;
>> +    return speed_names[speed];
>> +}
>> +
> 
> But I see it already defined here
> 
> https://elixir.bootlin.com/linux/latest/source/drivers/usb/common/common.c#L72

Sorry, I didn't realize this is for u-boot :)

cheers,
-roger

> 
>>   enum usb_device_speed usb_get_maximum_speed(ofnode node)
>>   {
>>       const char *max_speed;
>> diff --git a/include/linux/usb/ch9.h b/include/linux/usb/ch9.h
>> index 7d225ee..a8fa5d7 100644
>> --- a/include/linux/usb/ch9.h
>> +++ b/include/linux/usb/ch9.h
>> @@ -959,8 +959,6 @@ enum usb_device_speed {
>>       USB_SPEED_SUPER_PLUS,            /* usb 3.1 */
>>   };
>> -#ifdef __KERNEL__
>> -
> 
> Why do you have to remove this?
>>   /**
>>    * usb_speed_string() - Returns human readable-name of the speed.
>>    * @speed: The speed to return human-readable name for.  If it's not
>> @@ -969,8 +967,6 @@ enum usb_device_speed {
>>    */
>>   extern const char *usb_speed_string(enum usb_device_speed speed);
>> -#endif
>> -
>>   enum usb_device_state {
>>       /* NOTATTACHED isn't in the USB spec, and this state acts
>>        * the same as ATTACHED ... but it's clearer this way.
>>
> 
> cheers,
> -roger

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki


More information about the U-Boot mailing list