Re: [PATCH v1 2/3] usb: early: xhci-dbc: Move cpu_to_le16_array() to core

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andy,

I love your patch! Yet something to improve:

[auto build test ERROR on usb/usb-testing]
[also build test ERROR on peter.chen-usb/ci-for-usb-next balbi-usb/testing/next linus/master v5.9-rc1 next-20200817]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Andy-Shevchenko/usb-early-xhci-dbc-use-readl_poll_timeout-to-simplify-code/20200818-004710
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: x86_64-randconfig-r014-20200817 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project de71b46a519db014ce906a39f8a0e1b235ef1568)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

   In file included from drivers/usb/early/xhci-dbc.c:12:
   include/linux/byteorder/generic.h:144:33: error: unknown type name '__le16'
   static inline void le16_add_cpu(__le16 *var, u16 val)
                                   ^
   include/linux/byteorder/generic.h:144:46: error: unknown type name 'u16'
   static inline void le16_add_cpu(__le16 *var, u16 val)
                                                ^
>> include/linux/byteorder/generic.h:146:9: error: implicit declaration of function '__cpu_to_le16' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le16(le16_to_cpu(*var) + val);
                  ^
   include/linux/byteorder/generic.h:90:21: note: expanded from macro 'cpu_to_le16'
   #define cpu_to_le16 __cpu_to_le16
                       ^
>> include/linux/byteorder/generic.h:146:21: error: implicit declaration of function '__le16_to_cpu' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le16(le16_to_cpu(*var) + val);
                              ^
   include/linux/byteorder/generic.h:91:21: note: expanded from macro 'le16_to_cpu'
   #define le16_to_cpu __le16_to_cpu
                       ^
   include/linux/byteorder/generic.h:149:33: error: unknown type name '__le32'
   static inline void le32_add_cpu(__le32 *var, u32 val)
                                   ^
   include/linux/byteorder/generic.h:149:46: error: unknown type name 'u32'
   static inline void le32_add_cpu(__le32 *var, u32 val)
                                                ^
>> include/linux/byteorder/generic.h:151:9: error: implicit declaration of function '__cpu_to_le32' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le32(le32_to_cpu(*var) + val);
                  ^
   include/linux/byteorder/generic.h:88:21: note: expanded from macro 'cpu_to_le32'
   #define cpu_to_le32 __cpu_to_le32
                       ^
>> include/linux/byteorder/generic.h:151:21: error: implicit declaration of function '__le32_to_cpu' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le32(le32_to_cpu(*var) + val);
                              ^
   include/linux/byteorder/generic.h:89:21: note: expanded from macro 'le32_to_cpu'
   #define le32_to_cpu __le32_to_cpu
                       ^
   include/linux/byteorder/generic.h:154:33: error: unknown type name '__le64'
   static inline void le64_add_cpu(__le64 *var, u64 val)
                                   ^
   include/linux/byteorder/generic.h:154:46: error: unknown type name 'u64'
   static inline void le64_add_cpu(__le64 *var, u64 val)
                                                ^
>> include/linux/byteorder/generic.h:156:9: error: implicit declaration of function '__cpu_to_le64' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le64(le64_to_cpu(*var) + val);
                  ^
   include/linux/byteorder/generic.h:86:21: note: expanded from macro 'cpu_to_le64'
   #define cpu_to_le64 __cpu_to_le64
                       ^
>> include/linux/byteorder/generic.h:156:21: error: implicit declaration of function '__le64_to_cpu' [-Werror,-Wimplicit-function-declaration]
           *var = cpu_to_le64(le64_to_cpu(*var) + val);
                              ^
   include/linux/byteorder/generic.h:87:21: note: expanded from macro 'le64_to_cpu'
   #define le64_to_cpu __le64_to_cpu
                       ^
   include/linux/byteorder/generic.h:159:38: error: unknown type name '__le16'
   static inline void cpu_to_le16_array(__le16 *dst, const u16 *src, size_t len)
                                        ^
   include/linux/byteorder/generic.h:159:57: error: unknown type name 'u16'
   static inline void cpu_to_le16_array(__le16 *dst, const u16 *src, size_t len)
                                                           ^
   include/linux/byteorder/generic.h:159:67: error: unknown type name 'size_t'
   static inline void cpu_to_le16_array(__le16 *dst, const u16 *src, size_t len)
                                                                     ^
   include/linux/byteorder/generic.h:164:12: error: implicit declaration of function '__cpu_to_le16' [-Werror,-Wimplicit-function-declaration]
                   dst[i] = cpu_to_le16(src[i]);
                            ^
   include/linux/byteorder/generic.h:90:21: note: expanded from macro 'cpu_to_le16'
   #define cpu_to_le16 __cpu_to_le16
                       ^
   include/linux/byteorder/generic.h:167:38: error: unknown type name 'u16'
   static inline void le16_to_cpu_array(u16 *dst, const __le16 *src, size_t len)
                                        ^
   include/linux/byteorder/generic.h:167:54: error: unknown type name '__le16'
   static inline void le16_to_cpu_array(u16 *dst, const __le16 *src, size_t len)
                                                        ^
   include/linux/byteorder/generic.h:167:67: error: unknown type name 'size_t'
   static inline void le16_to_cpu_array(u16 *dst, const __le16 *src, size_t len)
                                                                     ^
   fatal error: too many errors emitted, stopping now [-ferror-limit=]
   20 errors generated.

# https://github.com/0day-ci/linux/commit/379f669600c31fef2fa3b729ebb9ecf706d69621
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Andy-Shevchenko/usb-early-xhci-dbc-use-readl_poll_timeout-to-simplify-code/20200818-004710
git checkout 379f669600c31fef2fa3b729ebb9ecf706d69621
vim +/__cpu_to_le16 +146 include/linux/byteorder/generic.h

^1da177e4c3f415 Linus Torvalds 2005-04-16  143  
8b5f6883683c91a Marcin Slusarz 2008-02-08  144  static inline void le16_add_cpu(__le16 *var, u16 val)
8b5f6883683c91a Marcin Slusarz 2008-02-08  145  {
8b5f6883683c91a Marcin Slusarz 2008-02-08 @146  	*var = cpu_to_le16(le16_to_cpu(*var) + val);
8b5f6883683c91a Marcin Slusarz 2008-02-08  147  }
8b5f6883683c91a Marcin Slusarz 2008-02-08  148  
8b5f6883683c91a Marcin Slusarz 2008-02-08  149  static inline void le32_add_cpu(__le32 *var, u32 val)
8b5f6883683c91a Marcin Slusarz 2008-02-08  150  {
8b5f6883683c91a Marcin Slusarz 2008-02-08 @151  	*var = cpu_to_le32(le32_to_cpu(*var) + val);
8b5f6883683c91a Marcin Slusarz 2008-02-08  152  }
8b5f6883683c91a Marcin Slusarz 2008-02-08  153  
8b5f6883683c91a Marcin Slusarz 2008-02-08  154  static inline void le64_add_cpu(__le64 *var, u64 val)
8b5f6883683c91a Marcin Slusarz 2008-02-08  155  {
8b5f6883683c91a Marcin Slusarz 2008-02-08 @156  	*var = cpu_to_le64(le64_to_cpu(*var) + val);
8b5f6883683c91a Marcin Slusarz 2008-02-08  157  }
8b5f6883683c91a Marcin Slusarz 2008-02-08  158  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux