Hi Luiz, I love your patch! Perhaps something to improve: [auto build test WARNING on bluetooth-next/master] [also build test WARNING on next-20220630] [cannot apply to bluetooth/master linus/master v5.19-rc4] [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/intel-lab-lkp/linux/commits/Luiz-Augusto-von-Dentz/Bluetooth-hci_core-Introduce-hci_recv_event_data/20220701-061626 base: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master config: sparc64-randconfig-r032-20220629 (https://download.01.org/0day-ci/archive/20220701/202207010714.VP7EmpRd-lkp@xxxxxxxxx/config) compiler: sparc64-linux-gcc (GCC) 11.3.0 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 # https://github.com/intel-lab-lkp/linux/commit/71de20339d16663211f831340ed16ed442bd3f35 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Luiz-Augusto-von-Dentz/Bluetooth-hci_core-Introduce-hci_recv_event_data/20220701-061626 git checkout 71de20339d16663211f831340ed16ed442bd3f35 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=sparc64 SHELL=/bin/bash net/bluetooth/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): >> net/bluetooth/hci_conn.c:1365:18: warning: no previous prototype for 'hci_add_bis' [-Wmissing-prototypes] 1365 | struct hci_conn *hci_add_bis(struct hci_dev *hdev, bdaddr_t *dst, | ^~~~~~~~~~~ >> net/bluetooth/hci_conn.c:1907:18: warning: no previous prototype for 'hci_bind_bis' [-Wmissing-prototypes] 1907 | struct hci_conn *hci_bind_bis(struct hci_conn *conn, struct bt_iso_qos *qos) | ^~~~~~~~~~~~ -- >> net/bluetooth/hci_sync.c:1066:5: warning: no previous prototype for 'hci_adv_bcast_annoucement' [-Wmissing-prototypes] 1066 | int hci_adv_bcast_annoucement(struct hci_dev *hdev, struct adv_info *adv) | ^~~~~~~~~~~~~~~~~~~~~~~~~ vim +/hci_add_bis +1365 net/bluetooth/hci_conn.c 1363 1364 /* This function requires the caller holds hdev->lock */ > 1365 struct hci_conn *hci_add_bis(struct hci_dev *hdev, bdaddr_t *dst, 1366 struct bt_iso_qos *qos) 1367 { 1368 struct hci_conn *conn; 1369 struct iso_list_data data; 1370 int err; 1371 1372 /* Let's make sure that le is enabled.*/ 1373 if (!hci_dev_test_flag(hdev, HCI_LE_ENABLED)) { 1374 if (lmp_le_capable(hdev)) 1375 return ERR_PTR(-ECONNREFUSED); 1376 return ERR_PTR(-EOPNOTSUPP); 1377 } 1378 1379 err = qos_set_big(hdev, qos); 1380 if (err) 1381 return ERR_PTR(err); 1382 1383 err = qos_set_bis(hdev, qos); 1384 if (err) 1385 return ERR_PTR(err); 1386 1387 data.big = qos->big; 1388 data.bis = qos->bis; 1389 data.count = 0; 1390 1391 /* Check if there is already a matching BIG/BIS */ 1392 hci_conn_hash_list_state(hdev, bis_list, ISO_LINK, BT_BOUND, &data); 1393 if (data.count) 1394 return ERR_PTR(-EADDRINUSE); 1395 1396 conn = hci_conn_hash_lookup_bis(hdev, dst, qos->big, qos->bis); 1397 if (conn) 1398 return ERR_PTR(-EADDRINUSE); 1399 1400 conn = hci_conn_add(hdev, ISO_LINK, dst, HCI_ROLE_MASTER); 1401 if (!conn) 1402 return ERR_PTR(-ENOMEM); 1403 1404 set_bit(HCI_CONN_PER_ADV, &conn->flags); 1405 conn->state = BT_CONNECT; 1406 1407 hci_conn_hold(conn); 1408 return conn; 1409 } 1410 -- 0-DAY CI Kernel Test Service https://01.org/lkp
