Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754662Ab0F0GsP (ORCPT ); Sun, 27 Jun 2010 02:48:15 -0400 Received: from mail-pw0-f46.google.com ([209.85.160.46]:64699 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753446Ab0F0Gru (ORCPT ); Sun, 27 Jun 2010 02:47:50 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=mWR3hmJEU/tqlHggrtQvXifUN0wrEJEOkaDcd21XE48Q4cLO4K7Q6sdr1T1ZeJYt3x d2Hh4WI2lhpS33WJ2wH1jOaW0e/QDftFeaS4BJbPlIK04HFAfdqrF6pAPdCU3yVDKTH+ WP/2SbFRprNnERHn0oQ/y8dPAA/1cMu61yvz4= From: "Justin P. Mattock" To: linux-kernel@vger.kernel.org Cc: dhowells@redhat.com, sds@tycho.nsa.gov, lenb@kernel.org, linux-bluetooth@vger.kernel.org, "Justin P. Mattock" Subject: [PATCH 5/5]bluetooth:hci_bcsp Fix operation on 'bcsp->msgq_txseq' may be undefined Date: Sat, 26 Jun 2010 23:47:26 -0700 Message-Id: <1277621246-10960-6-git-send-email-justinmattock@gmail.com> X-Mailer: git-send-email 1.7.1.rc1.21.gf3bd6 In-Reply-To: <1277621246-10960-1-git-send-email-justinmattock@gmail.com> References: <1277621246-10960-1-git-send-email-justinmattock@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1558 Lines: 44 Im seeing this building the kernel with gcc 4.6.0 CC [M] drivers/bluetooth/hci_bcsp.o drivers/bluetooth/hci_bcsp.c: In function 'bcsp_prepare_pkt': drivers/bluetooth/hci_bcsp.c:247:20: warning: operation on 'bcsp->msgq_txseq' may be undefined Hopefully the below is a fix for this. Please let me know. Signed-off-by: Justin P. Mattock --- drivers/bluetooth/hci_bcsp.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/bluetooth/hci_bcsp.c b/drivers/bluetooth/hci_bcsp.c index 40aec0f..0f892e7 100644 --- a/drivers/bluetooth/hci_bcsp.c +++ b/drivers/bluetooth/hci_bcsp.c @@ -182,7 +182,7 @@ static struct sk_buff *bcsp_prepare_pkt(struct bcsp_struct *bcsp, u8 *data, struct sk_buff *nskb; u8 hdr[4], chan; u16 BCSP_CRC_INIT(bcsp_txmsg_crc); - int rel, i; + int rel, i, ret; switch (pkt_type) { case HCI_ACLDATA_PKT: @@ -243,8 +243,8 @@ static struct sk_buff *bcsp_prepare_pkt(struct bcsp_struct *bcsp, u8 *data, if (rel) { hdr[0] |= 0x80 + bcsp->msgq_txseq; - BT_DBG("Sending packet with seqno %u", bcsp->msgq_txseq); - bcsp->msgq_txseq = ++(bcsp->msgq_txseq) & 0x07; + BT_DBG("Sending packet with seqno %u", bcsp->msgq_txseq | ret); + ret = ++(bcsp->msgq_txseq) & 0x07; } if (bcsp->use_crc) -- 1.7.1.rc1.21.gf3bd6 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/