Hi Pawel,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on usb/usb-testing]
[also build test WARNING on v4.20-rc7 next-20181221]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improve the system]

url:    
https://github.com/0day-ci/linux/commits/Pawel-Laszczak/Introduced-new-Cadence-USBSS-DRD-Driver/20181223-231813
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git 
usb-testing
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

>> drivers/usb/common/debug.c:253:25: warning: restricted __le16 degrades to 
>> integer
   drivers/usb/common/debug.c:254:25: warning: restricted __le16 degrades to 
integer
   drivers/usb/common/debug.c:255:25: warning: restricted __le16 degrades to 
integer
   drivers/usb/common/debug.c:256:25: warning: restricted __le16 degrades to 
integer
   drivers/usb/common/debug.c:257:25: warning: restricted __le16 degrades to 
integer
   drivers/usb/common/debug.c:258:25: warning: restricted __le16 degrades to 
integer

vim +253 drivers/usb/common/debug.c

cefb8b21 Pawel Laszczak 2018-12-23  205  
cefb8b21 Pawel Laszczak 2018-12-23  206  /**
cefb8b21 Pawel Laszczak 2018-12-23  207   * usb_decode_ctrl - returns a string 
representation of ctrl request
cefb8b21 Pawel Laszczak 2018-12-23  208   */
cefb8b21 Pawel Laszczak 2018-12-23  209  const char *usb_decode_ctrl(char *str, 
__u8 bRequestType, __u8 bRequest,
cefb8b21 Pawel Laszczak 2018-12-23  210                             __u16 
wValue,  __u16 wIndex, __u16 wLength)
cefb8b21 Pawel Laszczak 2018-12-23  211  {
cefb8b21 Pawel Laszczak 2018-12-23  212         switch (bRequest) {
cefb8b21 Pawel Laszczak 2018-12-23  213         case USB_REQ_GET_STATUS:
cefb8b21 Pawel Laszczak 2018-12-23  214                 
usb_decode_get_status(bRequestType, wIndex, wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  215                 break;
cefb8b21 Pawel Laszczak 2018-12-23  216         case USB_REQ_CLEAR_FEATURE:
cefb8b21 Pawel Laszczak 2018-12-23  217         case USB_REQ_SET_FEATURE:
cefb8b21 Pawel Laszczak 2018-12-23  218                 
usb_decode_set_clear_feature(bRequestType, bRequest, wValue,
cefb8b21 Pawel Laszczak 2018-12-23  219                                         
     wIndex, str);
cefb8b21 Pawel Laszczak 2018-12-23  220                 break;
cefb8b21 Pawel Laszczak 2018-12-23  221         case USB_REQ_SET_ADDRESS:
cefb8b21 Pawel Laszczak 2018-12-23  222                 
usb_decode_set_address(wValue, str);
cefb8b21 Pawel Laszczak 2018-12-23  223                 break;
cefb8b21 Pawel Laszczak 2018-12-23  224         case USB_REQ_GET_DESCRIPTOR:
cefb8b21 Pawel Laszczak 2018-12-23  225         case USB_REQ_SET_DESCRIPTOR:
cefb8b21 Pawel Laszczak 2018-12-23  226                 
usb_decode_get_set_descriptor(bRequestType, bRequest, wValue,
cefb8b21 Pawel Laszczak 2018-12-23  227                                         
      wIndex, wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  228                 break;
cefb8b21 Pawel Laszczak 2018-12-23  229         case USB_REQ_GET_CONFIGURATION:
cefb8b21 Pawel Laszczak 2018-12-23  230                 
usb_decode_get_configuration(wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  231                 break;
cefb8b21 Pawel Laszczak 2018-12-23  232         case USB_REQ_SET_CONFIGURATION:
cefb8b21 Pawel Laszczak 2018-12-23  233                 
usb_decode_set_configuration(wValue, str);
cefb8b21 Pawel Laszczak 2018-12-23  234                 break;
cefb8b21 Pawel Laszczak 2018-12-23  235         case USB_REQ_GET_INTERFACE:
cefb8b21 Pawel Laszczak 2018-12-23  236                 
usb_decode_get_intf(wIndex, wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  237                 break;
cefb8b21 Pawel Laszczak 2018-12-23  238         case USB_REQ_SET_INTERFACE:
cefb8b21 Pawel Laszczak 2018-12-23  239                 
usb_decode_set_intf(wValue, wIndex, str);
cefb8b21 Pawel Laszczak 2018-12-23  240                 break;
cefb8b21 Pawel Laszczak 2018-12-23  241         case USB_REQ_SYNCH_FRAME:
cefb8b21 Pawel Laszczak 2018-12-23  242                 
usb_decode_synch_frame(wIndex, wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  243                 break;
cefb8b21 Pawel Laszczak 2018-12-23  244         case USB_REQ_SET_SEL:
cefb8b21 Pawel Laszczak 2018-12-23  245                 
usb_decode_set_sel(wLength, str);
cefb8b21 Pawel Laszczak 2018-12-23  246                 break;
cefb8b21 Pawel Laszczak 2018-12-23  247         case USB_REQ_SET_ISOCH_DELAY:
cefb8b21 Pawel Laszczak 2018-12-23  248                 
usb_decode_set_isoch_delay(wValue, str);
cefb8b21 Pawel Laszczak 2018-12-23  249                 break;
cefb8b21 Pawel Laszczak 2018-12-23  250         default:
cefb8b21 Pawel Laszczak 2018-12-23  251                 sprintf(str, "%02x %02x 
%02x %02x %02x %02x %02x %02x",
cefb8b21 Pawel Laszczak 2018-12-23  252                         bRequestType, 
bRequest,
cefb8b21 Pawel Laszczak 2018-12-23 @253                         
cpu_to_le16(wValue) & 0xff,
cefb8b21 Pawel Laszczak 2018-12-23  254                         
cpu_to_le16(wValue) >> 8,
cefb8b21 Pawel Laszczak 2018-12-23  255                         
cpu_to_le16(wIndex) & 0xff,
cefb8b21 Pawel Laszczak 2018-12-23  256                         
cpu_to_le16(wIndex) >> 8,
cefb8b21 Pawel Laszczak 2018-12-23  257                         
cpu_to_le16(wLength) & 0xff,
cefb8b21 Pawel Laszczak 2018-12-23  258                         
cpu_to_le16(wLength) >> 8);
cefb8b21 Pawel Laszczak 2018-12-23  259         }
cefb8b21 Pawel Laszczak 2018-12-23  260  
cefb8b21 Pawel Laszczak 2018-12-23  261         return str;
cefb8b21 Pawel Laszczak 2018-12-23  262  }
cefb8b21 Pawel Laszczak 2018-12-23  263  EXPORT_SYMBOL_GPL(usb_decode_ctrl);
cefb8b21 Pawel Laszczak 2018-12-23  264  

:::::: The code at line 253 was first introduced by commit
:::::: cefb8b2176410ef0aacc22411a64f8a157612f14 usb:common Separated decoding 
functions from dwc3 driver.

:::::: TO: Pawel Laszczak <paw...@cadence.com>
:::::: CC: 0day robot <l...@intel.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

Reply via email to