tree: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master head: 52828ea60dfdd05246fc92dedfad1c8bbfa2dd66 commit: 52828ea60dfdd05246fc92dedfad1c8bbfa2dd66 [74/74] Bluetooth: btmtk: Fix btmtk.c undefined reference build error config: i386-buildonly-randconfig-005-20240720 (https://download.01.org/0day-ci/archive/20240720/202407200904.hRqT3JKD-lkp@xxxxxxxxx/config) compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240720/202407200904.hRqT3JKD-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202407200904.hRqT3JKD-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): >> drivers/bluetooth/btusb.c:2705:8: error: call to undeclared function 'btmtk_usb_subsys_reset'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 2705 | err = btmtk_usb_subsys_reset(hdev, btmtk_data->dev_id); | ^ >> drivers/bluetooth/btusb.c:2720:9: error: call to undeclared function 'alloc_mtk_intr_urb'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 2720 | urb = alloc_mtk_intr_urb(hdev, skb, btusb_tx_complete); | ^ >> drivers/bluetooth/btusb.c:2720:7: error: incompatible integer to pointer conversion assigning to 'struct urb *' from 'int' [-Wint-conversion] 2720 | urb = alloc_mtk_intr_urb(hdev, skb, btusb_tx_complete); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> drivers/bluetooth/btusb.c:2749:9: error: call to undeclared function 'btmtk_usb_setup'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 2749 | return btmtk_usb_setup(hdev); | ^ >> drivers/bluetooth/btusb.c:2760:9: error: call to undeclared function 'btmtk_usb_shutdown'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 2760 | return btmtk_usb_shutdown(hdev); | ^ drivers/bluetooth/btusb.c:2760:9: note: did you mean 'btusb_mtk_shutdown'? drivers/bluetooth/btusb.c:2752:12: note: 'btusb_mtk_shutdown' declared here 2752 | static int btusb_mtk_shutdown(struct hci_dev *hdev) | ^ 2753 | { 2754 | struct btusb_data *data = hci_get_drvdata(hdev); 2755 | struct btmtk_data *btmtk_data = hci_get_priv(hdev); 2756 | 2757 | if (test_bit(BTMTK_ISOPKT_RUNNING, &btmtk_data->flags)) 2758 | btusb_mtk_release_iso_intf(data); 2759 | 2760 | return btmtk_usb_shutdown(hdev); | ~~~~~~~~~~~~~~~~~~ | btusb_mtk_shutdown >> drivers/bluetooth/btusb.c:3869:20: error: use of undeclared identifier 'btmtk_usb_recv_acl'; did you mean 'btusb_recv_acl'? 3869 | data->recv_acl = btmtk_usb_recv_acl; | ^~~~~~~~~~~~~~~~~~ | btusb_recv_acl drivers/bluetooth/btusb.c:1179:12: note: 'btusb_recv_acl' declared here 1179 | static int btusb_recv_acl(struct btusb_data *data, struct sk_buff *skb) | ^ >> drivers/bluetooth/btusb.c:3870:19: error: use of undeclared identifier 'btmtk_usb_suspend' 3870 | data->suspend = btmtk_usb_suspend; | ^ >> drivers/bluetooth/btusb.c:3871:18: error: use of undeclared identifier 'btmtk_usb_resume' 3871 | data->resume = btmtk_usb_resume; | ^ 8 errors generated. vim +/btmtk_usb_subsys_reset +2705 drivers/bluetooth/btusb.c ceac1cb0259de682 Chris Lu 2024-07-04 2682 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2683 static int btusb_mtk_reset(struct hci_dev *hdev, void *rst_data) 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2684 { 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2685 struct btusb_data *data = hci_get_drvdata(hdev); d3e6236053958a8f Chris Lu 2024-07-04 2686 struct btmtk_data *btmtk_data = hci_get_priv(hdev); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2687 int err; 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2688 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2689 /* It's MediaTek specific bluetooth reset mechanism via USB */ 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2690 if (test_and_set_bit(BTMTK_HW_RESET_ACTIVE, &btmtk_data->flags)) { 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2691 bt_dev_err(hdev, "last reset failed? Not resetting again"); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2692 return -EBUSY; 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2693 } 25b6d7593a3af75a Jing Cai 2023-06-29 2694 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2695 err = usb_autopm_get_interface(data->intf); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2696 if (err < 0) 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2697 return err; 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2698 ceac1cb0259de682 Chris Lu 2024-07-04 2699 if (test_bit(BTMTK_ISOPKT_RUNNING, &btmtk_data->flags)) ceac1cb0259de682 Chris Lu 2024-07-04 2700 btusb_mtk_release_iso_intf(data); ceac1cb0259de682 Chris Lu 2024-07-04 2701 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2702 btusb_stop_traffic(data); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2703 usb_kill_anchored_urbs(&data->tx_anchor); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2704 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 @2705 err = btmtk_usb_subsys_reset(hdev, btmtk_data->dev_id); 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2706 4c0c28f2bbec0c51 Hao Qin 2024-05-15 2707 usb_queue_reset_device(data->intf); 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2708 clear_bit(BTMTK_HW_RESET_ACTIVE, &btmtk_data->flags); 25b6d7593a3af75a Jing Cai 2023-06-29 2709 25b6d7593a3af75a Jing Cai 2023-06-29 2710 return err; 25b6d7593a3af75a Jing Cai 2023-06-29 2711 } 25b6d7593a3af75a Jing Cai 2023-06-29 2712 ceac1cb0259de682 Chris Lu 2024-07-04 2713 static int btusb_send_frame_mtk(struct hci_dev *hdev, struct sk_buff *skb) ceac1cb0259de682 Chris Lu 2024-07-04 2714 { ceac1cb0259de682 Chris Lu 2024-07-04 2715 struct urb *urb; ceac1cb0259de682 Chris Lu 2024-07-04 2716 ceac1cb0259de682 Chris Lu 2024-07-04 2717 BT_DBG("%s", hdev->name); ceac1cb0259de682 Chris Lu 2024-07-04 2718 ceac1cb0259de682 Chris Lu 2024-07-04 2719 if (hci_skb_pkt_type(skb) == HCI_ISODATA_PKT) { ceac1cb0259de682 Chris Lu 2024-07-04 @2720 urb = alloc_mtk_intr_urb(hdev, skb, btusb_tx_complete); ceac1cb0259de682 Chris Lu 2024-07-04 2721 if (IS_ERR(urb)) ceac1cb0259de682 Chris Lu 2024-07-04 2722 return PTR_ERR(urb); ceac1cb0259de682 Chris Lu 2024-07-04 2723 ceac1cb0259de682 Chris Lu 2024-07-04 2724 return submit_or_queue_tx_urb(hdev, urb); ceac1cb0259de682 Chris Lu 2024-07-04 2725 } else { ceac1cb0259de682 Chris Lu 2024-07-04 2726 return btusb_send_frame(hdev, skb); ceac1cb0259de682 Chris Lu 2024-07-04 2727 } ceac1cb0259de682 Chris Lu 2024-07-04 2728 } ceac1cb0259de682 Chris Lu 2024-07-04 2729 a1c49c434e15050b Sean Wang 2019-06-02 2730 static int btusb_mtk_setup(struct hci_dev *hdev) a1c49c434e15050b Sean Wang 2019-06-02 2731 { a1c49c434e15050b Sean Wang 2019-06-02 2732 struct btusb_data *data = hci_get_drvdata(hdev); 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2733 struct btmtk_data *btmtk_data = hci_get_priv(hdev); 0b70151328781a89 Jing Cai 2023-06-29 2734 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2735 /* MediaTek WMT vendor cmd requiring below USB resources to 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2736 * complete the handshake. ccfc8948d7e4d93c Hao Qin 2024-05-15 2737 */ 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2738 btmtk_data->drv_name = btusb_driver.name; 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2739 btmtk_data->intf = data->intf; 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2740 btmtk_data->udev = data->udev; 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2741 btmtk_data->ctrl_anchor = &data->ctrl_anchor; 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 2742 btmtk_data->reset_sync = btusb_mtk_reset; ccfc8948d7e4d93c Hao Qin 2024-05-15 2743 ceac1cb0259de682 Chris Lu 2024-07-04 2744 /* Claim ISO data interface and endpoint */ ceac1cb0259de682 Chris Lu 2024-07-04 2745 btmtk_data->isopkt_intf = usb_ifnum_to_if(data->udev, MTK_ISO_IFNUM); ceac1cb0259de682 Chris Lu 2024-07-04 2746 if (btmtk_data->isopkt_intf) ceac1cb0259de682 Chris Lu 2024-07-04 2747 btusb_mtk_claim_iso_intf(data); ceac1cb0259de682 Chris Lu 2024-07-04 2748 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 @2749 return btmtk_usb_setup(hdev); a1c49c434e15050b Sean Wang 2019-06-02 2750 } a1c49c434e15050b Sean Wang 2019-06-02 2751 a1c49c434e15050b Sean Wang 2019-06-02 2752 static int btusb_mtk_shutdown(struct hci_dev *hdev) a1c49c434e15050b Sean Wang 2019-06-02 2753 { ceac1cb0259de682 Chris Lu 2024-07-04 2754 struct btusb_data *data = hci_get_drvdata(hdev); ceac1cb0259de682 Chris Lu 2024-07-04 2755 struct btmtk_data *btmtk_data = hci_get_priv(hdev); ceac1cb0259de682 Chris Lu 2024-07-04 2756 ceac1cb0259de682 Chris Lu 2024-07-04 2757 if (test_bit(BTMTK_ISOPKT_RUNNING, &btmtk_data->flags)) ceac1cb0259de682 Chris Lu 2024-07-04 2758 btusb_mtk_release_iso_intf(data); ceac1cb0259de682 Chris Lu 2024-07-04 2759 5c5e8c52e3cafaa6 Chris Lu 2024-07-04 @2760 return btmtk_usb_shutdown(hdev); a1c49c434e15050b Sean Wang 2019-06-02 2761 } a1c49c434e15050b Sean Wang 2019-06-02 2762 :::::: The code at line 2705 was first introduced by commit :::::: 5c5e8c52e3cafaa6f71efd44a42a674271e5b3ad Bluetooth: btmtk: move btusb_mtk_[setup, shutdown] to btmtk.c :::::: TO: Chris Lu <chris.lu@xxxxxxxxxxxx> :::::: CC: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
