Skip to contentSkip to conversation

Pull request: bluetooth-next-2.6 2009-06-14

PATCH New Marcel Holtmann 2 participants 4 messages Jun 14, 2009 – Jun 15, 2009

Subsystems (4)
lore
Share & tools
lore

Marcel Holtmann

author maintainer
#0 lore raw

Hi Dave,

I have two more patches that should go into your networking merge. The first is fixing a build fallout from missing dependencies on the RFKILL subsystem. The second is fixing a build warning that is potentially dangerous with DTL-1 hardware when using unknown packet types.

Regards

Marcel

Please pull from

git://git.kernel.org/pub/scm/linux/kernel/git/holtmann/bluetooth-next-2.6.git master

This will update the following files:

 drivers/bluetooth/dtl1_cs.c |    2 ++
 net/bluetooth/Kconfig       |    3 ++-
 2 files changed, 4 insertions(+), 1 deletions(-)

through these ChangeSets:

Marcel Holtmann (2):
Bluetooth: Fix Kconfig issue with RFKILL integration
Bluetooth: Fix issue with uninitialized nsh.type in DTL-1 driver

author maintainer
#1 lore raw

PATCH 1/2 Bluetooth: Fix Kconfig issue with RFKILL integration

Since the re-write of the RFKILL subsystem it is no longer good to just select RFKILL, but it is important to add a proper depends on rule.

Based on a report by Alexander Beregalov <[email protected]>

---
 net/bluetooth/Kconfig |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
net/bluetooth/Kconfig +2 -1
index 7725da9..59fdb1d 100644
@@ -3,8 +3,9 @@
 #
 
 menuconfig BT
-	depends on NET && !S390
 	tristate "Bluetooth subsystem support"
+	depends on NET && !S390
+	depends on RFKILL || !RFKILL
 	help
 	  Bluetooth is low-cost, low-power, short-range wireless technology.
 	  It was designed as a replacement for cables and other short-range
Signature · 1 line
1.6.2.2
author maintainer
#2 lore raw

PATCH 2/2 Bluetooth: Fix issue with uninitialized nsh.type in DTL-1 driver

The current build shows a warning with the DTL-1 driver:

CC [M] drivers/bluetooth/dtl1_cs.o
drivers/bluetooth/dtl1_cs.c: In function ‘dtl1_hci_send_frame’:
drivers/bluetooth/dtl1_cs.c:396: warning: ‘nsh.type’ may be used uninitialized i n this function

Fix this by adding a proper error for unknown packet types.

---
 drivers/bluetooth/dtl1_cs.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/bluetooth/dtl1_cs.c +2 -0
index 901bdd9..2cc7b32 100644
@@ -415,6 +415,8 @@ static int dtl1_hci_send_frame(struct sk_buff *skb)
 		hdev->stat.sco_tx++;
 		nsh.type = 0x83;
 		break;
+	default:
+		return -EILSEQ;
 	};
 
 	nsh.zero = 0;
Signature · 1 line
1.6.2.2
maintainer
#3 lore raw
In reply to Marcel Holtmann

From: Marcel Holtmann <[email protected]>
Date: Sun, 14 Jun 2009 15:39:09 +0200

Quoted 4 lines

I have two more patches that should go into your networking merge. The first is fixing a build fallout from missing dependencies on the RFKILL subsystem. The second is fixing a build warning that is potentially dangerous with DTL-1 hardware when using unknown packet types.

Pulled, thanks.