From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A25AFC433B4 for ; Thu, 20 May 2021 02:01:49 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 35574610A8 for ; Thu, 20 May 2021 02:01:49 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 35574610A8 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id AED086B006C; Wed, 19 May 2021 22:01:48 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A76596B006E; Wed, 19 May 2021 22:01:48 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8C7B36B0070; Wed, 19 May 2021 22:01:48 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0241.hostedemail.com [216.40.44.241]) by kanga.kvack.org (Postfix) with ESMTP id 52A746B006C for ; Wed, 19 May 2021 22:01:48 -0400 (EDT) Received: from smtpin36.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id EE784829C478 for ; Thu, 20 May 2021 02:01:47 +0000 (UTC) X-FDA: 78159958254.36.42F0A00 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by imf06.hostedemail.com (Postfix) with ESMTP id B1720C0042E7 for ; Thu, 20 May 2021 02:01:45 +0000 (UTC) IronPort-SDR: 9mU/7k7/4cynmQuktxuYKOktT9Ee7VepmzDU5zd6vqSNqvKZtnDkuxnja4MPnKL9aGFJLvs8Ll D8p8OqizAeSA== X-IronPort-AV: E=McAfee;i="6200,9189,9989"; a="222195197" X-IronPort-AV: E=Sophos;i="5.82,313,1613462400"; d="scan'208";a="222195197" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 May 2021 19:01:45 -0700 IronPort-SDR: YMt0+NI0wUzOr/msHvyjiQBfiD0adk53YjNpvvDNxxLCTqrGK18wQsDyHCQpFT1Gw1SQQRD6fy UqUcmc9QLcmQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.82,313,1613462400"; d="scan'208";a="440089562" Received: from pl-dbox.sh.intel.com (HELO pl-dbox) ([10.239.159.39]) by fmsmga008.fm.intel.com with ESMTP; 19 May 2021 19:01:43 -0700 Date: Thu, 20 May 2021 09:58:13 +0800 From: Philip Li To: Dan Carpenter Cc: kbuild@lists.01.org, Mark Chen , lkp@intel.com, kbuild-all@lists.01.org, Linux Memory Management List , Marcel Holtmann Subject: Re: [kbuild-all] [kbuild] [linux-next:master 7132/11526] drivers/bluetooth/btusb.c:3775 btusb_mtk_setup() error: uninitialized symbol 'fw_version'. Message-ID: <20210520015813.GA3327173@pl-dbox> References: <20210217074946.GI2222@kadam> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210217074946.GI2222@kadam> Authentication-Results: imf06.hostedemail.com; dkim=none; dmarc=fail reason="No valid SPF, No valid DKIM" header.from=intel.com (policy=none); spf=none (imf06.hostedemail.com: domain of philip.li@intel.com has no SPF policy when checking 192.55.52.88) smtp.mailfrom=philip.li@intel.com X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: B1720C0042E7 X-Stat-Signature: csfkwubbi9e77jqgngnamntwofde3afw X-HE-Tag: 1621476105-106848 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Wed, Feb 17, 2021 at 10:49:46AM +0300, Dan Carpenter wrote: > [ Why is linux-mm CC'd on these patches??? - dan ] sorry for late, Hi Dan, FYI, we have defined related rule like mail cc in repo definition of linux-next. This can be found at https://github.com/intel/lkp-tests/blob/master/repo/linux/linux-next, which could be a requirement in early days. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > head: 6553715b6db5ff5d4898895dad1b2926cfe406cf > commit: fc342c4dc408754f50f19dc832152fbb4b73f1e6 [7132/11526] Bluetooth: btusb: Add protocol support for MediaTek MT7921U USB devices > config: parisc-randconfig-m031-20210216 (attached as .config) > compiler: hppa-linux-gcc (GCC) 9.3.0 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > Reported-by: Dan Carpenter > > New smatch warnings: > drivers/bluetooth/btusb.c:3775 btusb_mtk_setup() error: uninitialized symbol 'fw_version'. > > Old smatch warnings: > drivers/bluetooth/btusb.c:2201 btusb_setup_intel() warn: inconsistent indenting > > vim +/fw_version +3775 drivers/bluetooth/btusb.c > > a1c49c434e15050 Sean Wang 2019-06-02 3729 static int btusb_mtk_setup(struct hci_dev *hdev) > a1c49c434e15050 Sean Wang 2019-06-02 3730 { > a1c49c434e15050 Sean Wang 2019-06-02 3731 struct btusb_data *data = hci_get_drvdata(hdev); > a1c49c434e15050 Sean Wang 2019-06-02 3732 struct btmtk_hci_wmt_params wmt_params; > a1c49c434e15050 Sean Wang 2019-06-02 3733 ktime_t calltime, delta, rettime; > a1c49c434e15050 Sean Wang 2019-06-02 3734 struct btmtk_tci_sleep tci_sleep; > a1c49c434e15050 Sean Wang 2019-06-02 3735 unsigned long long duration; > a1c49c434e15050 Sean Wang 2019-06-02 3736 struct sk_buff *skb; > a1c49c434e15050 Sean Wang 2019-06-02 3737 const char *fwname; > a1c49c434e15050 Sean Wang 2019-06-02 3738 int err, status; > a1c49c434e15050 Sean Wang 2019-06-02 3739 u32 dev_id; > fc342c4dc408754 Mark Chen 2021-02-02 3740 char fw_bin_name[64]; > fc342c4dc408754 Mark Chen 2021-02-02 3741 u32 fw_version; > a1c49c434e15050 Sean Wang 2019-06-02 3742 u8 param; > a1c49c434e15050 Sean Wang 2019-06-02 3743 > a1c49c434e15050 Sean Wang 2019-06-02 3744 calltime = ktime_get(); > a1c49c434e15050 Sean Wang 2019-06-02 3745 > 48c13301e6baba5 Mark Chen 2021-02-02 3746 err = btusb_mtk_id_get(data, 0x80000008, &dev_id); > a1c49c434e15050 Sean Wang 2019-06-02 3747 if (err < 0) { > a1c49c434e15050 Sean Wang 2019-06-02 3748 bt_dev_err(hdev, "Failed to get device id (%d)", err); > a1c49c434e15050 Sean Wang 2019-06-02 3749 return err; > a1c49c434e15050 Sean Wang 2019-06-02 3750 } > a1c49c434e15050 Sean Wang 2019-06-02 3751 > fc342c4dc408754 Mark Chen 2021-02-02 3752 if (!dev_id) { > fc342c4dc408754 Mark Chen 2021-02-02 3753 err = btusb_mtk_id_get(data, 0x70010200, &dev_id); > fc342c4dc408754 Mark Chen 2021-02-02 3754 if (err < 0) { > fc342c4dc408754 Mark Chen 2021-02-02 3755 bt_dev_err(hdev, "Failed to get device id (%d)", err); > fc342c4dc408754 Mark Chen 2021-02-02 3756 return err; > fc342c4dc408754 Mark Chen 2021-02-02 3757 } > fc342c4dc408754 Mark Chen 2021-02-02 3758 err = btusb_mtk_id_get(data, 0x80021004, &fw_version); > fc342c4dc408754 Mark Chen 2021-02-02 3759 if (err < 0) { > fc342c4dc408754 Mark Chen 2021-02-02 3760 bt_dev_err(hdev, "Failed to get fw version (%d)", err); > fc342c4dc408754 Mark Chen 2021-02-02 3761 return err; > fc342c4dc408754 Mark Chen 2021-02-02 3762 } > fc342c4dc408754 Mark Chen 2021-02-02 3763 } > > "fw_version" not initialized on else path. > > fc342c4dc408754 Mark Chen 2021-02-02 3764 > a1c49c434e15050 Sean Wang 2019-06-02 3765 switch (dev_id) { > 9ce67c3235be71e Sean Wang 2019-06-02 3766 case 0x7663: > 9ce67c3235be71e Sean Wang 2019-06-02 3767 fwname = FIRMWARE_MT7663; > 9ce67c3235be71e Sean Wang 2019-06-02 3768 break; > a1c49c434e15050 Sean Wang 2019-06-02 3769 case 0x7668: > a1c49c434e15050 Sean Wang 2019-06-02 3770 fwname = FIRMWARE_MT7668; > a1c49c434e15050 Sean Wang 2019-06-02 3771 break; > fc342c4dc408754 Mark Chen 2021-02-02 3772 case 0x7961: > fc342c4dc408754 Mark Chen 2021-02-02 3773 snprintf(fw_bin_name, sizeof(fw_bin_name), > fc342c4dc408754 Mark Chen 2021-02-02 3774 "mediatek/BT_RAM_CODE_MT%04x_1_%x_hdr.bin", > fc342c4dc408754 Mark Chen 2021-02-02 @3775 dev_id & 0xffff, (fw_version & 0xff) + 1); > ^^^^^^^^^^ > > fc342c4dc408754 Mark Chen 2021-02-02 3776 err = btusb_mtk_setup_firmware_79xx(hdev, fw_bin_name); > fc342c4dc408754 Mark Chen 2021-02-02 3777 > fc342c4dc408754 Mark Chen 2021-02-02 3778 /* Enable Bluetooth protocol */ > fc342c4dc408754 Mark Chen 2021-02-02 3779 param = 1; > fc342c4dc408754 Mark Chen 2021-02-02 3780 wmt_params.op = BTMTK_WMT_FUNC_CTRL; > fc342c4dc408754 Mark Chen 2021-02-02 3781 wmt_params.flag = 0; > fc342c4dc408754 Mark Chen 2021-02-02 3782 wmt_params.dlen = sizeof(param); > fc342c4dc408754 Mark Chen 2021-02-02 3783 wmt_params.data = ¶m; > fc342c4dc408754 Mark Chen 2021-02-02 3784 wmt_params.status = NULL; > fc342c4dc408754 Mark Chen 2021-02-02 3785 > fc342c4dc408754 Mark Chen 2021-02-02 3786 err = btusb_mtk_hci_wmt_sync(hdev, &wmt_params); > fc342c4dc408754 Mark Chen 2021-02-02 3787 if (err < 0) { > fc342c4dc408754 Mark Chen 2021-02-02 3788 bt_dev_err(hdev, "Failed to send wmt func ctrl (%d)", err); > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > _______________________________________________ > kbuild mailing list -- kbuild@lists.01.org > To unsubscribe send an email to kbuild-leave@lists.01.org > _______________________________________________ > kbuild-all mailing list -- kbuild-all@lists.01.org > To unsubscribe send an email to kbuild-all-leave@lists.01.org