Re: [RFC 5/5] Bluetooth: hci_sync: Make use of hci_cmd_sync_queue set 3

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3753 bytes --]

Hi Luiz,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on bluetooth/master]
[also build test WARNING on v5.13-rc3 next-20210527]
[cannot apply to bluetooth-next/master]
[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/Luiz-Augusto-von-Dentz/Bluetooth-Add-helper-for-serialized-HCI-command-execution/20210528-080351
base:   https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth.git master
config: arm-randconfig-r031-20210528 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 6505c630407c5feec818f0bb1c284f9eeebf2071)
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 arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # https://github.com/0day-ci/linux/commit/5322bf4209652456b7401c11d8be197636a78fca
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Luiz-Augusto-von-Dentz/Bluetooth-Add-helper-for-serialized-HCI-command-execution/20210528-080351
        git checkout 5322bf4209652456b7401c11d8be197636a78fca
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 

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

All warnings (new ones prefixed by >>):

>> net/bluetooth/hci_sync.c:958:5: warning: no previous prototype for function 'hci_scan_disable_sync' [-Wmissing-prototypes]
   int hci_scan_disable_sync(struct hci_dev *hdev, bool rpa_le_conn)
       ^
   net/bluetooth/hci_sync.c:958:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int hci_scan_disable_sync(struct hci_dev *hdev, bool rpa_le_conn)
   ^
   static 
>> net/bluetooth/hci_sync.c:1371:5: warning: no previous prototype for function 'hci_passive_scan_sync' [-Wmissing-prototypes]
   int hci_passive_scan_sync(struct hci_dev *hdev)
       ^
   net/bluetooth/hci_sync.c:1371:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int hci_passive_scan_sync(struct hci_dev *hdev)
   ^
   static 
   2 warnings generated.


vim +/hci_scan_disable_sync +958 net/bluetooth/hci_sync.c

   957	
 > 958	int hci_scan_disable_sync(struct hci_dev *hdev, bool rpa_le_conn)
   959	{
   960		int err;
   961	
   962		/* If controller is not scanning we are done. */
   963		if (!hci_dev_test_flag(hdev, HCI_LE_SCAN))
   964			return 0;
   965	
   966		if (hdev->scanning_paused) {
   967			bt_dev_dbg(hdev, "Scanning is paused for suspend");
   968			return 0;
   969		}
   970	
   971		if (hdev->suspended)
   972			set_bit(SUSPEND_SCAN_DISABLE, hdev->suspend_tasks);
   973	
   974		err = hci_le_set_scan_enable_sync(hdev, LE_SCAN_DISABLE, 0x00);
   975		if (err) {
   976			bt_dev_err(hdev, "Unable to disable scanning: %d", err);
   977			return err;
   978		}
   979	
   980		if (rpa_le_conn) {
   981			err = hci_le_set_addr_resolution_enable_sync(hdev, 0x00);
   982			if (err)
   983				bt_dev_err(hdev, "Unable to disable LL privacy: %d",
   984					   err);
   985		}
   986	
   987		return err;
   988	}
   989	

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 42557 bytes --]