Re: [PATCH v3 2/2] Bluetooth: Add support for wcn3990 soc.

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

 



Hi Balakrishna,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on bluetooth-next/master]
[also build test ERROR on next-20180424]
[cannot apply to v4.17-rc2]
[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/Balakrishna-Godavarthi/Enable-Bluetooth-functionality-for-WCN3990/20180426-103256
base:   https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master
config: alpha-allyesconfig (attached as .config)
compiler: alpha-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=alpha 

All errors (new ones prefixed by >>):

   In file included from drivers//bluetooth/hci_qca.c:40:0:
>> include/asm-generic/delay.h:12:13: error: conflicting types for '__delay'
    extern void __delay(unsigned long loops);
                ^~~~~~~
   In file included from include/linux/delay.h:26:0,
                    from include/linux/serdev.h:19,
                    from drivers//bluetooth/hci_qca.c:37:
   arch/alpha/include/asm/delay.h:5:13: note: previous declaration of '__delay' was here
    extern void __delay(int loops);
                ^~~~~~~
   In file included from drivers//bluetooth/hci_qca.c:40:0:
   include/asm-generic/delay.h:33:0: warning: "ndelay" redefined
    #define ndelay(n)       \
    
   In file included from include/linux/delay.h:26:0,
                    from include/linux/serdev.h:19,
                    from drivers//bluetooth/hci_qca.c:37:
   arch/alpha/include/asm/delay.h:9:0: note: this is the location of the previous definition
    #define ndelay ndelay
    
   In file included from drivers//bluetooth/hci_qca.c:38:0:
   drivers//bluetooth/hci_qca.c: In function 'qca_serdev_open':
   include/net/bluetooth/bluetooth.h:146:9: warning: format '%s' expects a matching 'char *' argument [-Wformat=]
     BT_ERR("%s: " fmt, (hdev)->name, ##__VA_ARGS__)
            ^
   include/net/bluetooth/bluetooth.h:136:33: note: in definition of macro 'BT_ERR'
    #define BT_ERR(fmt, ...) bt_err(fmt "\n", ##__VA_ARGS__)
                                    ^~~
   drivers//bluetooth/hci_qca.c:1041:3: note: in expansion of macro 'bt_dev_err'
      bt_dev_err(hu->hdev, "%s:open operation not supported");
      ^~~~~~~~~~
   drivers//bluetooth/hci_qca.c:1041:26: note: format string is defined here
      bt_dev_err(hu->hdev, "%s:open operation not supported");
                            ~^
   drivers//bluetooth/hci_qca.c: In function 'qca_serdev_probe':
   drivers//bluetooth/hci_qca.c:1334:7: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     data = of_device_get_match_data(&serdev->dev);
          ^
   drivers//bluetooth/hci_qca.c: In function 'btqca_power_setup':
   drivers//bluetooth/hci_qca.c:1288:4: warning: ignoring return value of 'regulator_enable', declared with attribute warn_unused_result [-Wunused-result]
       regulator_enable(qca->vreg_bulk[i].consumer);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
   In file included from drivers/bluetooth/hci_qca.c:40:0:
>> include/asm-generic/delay.h:12:13: error: conflicting types for '__delay'
    extern void __delay(unsigned long loops);
                ^~~~~~~
   In file included from include/linux/delay.h:26:0,
                    from include/linux/serdev.h:19,
                    from drivers/bluetooth/hci_qca.c:37:
   arch/alpha/include/asm/delay.h:5:13: note: previous declaration of '__delay' was here
    extern void __delay(int loops);
                ^~~~~~~
   In file included from drivers/bluetooth/hci_qca.c:40:0:
   include/asm-generic/delay.h:33:0: warning: "ndelay" redefined
    #define ndelay(n)       \
    
   In file included from include/linux/delay.h:26:0,
                    from include/linux/serdev.h:19,
                    from drivers/bluetooth/hci_qca.c:37:
   arch/alpha/include/asm/delay.h:9:0: note: this is the location of the previous definition
    #define ndelay ndelay
    
   In file included from drivers/bluetooth/hci_qca.c:38:0:
   drivers/bluetooth/hci_qca.c: In function 'qca_serdev_open':
   include/net/bluetooth/bluetooth.h:146:9: warning: format '%s' expects a matching 'char *' argument [-Wformat=]
     BT_ERR("%s: " fmt, (hdev)->name, ##__VA_ARGS__)
            ^
   include/net/bluetooth/bluetooth.h:136:33: note: in definition of macro 'BT_ERR'
    #define BT_ERR(fmt, ...) bt_err(fmt "\n", ##__VA_ARGS__)
                                    ^~~
   drivers/bluetooth/hci_qca.c:1041:3: note: in expansion of macro 'bt_dev_err'
      bt_dev_err(hu->hdev, "%s:open operation not supported");
      ^~~~~~~~~~
   drivers/bluetooth/hci_qca.c:1041:26: note: format string is defined here
      bt_dev_err(hu->hdev, "%s:open operation not supported");
                            ~^
   drivers/bluetooth/hci_qca.c: In function 'qca_serdev_probe':
   drivers/bluetooth/hci_qca.c:1334:7: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     data = of_device_get_match_data(&serdev->dev);
          ^
   drivers/bluetooth/hci_qca.c: In function 'btqca_power_setup':
   drivers/bluetooth/hci_qca.c:1288:4: warning: ignoring return value of 'regulator_enable', declared with attribute warn_unused_result [-Wunused-result]
       regulator_enable(qca->vreg_bulk[i].consumer);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vim +/__delay +12 include/asm-generic/delay.h

30ab2b03 Jonas Bonn    2011-07-02   8  
aafe4dbe Arnd Bergmann 2009-05-13   9  extern void __udelay(unsigned long usecs);
30ab2b03 Jonas Bonn    2011-07-02  10  extern void __ndelay(unsigned long nsecs);
30ab2b03 Jonas Bonn    2011-07-02  11  extern void __const_udelay(unsigned long xloops);
aafe4dbe Arnd Bergmann 2009-05-13 @12  extern void __delay(unsigned long loops);
aafe4dbe Arnd Bergmann 2009-05-13  13  

:::::: The code at line 12 was first introduced by commit
:::::: aafe4dbed0bf6cbdb2e9f03e1d42f8a540d8541d asm-generic: add generic versions of common headers

:::::: TO: Arnd Bergmann <arnd@xxxxxxxx>
:::::: CC: Arnd Bergmann <arnd@klappe2.(none)>

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux