* [PATCH] drivers: bluetooth: btintel.c: fixed format issue.
@ 2020-08-09 7:25 YourName
2020-08-09 11:35 ` kernel test robot
0 siblings, 1 reply; 2+ messages in thread
From: YourName @ 2020-08-09 7:25 UTC (permalink / raw)
To: marcel; +Cc: johan.hedberg, linux-bluetooth, linux-kernel, argoz1701
From: argoz1701 <argoz1701@gmail.com>
Fixed a coding style issue.
Signed-off-by: Daniel West <argoz1701@gmail.com>
---
drivers/bluetooth/btintel.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c
index 5fa5be3c5598..d0512506fa05 100644
--- a/drivers/bluetooth/btintel.c
+++ b/drivers/bluetooth/btintel.c
@@ -29,7 +29,7 @@ int btintel_check_bdaddr(struct hci_dev *hdev)
HCI_INIT_TIMEOUT);
if (IS_ERR(skb)) {
int err = PTR_ERR(skb);
- bt_dev_err(hdev, "Reading Intel device address failed (%d)",
+ bt_dev_err(hdev, "Reading Intel device address failed (%d)",
err);
return err;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drivers: bluetooth: btintel.c: fixed format issue.
2020-08-09 7:25 [PATCH] drivers: bluetooth: btintel.c: fixed format issue YourName
@ 2020-08-09 11:35 ` kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2020-08-09 11:35 UTC (permalink / raw)
To: YourName, marcel
Cc: kbuild-all, johan.hedberg, linux-bluetooth, linux-kernel, argoz1701
[-- Attachment #1: Type: text/plain, Size: 3950 bytes --]
Hi YourName,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on bluetooth-next/master]
[also build test WARNING on v5.8 next-20200807]
[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/YourName/drivers-bluetooth-btintel-c-fixed-format-issue/20200809-152614
base: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master
config: i386-randconfig-m021-20200809 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
smatch warnings:
drivers/bluetooth/btintel.c:32 btintel_check_bdaddr() warn: inconsistent indenting
vim +32 drivers/bluetooth/btintel.c
48f0ed1bb68589 Marcel Holtmann 2015-04-06 22
48f0ed1bb68589 Marcel Holtmann 2015-04-06 23 int btintel_check_bdaddr(struct hci_dev *hdev)
48f0ed1bb68589 Marcel Holtmann 2015-04-06 24 {
48f0ed1bb68589 Marcel Holtmann 2015-04-06 25 struct hci_rp_read_bd_addr *bda;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 26 struct sk_buff *skb;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 27
48f0ed1bb68589 Marcel Holtmann 2015-04-06 28 skb = __hci_cmd_sync(hdev, HCI_OP_READ_BD_ADDR, 0, NULL,
48f0ed1bb68589 Marcel Holtmann 2015-04-06 29 HCI_INIT_TIMEOUT);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 30 if (IS_ERR(skb)) {
48f0ed1bb68589 Marcel Holtmann 2015-04-06 31 int err = PTR_ERR(skb);
2064ee332e4c1b Marcel Holtmann 2017-10-30 @32 bt_dev_err(hdev, "Reading Intel device address failed (%d)",
2064ee332e4c1b Marcel Holtmann 2017-10-30 33 err);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 34 return err;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 35 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 36
48f0ed1bb68589 Marcel Holtmann 2015-04-06 37 if (skb->len != sizeof(*bda)) {
2064ee332e4c1b Marcel Holtmann 2017-10-30 38 bt_dev_err(hdev, "Intel device address length mismatch");
48f0ed1bb68589 Marcel Holtmann 2015-04-06 39 kfree_skb(skb);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 40 return -EIO;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 41 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 42
48f0ed1bb68589 Marcel Holtmann 2015-04-06 43 bda = (struct hci_rp_read_bd_addr *)skb->data;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 44
48f0ed1bb68589 Marcel Holtmann 2015-04-06 45 /* For some Intel based controllers, the default Bluetooth device
48f0ed1bb68589 Marcel Holtmann 2015-04-06 46 * address 00:03:19:9E:8B:00 can be found. These controllers are
48f0ed1bb68589 Marcel Holtmann 2015-04-06 47 * fully operational, but have the danger of duplicate addresses
48f0ed1bb68589 Marcel Holtmann 2015-04-06 48 * and that in turn can cause problems with Bluetooth operation.
48f0ed1bb68589 Marcel Holtmann 2015-04-06 49 */
48f0ed1bb68589 Marcel Holtmann 2015-04-06 50 if (!bacmp(&bda->bdaddr, BDADDR_INTEL)) {
2064ee332e4c1b Marcel Holtmann 2017-10-30 51 bt_dev_err(hdev, "Found Intel default device address (%pMR)",
2064ee332e4c1b Marcel Holtmann 2017-10-30 52 &bda->bdaddr);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 53 set_bit(HCI_QUIRK_INVALID_BDADDR, &hdev->quirks);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 54 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 55
48f0ed1bb68589 Marcel Holtmann 2015-04-06 56 kfree_skb(skb);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 57
48f0ed1bb68589 Marcel Holtmann 2015-04-06 58 return 0;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 59 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 60 EXPORT_SYMBOL_GPL(btintel_check_bdaddr);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 61
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 42074 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-08-09 11:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-09 7:25 [PATCH] drivers: bluetooth: btintel.c: fixed format issue YourName
2020-08-09 11:35 ` kernel test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).