Re: H5 Line Discipline for Bluetooth

From: Andrei Emeltchenko
Date: Wed Oct 26 2011 - 06:05:47 EST


Hi Mark,

On Tue, Oct 25, 2011 at 6:40 PM, Mark Mendelsohn <mendelso@xxxxxxxxxxxx> wrote:
> From ab30529d2c1d32dd397e4e5c165572a2ab43f597 Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Wed, 10 Aug 2011 14:26:34 -0700
> Subject: [PATCH 01/10] adding H5
>
> ---
>  drivers/bluetooth/1           |  812 ++++++++++++++++++++++++++++++++++++++++
>  drivers/bluetooth/Kconfig     |   11 +
>  drivers/bluetooth/Makefile    |    1 +
>  drivers/bluetooth/hci_h5.c    |  817 +++++++++++++++++++++++++++++++++++++++++
>  drivers/bluetooth/hci_ldisc.c |   32 +-
>  drivers/bluetooth/hci_uart.h  |    8 +-
>  6 files changed, 1669 insertions(+), 12 deletions(-)
>  create mode 100644 drivers/bluetooth/1

Is this a typo?

BTW: CC bluetooth patches to linux-bluetooth mailing list.

-- andrei

>  create mode 100644 drivers/bluetooth/hci_h5.c
>
> diff --git a/drivers/bluetooth/1 b/drivers/bluetooth/1
> new file mode 100644
> index 0000000..0d6eb8a
> --- /dev/null
> +++ b/drivers/bluetooth/1
> @@ -0,0 +1,812 @@
> +/*
> + *
> + *  Bluetooth HCI UART driver
> + *
> + *  Copyright (C) 2002-2003  Fabrizio Gennari <fabrizio.gennari@xxxxxxxxxxx>
> + *  Copyright (C) 2004-2005  Marcel Holtmann <marcel@xxxxxxxxxxxx>
> + *
> + *
> + *  This program is free software; you can redistribute it and/or modify
> + *  it under the terms of the GNU General Public License as published by
> + *  the Free Software Foundation; either version 2 of the License, or
> + *  (at your option) any later version.
> + *
> + *  This program is distributed in the hope that it will be useful,
> + *  but WITHOUT ANY WARRANTY; without even the implied warranty of
> + *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + *  GNU General Public License for more details.
> + *
> + *  You should have received a copy of the GNU General Public License
> + *  along with this program; if not, write to the Free Software
> + *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
> + *
> + */
> +
> +#include <linux/module.h>
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/types.h>
> +#include <linux/fcntl.h>
> +#include <linux/interrupt.h>
> +#include <linux/ptrace.h>
> +#include <linux/poll.h>
> +
> +#include <linux/slab.h>
> +#include <linux/tty.h>
> +#include <linux/errno.h>
> +#include <linux/string.h>
> +#include <linux/signal.h>
> +#include <linux/ioctl.h>
> +#include <linux/skbuff.h>
> +#include <linux/bitrev.h>
> +#include <asm/unaligned.h>
> +
> +#include <net/bluetooth/bluetooth.h>
> +#include <net/bluetooth/hci_core.h>
> +
> +#include "hci_uart.h"
> +
> +#define VERSION "0.3"
> +#undef BT_DBG
> +#define BT_DBG printk
> +
> +static int h5txcrc = 1;
> +static int h5extn = 1;
> +
> +#define H5_TXWINSIZE   4
> +
> +#define H5_ACK_PKT     0x05
> +#define H5_LE_PKT      0x06
> +
> +struct h5_struct {
> +       struct sk_buff_head unack;      /* Unack'ed packets queue */
> +       struct sk_buff_head rel;        /* Reliable packets queue */
> +       struct sk_buff_head unrel;      /* Unreliable packets queue */
> +
> +       unsigned long rx_count;
> +       struct  sk_buff *rx_skb;
> +       u8      rxseq_txack;            /* rxseq == txack. */
> +       u8      rxack;                  /* Last packet sent by us that the peer ack'ed */
> +       struct  timer_list th5;
> +
> +       enum {
> +               H5_W4_PKT_DELIMITER,
> +               H5_W4_PKT_START,
> +               H5_W4_H5_HDR,
> +               H5_W4_DATA,
> +               H5_W4_CRC
> +       } rx_state;
> +
> +       enum {
> +               H5_ESCSTATE_NOESC,
> +               H5_ESCSTATE_ESC
> +       } rx_esc_state;
> +
> +       u8      use_crc;
> +       u16     message_crc;
> +       u8      txack_req;              /* Do we need to send ack's to the peer? */
> +
> +       /* Reliable packet sequence number - used to assign seq to each rel pkt. */
> +       u8      msgq_txseq;
> +};
> +
> +/* ---- H5 CRC calculation ---- */
> +
> +/* Table for calculating CRC for polynomial 0x1021, LSB processed first,
> +initial value 0xffff, bits shifted in reverse order. */
> +
> +static const u16 crc_table[] = {
> +       0x0000, 0x1081, 0x2102, 0x3183,
> +       0x4204, 0x5285, 0x6306, 0x7387,
> +       0x8408, 0x9489, 0xa50a, 0xb58b,
> +       0xc60c, 0xd68d, 0xe70e, 0xf78f
> +};
> +
> +/* Initialise the crc calculator */
> +#define H5_CRC_INIT(x) x = 0xffff
> +
> +/*
> +   Update crc with next data byte
> +
> +   Implementation note
> +        The data byte is treated as two nibbles.  The crc is generated
> +        in reverse, i.e., bits are fed into the register from the top.
> +*/
> +static void h5_crc_update(u16 *crc, u8 d)
> +{
> +       u16 reg = *crc;
> +
> +       printk("%s\n", __func__);
> +
> +       reg = (reg >> 4) ^ crc_table[(reg ^ d) & 0x000f];
> +       reg = (reg >> 4) ^ crc_table[(reg ^ (d >> 4)) & 0x000f];
> +
> +       *crc = reg;
> +}
> +
> +/* ---- H5 core ---- */
> +
> +static void h5_slip_msgdelim(struct sk_buff *skb)
> +{
> +       const char pkt_delim = 0xc0;
> +
> +       printk("%s\n", __func__);
> +
> +       memcpy(skb_put(skb, 1), &pkt_delim, 1);
> +}
> +
> +static void h5_slip_one_byte(struct sk_buff *skb, u8 c)
> +{
> +       const char esc_c0[2] = { 0xdb, 0xdc };
> +       const char esc_db[2] = { 0xdb, 0xdd };
> +
> +       printk("%s\n", __func__);
> +
> +       switch (c) {
> +       case 0xc0:
> +               memcpy(skb_put(skb, 2), &esc_c0, 2);
> +               break;
> +       case 0xdb:
> +               memcpy(skb_put(skb, 2), &esc_db, 2);
> +               break;
> +       default:
> +               memcpy(skb_put(skb, 1), &c, 1);
> +       }
> +}
> +
> +static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +
> +       printk("%s\n", __func__);
> +
> +       if (skb->len > 0xFFF) {
> +               BT_ERR("Packet too long");
> +               kfree_skb(skb);
> +               return 0;
> +       }
> +
> +       switch (bt_cb(skb)->pkt_type) {
> +       case HCI_ACLDATA_PKT:
> +       case HCI_COMMAND_PKT:
> +               skb_queue_tail(&h5->rel, skb);
> +               break;
> +
> +       case HCI_SCODATA_PKT:
> +               skb_queue_tail(&h5->unrel, skb);
> +               break;
> +
> +       default:
> +               BT_ERR("Unknown packet type");
> +               kfree_skb(skb);
> +               break;
> +       }
> +
> +       return 0;
> +}
> +
> +static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> +               int len, int pkt_type)
> +{
> +       struct sk_buff *nskb;
> +       u8 hdr[4], chan;
> +       u16 H5_CRC_INIT(h5_txmsg_crc);
> +       int rel, i;
> +
> +       printk("%s\n", __func__);
> +       printk("packet type %d\n", pkt_type);
> +
> +       switch (pkt_type) {
> +       case HCI_ACLDATA_PKT:
> +               chan = 2;       /* H5 ACL channel */
> +               rel = 1;        /* reliable channel */
> +               break;
> +       case HCI_COMMAND_PKT:
> +               chan = 1;       /* H5 cmd/evt channel */
> +               rel = 1;        /* reliable channel */
> +               break;
> +       case HCI_SCODATA_PKT:
> +               chan = 3;       /* H5 SCO channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       case H5_LE_PKT:
> +               chan = 15;      /* H5 LE channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       case H5_ACK_PKT:
> +               chan = 0;       /* H5 internal channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       default:
> +               BT_ERR("Unknown packet type");
> +               return NULL;
> +       }
> +
> +       printk("chan %d\n", chan);
> +
> +       if (h5extn && chan == 14) {
> +               __le16 opcode = ((struct hci_command_hdr *)data)->opcode;
> +
> +               /* Vendor specific commands */
> +               if (hci_opcode_ogf(__le16_to_cpu(opcode)) == 0x3f) {
> +                       u8 desc = *(data + HCI_COMMAND_HDR_SIZE);
> +                       if ((desc & 0xf0) == 0xc0) {
> +                               data += HCI_COMMAND_HDR_SIZE + 1;
> +                               len  -= HCI_COMMAND_HDR_SIZE + 1;
> +                               chan = desc & 0x0f;
> +                       }
> +               }
> +       }
> +
> +       /* Max len of packet: (original len +4(h5 hdr) +2(crc))*2
> +          (because bytes 0xc0 and 0xdb are escaped, worst case is
> +          when the packet is all made of 0xc0 and 0xdb :) )
> +          + 2 (0xc0 delimiters at start and end). */
> +
> +       nskb = alloc_skb((len + 6) * 2 + 2, GFP_ATOMIC);
> +       if (!nskb)
> +               return NULL;
> +
> +       bt_cb(nskb)->pkt_type = pkt_type;
> +
> +       printk("starting delim\n");
> +       h5_slip_msgdelim(nskb);
> +
> +       hdr[0] = h5->rxseq_txack << 2;
> +       h5->txack_req = 0;
> +       BT_DBG("We request packet no %u to card\n", h5->rxseq_txack);
> +
> +       if (rel) {
> +               hdr[0] |= 0x01 + (h5->msgq_txseq << 5);
> +               BT_DBG("Sending packet with seqno %u\n", h5->msgq_txseq);
> +               h5->msgq_txseq = (h5->msgq_txseq + 1) & 0x07;
> +       }
> +
> +       if (h5->use_crc)
> +               hdr[0] |= 0x02;
> +
> +       hdr[1] = (len >> 4) | chan << 4;
> +       hdr[2] = (len & 0xff) << 4;
> +       hdr[3] = ~(hdr[0] + hdr[1] + hdr[2]);
> +
> +       /* Put H5 header */
> +       for (i = 0; i < 4; i++) {
> +               h5_slip_one_byte(nskb, hdr[i]);
> +
> +               if (h5->use_crc)
> +                       h5_crc_update(&h5_txmsg_crc, hdr[i]);
> +       }
> +
> +       /* Put payload */
> +       for (i = 0; i < len; i++) {
> +               h5_slip_one_byte(nskb, data[i]);
> +
> +               if (h5->use_crc)
> +                       h5_crc_update(&h5_txmsg_crc, data[i]);
> +       }
> +
> +       /* Put CRC */
> +       if (h5->use_crc) {
> +               h5_txmsg_crc = bitrev16(h5_txmsg_crc);
> +               h5_slip_one_byte(nskb, (u8) ((h5_txmsg_crc >> 8) & 0x00ff));
> +               h5_slip_one_byte(nskb, (u8) (h5_txmsg_crc & 0x00ff));
> +       }
> +
> +       printk("ending delim\n");
> +       h5_slip_msgdelim(nskb);
> +       return nskb;
> +}
> +
> +/* This is a rewrite of pkt_avail in AH5 */
> +static struct sk_buff *h5_dequeue(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       unsigned long flags;
> +       struct sk_buff *skb;
> +
> +       printk("%s\n", __func__);
> +
> +       /* First of all, check for unreliable messages in the queue,
> +          since they have priority */
> +
> +       if ((skb = skb_dequeue(&h5->unrel)) != NULL) {
> +               printk("got skb\n");
> +
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, skb->data, skb->len, bt_cb(skb)->pkt_type);
> +               if (nskb) {
> +                       kfree_skb(skb);
> +                       return nskb;
> +               } else {
> +                       skb_queue_head(&h5->unrel, skb);
> +                       BT_ERR("Could not dequeue pkt because alloc_skb failed");
> +               }
> +       }
> +
> +       printk("after skb\n");
> +
> +       /* Now, try to send a reliable pkt. We can only send a
> +          reliable packet if the number of packets sent but not yet ack'ed
> +          is < than the winsize */
> +
> +       spin_lock_irqsave_nested(&h5->unack.lock, flags, SINGLE_DEPTH_NESTING);
> +
> +       if (h5->unack.qlen < H5_TXWINSIZE && (skb = skb_dequeue(&h5->rel)) != NULL) {
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, skb->data, skb->len, bt_cb(skb)->pkt_type);
> +               if (nskb) {
> +                       __skb_queue_tail(&h5->unack, skb);
> +                       mod_timer(&h5->th5, jiffies + HZ / 4);
> +                       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +                       return nskb;
> +               } else {
> +                       skb_queue_head(&h5->rel, skb);
> +                       BT_ERR("Could not dequeue pkt because alloc_skb failed");
> +               }
> +       }
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       printk("after spin_unlock\n");
> +
> +       /* We could not send a reliable packet, either because there are
> +          none or because there are too many unack'ed pkts. Did we receive
> +          any packets we have not acknowledged yet ? */
> +
> +       if (h5->txack_req) {
> +               /* if so, craft an empty ACK pkt and send it on H5 unreliable
> +                  channel 0 */
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, NULL, 0, H5_ACK_PKT);
> +               return nskb;
> +       }
> +
> +       printk("we have nothing to send\n");
> +       /* We have nothing to send */
> +       return NULL;
> +}
> +
> +static int h5_flush(struct hci_uart *hu)
> +{
> +       BT_DBG("hu %p", hu);
> +       printk("%s\n", __func__);
> +       return 0;
> +}
> +
> +/* Remove ack'ed packets */
> +static void h5_pkt_cull(struct h5_struct *h5)
> +{
> +       struct sk_buff *skb, *tmp;
> +       unsigned long flags;
> +       int i, pkts_to_be_removed;
> +       u8 seqno;
> +
> +       printk("%s\n", __func__);
> +
> +       spin_lock_irqsave(&h5->unack.lock, flags);
> +
> +       pkts_to_be_removed = skb_queue_len(&h5->unack);
> +       seqno = h5->msgq_txseq;
> +
> +       while (pkts_to_be_removed) {
> +               if (h5->rxack == seqno)
> +                       break;
> +               pkts_to_be_removed--;
> +               seqno = (seqno - 1) & 0x07;
> +       }
> +
> +       if (h5->rxack != seqno)
> +               BT_ERR("Peer acked invalid packet");
> +
> +       BT_DBG("Removing %u pkts out of %u, up to seqno %u",
> +              pkts_to_be_removed, skb_queue_len(&h5->unack),
> +              (seqno - 1) & 0x07);
> +
> +       i = 0;
> +       skb_queue_walk_safe(&h5->unack, skb, tmp) {
> +               if (i >= pkts_to_be_removed)
> +                       break;
> +               i++;
> +
> +               __skb_unlink(skb, &h5->unack);
> +               kfree_skb(skb);
> +       }
> +
> +       if (skb_queue_empty(&h5->unack))
> +               del_timer(&h5->th5);
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       if (i != pkts_to_be_removed)
> +               BT_ERR("Removed only %u out of %u pkts", i, pkts_to_be_removed);
> +}
> +
> +/* Handle H5 link-establishment packets. When we
> +   detect a "sync" packet, symptom that the BT module has reset,
> +   we do nothing :) (yet) */
> +static void h5_handle_le_pkt(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       u8 conf_pkt[4]     = { 0xad, 0xef, 0xac, 0xed };
> +       u8 conf_rsp_pkt[4] = { 0xde, 0xad, 0xd0, 0xd0 };
> +       u8 sync_pkt[4]     = { 0x00, 0xd0, 0x01, 0x7e };
> +
> +       printk("%s\n", __func__);
> +
> +       /* spot "conf" pkts and reply with a "conf rsp" pkt */
> +       if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], conf_pkt, 4)) {
> +               struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> +
> +               BT_DBG("Found a LE conf pkt");
> +               if (!nskb)
> +                       return;
> +               memcpy(skb_put(nskb, 4), conf_rsp_pkt, 4);
> +               bt_cb(nskb)->pkt_type = H5_LE_PKT;
> +
> +               skb_queue_head(&h5->unrel, nskb);
> +               hci_uart_tx_wakeup(hu);
> +       }
> +       /* Spot "sync" pkts. If we find one...disaster! */
> +       else if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], sync_pkt, 4)) {
> +               BT_ERR("Found a LE sync pkt, card has reset");
> +       }
> +}
> +
> +static inline void h5_unslip_one_byte(struct h5_struct *h5, unsigned char byte)
> +{
> +       const u8 c0 = 0xc0, db = 0xdb;
> +
> +       printk("%s\n", __func__);
> +
> +       switch (h5->rx_esc_state) {
> +       case H5_ESCSTATE_NOESC:
> +               switch (byte) {
> +               case 0xdb:
> +                       h5->rx_esc_state = H5_ESCSTATE_ESC;
> +                       break;
> +               default:
> +                       memcpy(skb_put(h5->rx_skb, 1), &byte, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5->message_crc, byte);
> +                       h5->rx_count--;
> +               }
> +               break;
> +
> +       case H5_ESCSTATE_ESC:
> +               switch (byte) {
> +               case 0xdc:
> +                       memcpy(skb_put(h5->rx_skb, 1), &c0, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0xc0);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               case 0xdd:
> +                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0xdb);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               default:
> +                       BT_ERR ("Invalid byte %02x after esc byte", byte);
> +                       kfree_skb(h5->rx_skb);
> +                       h5->rx_skb = NULL;
> +                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                       h5->rx_count = 0;
> +               }
> +       }
> +}
> +
> +static void h5_complete_rx_pkt(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       int pass_up;
> +
> +       printk("%s\n", __func__);
> +
> +       if (h5->rx_skb->data[0] & 0x80) {       /* reliable pkt */
> +               BT_DBG("Received seqno %u from card", h5->rxseq_txack);
> +               h5->rxseq_txack++;
> +               h5->rxseq_txack %= 0x8;
> +               h5->txack_req    = 1;
> +
> +               /* If needed, transmit an ack pkt */
> +               hci_uart_tx_wakeup(hu);
> +       }
> +
> +       h5->rxack = (h5->rx_skb->data[0] >> 3) & 0x07;
> +       BT_DBG("Request for pkt %u from card", h5->rxack);
> +
> +       h5_pkt_cull(h5);
> +       if ((h5->rx_skb->data[1] & 0x0f) == 2 &&
> +                       h5->rx_skb->data[0] & 0x80) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_ACLDATA_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 4 &&
> +                       h5->rx_skb->data[0] & 0x80) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_EVENT_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 3) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_SCODATA_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 15 &&
> +                       !(h5->rx_skb->data[0] & 0x80)) {
> +               h5_handle_le_pkt(hu);
> +               pass_up = 0;
> +       } else
> +               pass_up = 0;
> +
> +       if (!pass_up) {
> +               struct hci_event_hdr hdr;
> +               u8 desc = (h5->rx_skb->data[1] & 0x0f);
> +
> +               if (desc != 0 && desc != 1) {
> +                       if (h5extn) {
> +                               desc |= 0xc0;
> +                               skb_pull(h5->rx_skb, 4);
> +                               memcpy(skb_push(h5->rx_skb, 1), &desc, 1);
> +
> +                               hdr.evt = 0xff;
> +                               hdr.plen = h5->rx_skb->len;
> +                               memcpy(skb_push(h5->rx_skb, HCI_EVENT_HDR_SIZE), &hdr, HCI_EVENT_HDR_SIZE);
> +                               bt_cb(h5->rx_skb)->pkt_type = HCI_EVENT_PKT;
> +
> +                               hci_recv_frame(h5->rx_skb);
> +                       } else {
> +                               BT_ERR ("Packet for unknown channel (%u %s)",
> +                                       h5->rx_skb->data[1] & 0x0f,
> +                                       h5->rx_skb->data[0] & 0x80 ?
> +                                       "reliable" : "unreliable");
> +                               kfree_skb(h5->rx_skb);
> +                       }
> +               } else
> +                       kfree_skb(h5->rx_skb);
> +       } else {
> +               /* Pull out H5 hdr */
> +               skb_pull(h5->rx_skb, 4);
> +
> +               hci_recv_frame(h5->rx_skb);
> +       }
> +
> +       h5->rx_state = H5_W4_PKT_DELIMITER;
> +       h5->rx_skb = NULL;
> +}
> +
> +static u16 h5_get_crc(struct h5_struct *h5)
> +{
> +       printk("%s\n", __func__);
> +
> +       return get_unaligned_be16(&h5->rx_skb->data[h5->rx_skb->len - 2]);
> +}
> +
> +/* Recv data */
> +static int h5_recv(struct hci_uart *hu, void *data, int count)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       register unsigned char *ptr;
> +
> +       printk("%s\n", __func__);
> +
> +       BT_DBG("hu %p count %d rx_state %d rx_count %ld",
> +               hu, count, h5->rx_state, h5->rx_count);
> +
> +       ptr = data;
> +       while (count) {
> +               if (h5->rx_count) {
> +                       if (*ptr == 0xc0) {
> +                               BT_ERR("Short H5 packet");
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_START;
> +                               h5->rx_count = 0;
> +                       } else
> +                               h5_unslip_one_byte(h5, *ptr);
> +
> +                       ptr++; count--;
> +                       continue;
> +               }
> +
> +               switch (h5->rx_state) {
> +               case H5_W4_H5_HDR:
> +                       if ((0xff & (u8) ~ (h5->rx_skb->data[0] + h5->rx_skb->data[1] +
> +                                       h5->rx_skb->data[2])) != h5->rx_skb->data[3]) {
> +                               BT_ERR("Error in H5 hdr checksum");
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_DELIMITER;
> +                               h5->rx_count = 0;
> +                               continue;
> +                       }
> +                       if (h5->rx_skb->data[0] & 0x80  /* reliable pkt */
> +                                       && (h5->rx_skb->data[0] & 0x07) != h5->rxseq_txack) {
> +                               BT_ERR ("Out-of-order packet arrived, got %u expected %u",
> +                                       h5->rx_skb->data[0] & 0x07, h5->rxseq_txack);
> +
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_DELIMITER;
> +                               h5->rx_count = 0;
> +                               continue;
> +                       }
> +                       h5->rx_state = H5_W4_DATA;
> +                       h5->rx_count = (h5->rx_skb->data[1] >> 4) +
> +                                       (h5->rx_skb->data[2] << 4);     /* May be 0 */
> +                       continue;
> +
> +               case H5_W4_DATA:
> +                       if (h5->rx_skb->data[0] & 0x40) {       /* pkt with crc */
> +                               h5->rx_state = H5_W4_CRC;
> +                               h5->rx_count = 2;
> +                       } else
> +                               h5_complete_rx_pkt(hu);
> +                       continue;
> +
> +               case H5_W4_CRC:
> +                       if (bitrev16(h5->message_crc) != h5_get_crc(h5)) {
> +                               BT_ERR ("Checksum failed: computed %04x received %04x",
> +                                       bitrev16(h5->message_crc),
> +                                       h5_get_crc(h5));
> +
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_DELIMITER;
> +                               h5->rx_count = 0;
> +                               continue;
> +                       }
> +                       skb_trim(h5->rx_skb, h5->rx_skb->len - 2);
> +                       h5_complete_rx_pkt(hu);
> +                       continue;
> +
> +               case H5_W4_PKT_DELIMITER:
> +                       switch (*ptr) {
> +                       case 0xc0:
> +                               h5->rx_state = H5_W4_PKT_START;
> +                               break;
> +                       default:
> +                               /*BT_ERR("Ignoring byte %02x", *ptr);*/
> +                               break;
> +                       }
> +                       ptr++; count--;
> +                       break;
> +
> +               case H5_W4_PKT_START:
> +                       switch (*ptr) {
> +                       case 0xc0:
> +                               ptr++; count--;
> +                               break;
> +
> +                       default:
> +                               h5->rx_state = H5_W4_H5_HDR;
> +                               h5->rx_count = 4;
> +                               h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                               H5_CRC_INIT(h5->message_crc);
> +
> +                               /* Do not increment ptr or decrement count
> +                                * Allocate packet. Max len of a H5 pkt=
> +                                * 0xFFF (payload) +4 (header) +2 (crc) */
> +
> +                               h5->rx_skb = bt_skb_alloc(0x1005, GFP_ATOMIC);
> +                               if (!h5->rx_skb) {
> +                                       BT_ERR("Can't allocate mem for new packet");
> +                                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                                       h5->rx_count = 0;
> +                                       return 0;
> +                               }
> +                               h5->rx_skb->dev = (void *) hu->hdev;
> +                               break;
> +                       }
> +                       break;
> +               }
> +       }
> +       return count;
> +}
> +
> +       /* Arrange to retransmit all messages in the relq. */
> +static void h5_timed_event(unsigned long arg)
> +{
> +       struct hci_uart *hu = (struct hci_uart *) arg;
> +       struct h5_struct *h5 = hu->priv;
> +       struct sk_buff *skb;
> +       unsigned long flags;
> +
> +       printk("%s\n", __func__);
> +
> +       BT_DBG("hu %p retransmitting %u pkts", hu, h5->unack.qlen);
> +
> +       spin_lock_irqsave_nested(&h5->unack.lock, flags, SINGLE_DEPTH_NESTING);
> +
> +       while ((skb = __skb_dequeue_tail(&h5->unack)) != NULL) {
> +               h5->msgq_txseq = (h5->msgq_txseq - 1) & 0x07;
> +               skb_queue_head(&h5->rel, skb);
> +       }
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       hci_uart_tx_wakeup(hu);
> +}
> +
> +static int h5_open(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5;
> +
> +       printk("%s\n", __func__);
> +
> +       BT_DBG("hu %p", hu);
> +
> +       h5 = kzalloc(sizeof(*h5), GFP_ATOMIC);
> +       if (!h5)
> +               return -ENOMEM;
> +
> +       hu->priv = h5;
> +       skb_queue_head_init(&h5->unack);
> +       skb_queue_head_init(&h5->rel);
> +       skb_queue_head_init(&h5->unrel);
> +
> +       init_timer(&h5->th5);
> +       h5->th5.function = h5_timed_event;
> +       h5->th5.data     = (u_long) hu;
> +
> +       h5->rx_state = H5_W4_PKT_DELIMITER;
> +
> +       if (h5txcrc)
> +               h5->use_crc = 1;
> +
> +       return 0;
> +}
> +
> +static int h5_close(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       hu->priv = NULL;
> +
> +       printk("%s\n", __func__);
> +
> +       BT_DBG("hu %p", hu);
> +
> +       skb_queue_purge(&h5->unack);
> +       skb_queue_purge(&h5->rel);
> +       skb_queue_purge(&h5->unrel);
> +       del_timer(&h5->th5);
> +
> +       kfree(h5);
> +       return 0;
> +}
> +
> +static struct hci_uart_proto h5 = {
> +       .id             = HCI_UART_H5,
> +       .open           = h5_open,
> +       .close          = h5_close,
> +       .enqueue        = h5_enqueue,
> +       .dequeue        = h5_dequeue,
> +       .recv           = h5_recv,
> +       .flush          = h5_flush
> +};
> +
> +int h5_init(void)
> +{
> +       int err = hci_uart_register_proto(&h5);
> +
> +       printk("%s\n", __func__);
> +
> +       if (!err)
> +               BT_INFO("HCI H5 protocol initialized");
> +       else
> +               BT_ERR("HCI H5 protocol registration failed");
> +
> +       return err;
> +}
> +
> +int h5_deinit(void)
> +{
> +       printk("%s\n", __func__);
> +
> +       return hci_uart_unregister_proto(&h5);
> +}
> +
> +module_param(h5txcrc, bool, 0644);
> +MODULE_PARM_DESC(h5txcrc, "Transmit CRC with every H5 packet");
> +
> +module_param(h5extn, bool, 0644);
> +MODULE_PARM_DESC(h5extn, "Convert HCI Extensions into H5 packets");
> diff --git a/drivers/bluetooth/Kconfig b/drivers/bluetooth/Kconfig
> index 058fbcc..28daf2d 100644
> --- a/drivers/bluetooth/Kconfig
> +++ b/drivers/bluetooth/Kconfig
> @@ -58,6 +58,17 @@ config BT_HCIUART_BCSP
>
>          Say Y here to compile support for HCI BCSP protocol.
>
> +config BT_HCIUART_H5
> +       bool "H5 protocol support"
> +       depends on BT_HCIUART
> +       select BITREVERSE
> +       help
> +         H5, also called 3 Wire or SLIP, is serial protocol for communication
> +         between Bluetooth device and host. This protocol can be used when
> +         no CTS/RTS, hardware flow control lines, are available.
> +
> +         Say Y here to compile support for HCI H5 protocol.
> +
>  config BT_HCIUART_LL
>        bool "HCILL protocol support"
>        depends on BT_HCIUART
> diff --git a/drivers/bluetooth/Makefile b/drivers/bluetooth/Makefile
> index 7e5aed5..452fc24 100644
> --- a/drivers/bluetooth/Makefile
> +++ b/drivers/bluetooth/Makefile
> @@ -26,4 +26,5 @@ hci_uart-y                            := hci_ldisc.o
>  hci_uart-$(CONFIG_BT_HCIUART_H4)       += hci_h4.o
>  hci_uart-$(CONFIG_BT_HCIUART_BCSP)     += hci_bcsp.o
>  hci_uart-$(CONFIG_BT_HCIUART_LL)       += hci_ll.o
> +hci_uart-$(CONFIG_BT_HCIUART_H5)       += hci_h5.o
>  hci_uart-objs                          := $(hci_uart-y)
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> new file mode 100644
> index 0000000..e2a1f69
> --- /dev/null
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -0,0 +1,817 @@
> +/*
> + *
> + *  Bluetooth HCI UART driver
> + *
> + *  Copyright (C) 2002-2003  Fabrizio Gennari <fabrizio.gennari@xxxxxxxxxxx>
> + *  Copyright (C) 2004-2005  Marcel Holtmann <marcel@xxxxxxxxxxxx>
> + *
> + *
> + *  This program is free software; you can redistribute it and/or modify
> + *  it under the terms of the GNU General Public License as published by
> + *  the Free Software Foundation; either version 2 of the License, or
> + *  (at your option) any later version.
> + *
> + *  This program is distributed in the hope that it will be useful,
> + *  but WITHOUT ANY WARRANTY; without even the implied warranty of
> + *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + *  GNU General Public License for more details.
> + *
> + *  You should have received a copy of the GNU General Public License
> + *  along with this program; if not, write to the Free Software
> + *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
> + *
> + */
> +
> +#include <linux/module.h>
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/types.h>
> +#include <linux/fcntl.h>
> +#include <linux/interrupt.h>
> +#include <linux/ptrace.h>
> +#include <linux/poll.h>
> +
> +#include <linux/slab.h>
> +#include <linux/tty.h>
> +#include <linux/errno.h>
> +#include <linux/string.h>
> +#include <linux/signal.h>
> +#include <linux/ioctl.h>
> +#include <linux/skbuff.h>
> +#include <linux/bitrev.h>
> +#include <asm/unaligned.h>
> +
> +#include <net/bluetooth/bluetooth.h>
> +#include <net/bluetooth/hci_core.h>
> +
> +#include <linux/crc16.h>
> +
> +#include "hci_uart.h"
> +
> +#define VERSION "0.3"
> +#undef BT_DBG
> +#define BT_DBG printk
> +
> +static int h5txcrc = 1;
> +static int h5extn = 1;
> +
> +#define H5_TXWINSIZE   4
> +
> +#define H5_ACK_PKT     0x00
> +#define H5_LE_PKT      0x0f
> +
> +struct h5_struct {
> +       struct sk_buff_head unack;      /* Unack'ed packets queue */
> +       struct sk_buff_head rel;        /* Reliable packets queue */
> +       struct sk_buff_head unrel;      /* Unreliable packets queue */
> +
> +       unsigned long rx_count;
> +       struct  sk_buff *rx_skb;
> +       u8      rxseq_txack;            /* rxseq == txack. */
> +       u8      rxack;                  /* Last packet sent by us that the peer ack'ed */
> +       struct  timer_list th5;
> +
> +       enum {
> +               H5_W4_PKT_DELIMITER,
> +               H5_W4_PKT_START,
> +               H5_W4_H5_HDR,
> +               H5_W4_DATA,
> +               H5_W4_CRC1,
> +               H5_W4_CRC
> +       } rx_state;
> +
> +       enum {
> +               H5_ESCSTATE_NOESC,
> +               H5_ESCSTATE_ESC
> +       } rx_esc_state;
> +
> +       u8      use_crc;
> +       u16     message_crc;
> +       u8      txack_req;              /* Do we need to send ack's to the peer? */
> +
> +       /* Reliable packet sequence number - used to assign seq to each rel pkt. */
> +       u8      msgq_txseq;
> +};
> +
> +/* ---- H5 CRC calculation ---- */
> +
> +/* Table for calculating CRC for polynomial 0x1021, LSB processed first,
> +initial value 0xffff, bits shifted in reverse order. */
> +
> +static const u16 crc_table[] = {
> +       0x0000, 0x1081, 0x2102, 0x3183,
> +       0x4204, 0x5285, 0x6306, 0x7387,
> +       0x8408, 0x9489, 0xa50a, 0xb58b,
> +       0xc60c, 0xd68d, 0xe70e, 0xf78f
> +};
> +
> +/* Initialise the crc calculator */
> +#define H5_CRC_INIT(x) x = 0xffff
> +
> +/*
> +   Update crc with next data byte
> +
> +   Implementation note
> +        The data byte is treated as two nibbles.  The crc is generated
> +        in reverse, i.e., bits are fed into the register from the top.
> +*/
> +static void h5_crc_update(u16 *crc, u8 d)
> +{
> +       u16 reg = *crc;
> +
> +       reg = (reg >> 4) ^ crc_table[(reg ^ d) & 0x000f];
> +       reg = (reg >> 4) ^ crc_table[(reg ^ (d >> 4)) & 0x000f];
> +
> +       *crc = reg;
> +}
> +
> +/* ---- H5 core ---- */
> +
> +static void h5_slip_msgdelim(struct sk_buff *skb)
> +{
> +       const char pkt_delim = 0xc0;
> +
> +       memcpy(skb_put(skb, 1), &pkt_delim, 1);
> +}
> +
> +static void h5_slip_one_byte(struct sk_buff *skb, u8 c)
> +{
> +       const char esc_c0[2] = { 0xdb, 0xdc };
> +       const char esc_db[2] = { 0xdb, 0xdd };
> +
> +       switch (c) {
> +       case 0xc0:
> +               memcpy(skb_put(skb, 2), &esc_c0, 2);
> +               break;
> +       case 0xdb:
> +               memcpy(skb_put(skb, 2), &esc_db, 2);
> +               break;
> +       default:
> +               memcpy(skb_put(skb, 1), &c, 1);
> +       }
> +}
> +
> +static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +
> +       if (skb->len > 0xFFF) {
> +               BT_ERR("Packet too long");
> +               kfree_skb(skb);
> +               return 0;
> +       }
> +
> +       switch (bt_cb(skb)->pkt_type) {
> +       case HCI_ACLDATA_PKT:
> +       case HCI_COMMAND_PKT:
> +               skb_queue_tail(&h5->rel, skb);
> +               break;
> +
> +       case HCI_SCODATA_PKT:
> +               skb_queue_tail(&h5->unrel, skb);
> +               break;
> +
> +       default:
> +               BT_ERR("Unknown packet type");
> +               kfree_skb(skb);
> +               break;
> +       }
> +
> +       return 0;
> +}
> +
> +static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> +               int len, int pkt_type)
> +{
> +       struct sk_buff *nskb;
> +       u8 hdr[4], chan;
> +       u16 H5_CRC_INIT(h5_txmsg_crc);
> +       int rel, i;
> +
> +       print_hex_dump_bytes("h5_prepare_pkt ", DUMP_PREFIX_NONE, data, len);
> +
> +       switch (pkt_type) {
> +       case HCI_ACLDATA_PKT:
> +               chan = 2;       /* H5 ACL channel */
> +               rel = 1;        /* reliable channel */
> +               break;
> +       case HCI_COMMAND_PKT:
> +               chan = 1;       /* H5 cmd/evt channel */
> +               rel = 1;        /* reliable channel */
> +               break;
> +       case HCI_SCODATA_PKT:
> +               chan = 3;       /* H5 SCO channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       case H5_LE_PKT:
> +               chan = 15;      /* H5 LE channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       case H5_ACK_PKT:
> +               chan = 0;       /* H5 internal channel */
> +               rel = 0;        /* unreliable channel */
> +               break;
> +       default:
> +               BT_ERR("Unknown packet type");
> +               return NULL;
> +       }
> +
> +       if (h5extn && chan == 14) {
> +               __le16 opcode = ((struct hci_command_hdr *)data)->opcode;
> +
> +               /* Vendor specific commands */
> +               if (hci_opcode_ogf(__le16_to_cpu(opcode)) == 0x3f) {
> +                       u8 desc = *(data + HCI_COMMAND_HDR_SIZE);
> +                       if ((desc & 0xf0) == 0xc0) {
> +                               data += HCI_COMMAND_HDR_SIZE + 1;
> +                               len  -= HCI_COMMAND_HDR_SIZE + 1;
> +                               chan = desc & 0x0f;
> +                       }
> +               }
> +       }
> +
> +       /* Max len of packet: (original len +4(h5 hdr) +2(crc))*2
> +          (because bytes 0xc0 and 0xdb are escaped, worst case is
> +          when the packet is all made of 0xc0 and 0xdb :) )
> +          + 2 (0xc0 delimiters at start and end). */
> +
> +       nskb = alloc_skb((len + 6) * 2 + 2, GFP_ATOMIC);
> +       if (!nskb)
> +               return NULL;
> +
> +       bt_cb(nskb)->pkt_type = pkt_type;
> +
> +       h5_slip_msgdelim(nskb);
> +
> +       hdr[0] = h5->rxseq_txack << 3;
> +       h5->txack_req = 0;
> +
> +       if (rel) {
> +               hdr[0] |= 0x80 + h5->msgq_txseq;
> +               BT_DBG("Sending packet with seqno %u\n", h5->msgq_txseq);
> +       }
> +
> +       if (h5->use_crc)
> +               hdr[0] |= 0x40;
> +
> +       hdr[1] = ((len << 4) & 0xff) | chan;
> +       hdr[2] = len >> 4;
> +       hdr[3] = ~(hdr[0] + hdr[1] + hdr[2]);
> +
> +       /* Put H5 header */
> +       for (i = 0; i < 4; i++) {
> +               h5_slip_one_byte(nskb, hdr[i]);
> +
> +               if (h5->use_crc) {
> +                       h5_crc_update(&h5_txmsg_crc, hdr[i]);
> +               }
> +       }
> +
> +       /* Put payload */
> +       for (i = 0; i < len; i++) {
> +               h5_slip_one_byte(nskb, data[i]);
> +
> +               if (h5->use_crc) {
> +                       h5_crc_update(&h5_txmsg_crc, data[i]);
> +               }
> +       }
> +
> +       /* Put CRC */
> +       if (h5->use_crc) {
> +               h5_txmsg_crc = bitrev16(h5_txmsg_crc);
> +               h5_slip_one_byte(nskb, (u8) ((h5_txmsg_crc >> 8) & 0x00ff));
> +               h5_slip_one_byte(nskb, (u8) (h5_txmsg_crc & 0x00ff));
> +       }
> +
> +       h5_slip_msgdelim(nskb);
> +
> +       print_hex_dump_bytes("h5_tx ", DUMP_PREFIX_NONE, nskb->data, nskb->len);
> +       return nskb;
> +}
> +
> +/* This is a rewrite of pkt_avail in AH5 */
> +static struct sk_buff *h5_dequeue(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       unsigned long flags;
> +       struct sk_buff *skb;
> +
> +       /* First of all, check for unreliable messages in the queue,
> +          since they have priority */
> +
> +       if ((skb = skb_dequeue(&h5->unrel)) != NULL) {
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, skb->data, skb->len, bt_cb(skb)->pkt_type);
> +               if (nskb) {
> +                       return nskb;
> +               } else {
> +                       skb_queue_head(&h5->unrel, skb);
> +                       BT_ERR("Could not dequeue pkt because alloc_skb failed");
> +               }
> +       }
> +
> +       /* Now, try to send a reliable pkt. We can only send a
> +          reliable packet if the number of packets sent but not yet ack'ed
> +          is < than the winsize */
> +
> +       spin_lock_irqsave_nested(&h5->unack.lock, flags, SINGLE_DEPTH_NESTING);
> +
> +       if (h5->unack.qlen < H5_TXWINSIZE && (skb = skb_dequeue(&h5->rel)) != NULL) {
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, skb->data, skb->len, bt_cb(skb)->pkt_type);
> +               if (nskb) {
> +                       __skb_queue_tail(&h5->unack, skb);
> +                       mod_timer(&h5->th5, jiffies + HZ / 4);
> +                       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +                       return nskb;
> +               } else {
> +                       skb_queue_head(&h5->rel, skb);
> +                       BT_ERR("Could not dequeue pkt because alloc_skb failed");
> +               }
> +       }
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       /* We could not send a reliable packet, either because there are
> +          none or because there are too many unack'ed pkts. Did we receive
> +          any packets we have not acknowledged yet ? */
> +
> +       if (h5->txack_req) {
> +               /* if so, craft an empty ACK pkt and send it on H5 unreliable
> +                  channel 0 */
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, NULL, 0, H5_ACK_PKT);
> +               return nskb;
> +       }
> +
> +       /* We have nothing to send */
> +       return NULL;
> +}
> +
> +static int h5_flush(struct hci_uart *hu)
> +{
> +       BT_DBG("hu %p\n", hu);
> +       return 0;
> +}
> +
> +/* Remove ack'ed packets */
> +static void h5_pkt_cull(struct h5_struct *h5)
> +{
> +       struct sk_buff *skb, *tmp;
> +       unsigned long flags;
> +       int i, pkts_to_be_removed;
> +       u8 seqno;
> +
> +       spin_lock_irqsave(&h5->unack.lock, flags);
> +
> +       pkts_to_be_removed = skb_queue_len(&h5->unack);
> +
> +       seqno = h5->msgq_txseq;
> +
> +       while (pkts_to_be_removed) {
> +               if (((h5->rxack - 1) & 0x07) == seqno)
> +                       break;
> +
> +               pkts_to_be_removed--;
> +               seqno = (seqno + 1) & 0x07;
> +       }
> +
> +/*
> +       if (h5->rxack != seqno)
> +               BT_ERR("Peer acked invalid packet");
> +*/
> +
> +       BT_DBG("Removing %u pkts out of %u, up to seqno %u\n",
> +              pkts_to_be_removed, skb_queue_len(&h5->unack),
> +              (seqno - 1) & 0x07);
> +
> +       i = 0;
> +       skb_queue_walk_safe(&h5->unack, skb, tmp) {
> +               if (i >= pkts_to_be_removed)
> +                       break;
> +               i++;
> +
> +               __skb_unlink(skb, &h5->unack);
> +               kfree_skb(skb);
> +       }
> +
> +       if (skb_queue_empty(&h5->unack))
> +               del_timer(&h5->th5);
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       if (i != pkts_to_be_removed)
> +               BT_ERR("Removed only %u out of %u pkts", i, pkts_to_be_removed);
> +}
> +
> +/* Handle H5 link-establishment packets. When we
> +   detect a "sync" packet, symptom that the BT module has reset,
> +   we do nothing :) (yet) */
> +
> +static void h5_handle_le_pkt(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       u8 conf_pkt[4]     = { 0xad, 0xef, 0xac, 0xed };
> +       u8 conf_rsp_pkt[4] = { 0xde, 0xad, 0xd0, 0xd0 };
> +       u8 sync_pkt[4]     = { 0x00, 0xd0, 0x01, 0x7e };
> +
> +       /* spot "conf" pkts and reply with a "conf rsp" pkt */
> +       if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], conf_pkt, 4)) {
> +               struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> +
> +               BT_DBG("Found a LE conf pkt\n");
> +               if (!nskb)
> +                       return;
> +               memcpy(skb_put(nskb, 4), conf_rsp_pkt, 4);
> +               bt_cb(nskb)->pkt_type = H5_LE_PKT;
> +
> +               skb_queue_head(&h5->unrel, nskb);
> +               hci_uart_tx_wakeup(hu);
> +       }
> +       /* Spot "sync" pkts. If we find one...disaster! */
> +       else if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], sync_pkt, 4)) {
> +               BT_ERR("Found a LE sync pkt, card has reset");
> +       }
> +}
> +
> +static inline void h5_unslip_one_byte(struct h5_struct *h5, unsigned char byte)
> +{
> +       const u8 c0 = 0xc0, db = 0xdb;
> +
> +       switch (h5->rx_esc_state) {
> +       case H5_ESCSTATE_NOESC:
> +               switch (byte) {
> +               case 0xdb:
> +                       h5->rx_esc_state = H5_ESCSTATE_ESC;
> +                       break;
> +               default:
> +                       memcpy(skb_put(h5->rx_skb, 1), &byte, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5->message_crc, byte);
> +                       h5->rx_count--;
> +               }
> +               break;
> +
> +       case H5_ESCSTATE_ESC:
> +               switch (byte) {
> +               case 0xdc:
> +                       memcpy(skb_put(h5->rx_skb, 1), &c0, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0xc0);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               case 0xdd:
> +                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0xdb);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               case 0xdf:
> +                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0x13);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               case 0xde:
> +                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +                       if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
> +                                       h5->rx_state != H5_W4_CRC)
> +                               h5_crc_update(&h5-> message_crc, 0x11);
> +                       h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                       h5->rx_count--;
> +                       break;
> +
> +               default:
> +                       BT_ERR ("Invalid byte %02x after esc byte", byte);
> +                       kfree_skb(h5->rx_skb);
> +                       h5->rx_skb = NULL;
> +                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                       h5->rx_count = 0;
> +               }
> +       }
> +}
> +
> +static void h5_complete_rx_pkt(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       int pass_up;
> +
> +       if (h5->rx_skb->data[0] & 0x80) {       /* reliable pkt */
> +               BT_DBG("Received seqno %u from card\n", h5->rxseq_txack);
> +               h5->rxseq_txack++;
> +               h5->rxseq_txack %= 0x8;
> +               h5->txack_req    = 1;
> +
> +               /* If needed, transmit an ack pkt */
> +               hci_uart_tx_wakeup(hu);
> +       }
> +
> +       h5->rxack = (h5->rx_skb->data[0] >> 3) & 0x07;
> +       BT_DBG("Request for pkt %u from card\n", h5->rxack);
> +
> +       h5_pkt_cull(h5);
> +
> +       if ((h5->rx_skb->data[1] & 0x0f) == 2 &&
> +                       h5->rx_skb->data[0] & 0x80) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_ACLDATA_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 4 &&
> +                       h5->rx_skb->data[0] & 0x80) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_EVENT_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 3) {
> +               bt_cb(h5->rx_skb)->pkt_type = HCI_SCODATA_PKT;
> +               pass_up = 1;
> +       } else if ((h5->rx_skb->data[1] & 0x0f) == 15 &&
> +                       !(h5->rx_skb->data[0] & 0x80)) {
> +               h5_handle_le_pkt(hu);
> +               pass_up = 0;
> +       } else
> +               pass_up = 0;
> +
> +       if (!pass_up) {
> +               struct hci_event_hdr hdr;
> +               u8 desc = (h5->rx_skb->data[1] & 0x0f);
> +
> +               if (desc != 0 && desc != 1) {
> +                       if (h5extn) {
> +                               desc |= 0xc0;
> +                               skb_pull(h5->rx_skb, 4);
> +                               memcpy(skb_push(h5->rx_skb, 1), &desc, 1);
> +
> +                               hdr.evt = 0xff;
> +                               hdr.plen = h5->rx_skb->len;
> +                               memcpy(skb_push(h5->rx_skb, HCI_EVENT_HDR_SIZE), &hdr, HCI_EVENT_HDR_SIZE);
> +                               bt_cb(h5->rx_skb)->pkt_type = HCI_EVENT_PKT;
> +
> +                               hci_recv_frame(h5->rx_skb);
> +                       } else {
> +                               BT_ERR ("Packet for unknown channel (%u %s)",
> +                                       h5->rx_skb->data[1] & 0x0f,
> +                                       h5->rx_skb->data[0] & 0x80 ?
> +                                       "reliable" : "unreliable");
> +                               kfree_skb(h5->rx_skb);
> +                       }
> +               } else
> +                       kfree_skb(h5->rx_skb);
> +       } else {
> +               /* Pull out H5 hdr */
> +               skb_pull(h5->rx_skb, 4);
> +
> +               hci_recv_frame(h5->rx_skb);
> +       }
> +
> +       h5->rx_state = H5_W4_PKT_DELIMITER;
> +       h5->rx_skb = NULL;
> +}
> +
> +static u16 h5_get_crc(struct h5_struct *h5)
> +{
> +       return get_unaligned_be16(&h5->rx_skb->data[h5->rx_skb->len - 2]);
> +}
> +
> +/* Recv data */
> +static int h5_recv(struct hci_uart *hu, void *data, int count)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       register unsigned char *ptr;
> +       int i;
> +
> +       BT_DBG("hu %p count %d rx_state %d rx_count %ld\n",
> +               hu, count, h5->rx_state, h5->rx_count);
> +
> +       print_hex_dump_bytes("h5_recv ", DUMP_PREFIX_NONE, data, count);
> +
> +       ptr = data;
> +
> +       while (count) {
> +               if (h5->rx_count) {
> +                       if (*ptr == 0xc0) {
> +                               BT_ERR("Short H5 packet");
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_START;
> +                               h5->rx_count = 0;
> +                       } else
> +                               h5_unslip_one_byte(h5, *ptr);
> +
> +                       ptr++; count--;
> +                       continue;
> +               }
> +
> +               switch (h5->rx_state) {
> +               case H5_W4_H5_HDR:
> +                       if ((0xff & (u8) ~ (h5->rx_skb->data[0] + h5->rx_skb->data[1] +
> +                                       h5->rx_skb->data[2])) != h5->rx_skb->data[3]) {
> +                               BT_ERR("Error in H5 hdr checksum");
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_DELIMITER;
> +                               h5->rx_count = 0;
> +                               continue;
> +                       }
> +
> +
> +                       if (h5->rx_skb->data[0] & 0x80) {       /* reliable pkt */
> +                               printk("%0x != %0x ? %d\n", h5->rx_skb->data[0] & 0x07,
> +                                       h5->rxseq_txack,
> +                                       (h5->rx_skb->data[0] & 0x07) != h5->rxseq_txack);
> +
> +                               if ((h5->rx_skb->data[0] & 0x07) != h5->rxseq_txack) {
> +                                       BT_ERR ("Out-of-order packet arrived, got %u expected %u",
> +                                               h5->rx_skb->data[0] & 0x07, h5->rxseq_txack);
> +
> +                                       kfree_skb(h5->rx_skb);
> +                                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                                       h5->rx_count = 0;
> +                                       continue;
> +                               }
> +                       }
> +
> +                       h5->rx_state = H5_W4_DATA;
> +                       h5->rx_count = (h5->rx_skb->data[1] >> 4) +
> +                                       (h5->rx_skb->data[2] << 4);     /* May be 0 */
> +                       continue;
> +
> +               case H5_W4_DATA:
> +                       if (h5->rx_skb->data[0] & 0x40) {       /* pkt with crc */
> +                               // h5->rx_state = H5_W4_CRC1;
> +                               h5->rx_state = H5_W4_CRC;
> +                               h5->rx_count = 2;
> +                       } else {
> +                               h5_complete_rx_pkt(hu);
> +                       }
> +
> +                       continue;
> +
> +               case H5_W4_CRC1:
> +                       h5->rx_state = H5_W4_CRC;
> +                       count--;
> +                       continue;
> +
> +               case H5_W4_CRC:
> +                       if (bitrev16(h5->message_crc) != h5_get_crc(h5)) {
> +                               BT_ERR ("Checksum failed: computed %04x received %04x",
> +                                       bitrev16(h5->message_crc),
> +                                       h5_get_crc(h5));
> +
> +                               kfree_skb(h5->rx_skb);
> +                               h5->rx_state = H5_W4_PKT_DELIMITER;
> +                               h5->rx_count = 0;
> +                               continue;
> +                       }
> +
> +                       skb_trim(h5->rx_skb, h5->rx_skb->len - 2);
> +                       h5_complete_rx_pkt(hu);
> +                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                       count--;
> +                       continue;
> +
> +               case H5_W4_PKT_DELIMITER:
> +                       switch (*ptr) {
> +                       case 0xc0:
> +                               h5->rx_state = H5_W4_PKT_START;
> +                               break;
> +                       default:
> +                               /*BT_ERR("Ignoring byte %02x", *ptr);*/
> +                               break;
> +                       }
> +                       ptr++; count--;
> +                       break;
> +
> +               case H5_W4_PKT_START:
> +                       switch (*ptr) {
> +                       case 0xc0:
> +                               ptr++; count--;
> +                               break;
> +
> +                       default:
> +                               h5->rx_state = H5_W4_H5_HDR;
> +                               h5->rx_count = 4;
> +                               h5->rx_esc_state = H5_ESCSTATE_NOESC;
> +                               H5_CRC_INIT(h5->message_crc);
> +
> +                               /* Do not increment ptr or decrement count
> +                                * Allocate packet. Max len of a H5 pkt=
> +                                * 0xFFF (payload) +4 (header) +2 (crc) */
> +
> +                               h5->rx_skb = bt_skb_alloc(0x1005, GFP_ATOMIC);
> +                               if (!h5->rx_skb) {
> +                                       BT_ERR("Can't allocate mem for new packet");
> +                                       h5->rx_state = H5_W4_PKT_DELIMITER;
> +                                       h5->rx_count = 0;
> +                                       return 0;
> +                               }
> +                               h5->rx_skb->dev = (void *) hu->hdev;
> +                               break;
> +                       }
> +                       break;
> +               }
> +       }
> +       return count;
> +}
> +
> +static int h5_timed_event(unsigned long arg)
> +{
> +       struct hci_uart *hu = (struct hci_uart *) arg;
> +       struct h5_struct *h5 = hu->priv;
> +       struct sk_buff *skb;
> +       unsigned long flags;
> +
> +       BT_DBG("hu %p retransmitting %u pkts\n", hu, h5->unack.qlen);
> +
> +       spin_lock_irqsave_nested(&h5->unack.lock, flags, SINGLE_DEPTH_NESTING);
> +
> +       while ((skb = __skb_dequeue_tail(&h5->unack)) != NULL) {
> +               h5->msgq_txseq = (h5->msgq_txseq - 1) & 0x07;
> +               skb_queue_head(&h5->rel, skb);
> +       }
> +
> +       spin_unlock_irqrestore(&h5->unack.lock, flags);
> +
> +       hci_uart_tx_wakeup(hu);
> +}
> +
> +static int h5_open(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5;
> +
> +       BT_DBG("hu %p\n", hu);
> +
> +       h5 = kzalloc(sizeof(*h5), GFP_ATOMIC);
> +       if (!h5)
> +               return -ENOMEM;
> +
> +       hu->priv = h5;
> +       skb_queue_head_init(&h5->unack);
> +       skb_queue_head_init(&h5->rel);
> +       skb_queue_head_init(&h5->unrel);
> +
> +       init_timer(&h5->th5);
> +       h5->th5.function = h5_timed_event;
> +       h5->th5.data     = (u_long) hu;
> +
> +       h5->rx_state = H5_W4_PKT_DELIMITER;
> +
> +       if (h5txcrc)
> +               h5->use_crc = 1;
> +
> +       return 0;
> +}
> +
> +static int h5_close(struct hci_uart *hu)
> +{
> +       struct h5_struct *h5 = hu->priv;
> +       hu->priv = NULL;
> +
> +       BT_DBG("hu %p\n", hu);
> +
> +       skb_queue_purge(&h5->unack);
> +       skb_queue_purge(&h5->rel);
> +       skb_queue_purge(&h5->unrel);
> +       del_timer(&h5->th5);
> +
> +       kfree(h5);
> +       return 0;
> +}
> +
> +static struct hci_uart_proto h5 = {
> +       .id             = HCI_UART_H5,
> +       .open           = h5_open,
> +       .close          = h5_close,
> +       .enqueue        = h5_enqueue,
> +       .dequeue        = h5_dequeue,
> +       .recv           = h5_recv,
> +       .flush          = h5_flush
> +};
> +
> +int h5_init(void)
> +{
> +       int err = hci_uart_register_proto(&h5);
> +
> +       if (!err)
> +               BT_INFO("HCI H5 protocol initialized");
> +       else
> +               BT_ERR("HCI H5 protocol registration failed");
> +
> +       return err;
> +}
> +
> +int h5_deinit(void)
> +{
> +       return hci_uart_unregister_proto(&h5);
> +}
> +
> +module_param(h5txcrc, bool, 0644);
> +MODULE_PARM_DESC(h5txcrc, "Transmit CRC with every H5 packet");
> +
> +module_param(h5extn, bool, 0644);
> +MODULE_PARM_DESC(h5extn, "Convert HCI Extensions into H5 packets");
> diff --git a/drivers/bluetooth/hci_ldisc.c b/drivers/bluetooth/hci_ldisc.c
> index 283b127..8099d9e 100644
> --- a/drivers/bluetooth/hci_ldisc.c
> +++ b/drivers/bluetooth/hci_ldisc.c
> @@ -47,6 +47,8 @@
>  #include "hci_uart.h"
>
>  #define VERSION "2.2"
> +#undef BT_DBG
> +#define BT_DBG printk
>
>  static int reset = 0;
>
> @@ -123,13 +125,14 @@ int hci_uart_tx_wakeup(struct hci_uart *hu)
>        struct tty_struct *tty = hu->tty;
>        struct hci_dev *hdev = hu->hdev;
>        struct sk_buff *skb;
> +       int i;
>
>        if (test_and_set_bit(HCI_UART_SENDING, &hu->tx_state)) {
>                set_bit(HCI_UART_TX_WAKEUP, &hu->tx_state);
>                return 0;
>        }
>
> -       BT_DBG("");
> +       BT_DBG("\n");
>
>  restart:
>        clear_bit(HCI_UART_TX_WAKEUP, &hu->tx_state);
> @@ -138,6 +141,7 @@ restart:
>                int len;
>
>                set_bit(TTY_DO_WRITE_WAKEUP, &tty->flags);
> +
>                len = tty->ops->write(tty, skb->data, skb->len);
>                hdev->stat.byte_tx += len;
>
> @@ -162,7 +166,7 @@ restart:
>  /* Initialize device */
>  static int hci_uart_open(struct hci_dev *hdev)
>  {
> -       BT_DBG("%s %p", hdev->name, hdev);
> +       BT_DBG("%s %p\n", hdev->name, hdev);
>
>        /* Nothing to do for UART driver */
>
> @@ -177,7 +181,7 @@ static int hci_uart_flush(struct hci_dev *hdev)
>        struct hci_uart *hu  = (struct hci_uart *) hdev->driver_data;
>        struct tty_struct *tty = hu->tty;
>
> -       BT_DBG("hdev %p tty %p", hdev, tty);
> +       BT_DBG("hdev %p tty %p\n", hdev, tty);
>
>        if (hu->tx_skb) {
>                kfree_skb(hu->tx_skb); hu->tx_skb = NULL;
> @@ -196,7 +200,7 @@ static int hci_uart_flush(struct hci_dev *hdev)
>  /* Close device */
>  static int hci_uart_close(struct hci_dev *hdev)
>  {
> -       BT_DBG("hdev %p", hdev);
> +       BT_DBG("hdev %p\n", hdev);
>
>        if (!test_and_clear_bit(HCI_RUNNING, &hdev->flags))
>                return 0;
> @@ -224,7 +228,7 @@ static int hci_uart_send_frame(struct sk_buff *skb)
>        hu = (struct hci_uart *) hdev->driver_data;
>        tty = hu->tty;
>
> -       BT_DBG("%s: type %d len %d", hdev->name, bt_cb(skb)->pkt_type, skb->len);
> +       BT_DBG("%s: type %d len %d\n", hdev->name, bt_cb(skb)->pkt_type, skb->len);
>
>        hu->proto->enqueue(hu, skb);
>
> @@ -238,7 +242,7 @@ static void hci_uart_destruct(struct hci_dev *hdev)
>        if (!hdev)
>                return;
>
> -       BT_DBG("%s", hdev->name);
> +       BT_DBG("%s\n", hdev->name);
>        kfree(hdev->driver_data);
>  }
>
> @@ -256,7 +260,7 @@ static int hci_uart_tty_open(struct tty_struct *tty)
>  {
>        struct hci_uart *hu = (void *) tty->disc_data;
>
> -       BT_DBG("tty %p", tty);
> +       BT_DBG("tty %p\n", tty);
>
>        /* FIXME: This btw is bogus, nothing requires the old ldisc to clear
>           the pointer */
> @@ -300,7 +304,7 @@ static void hci_uart_tty_close(struct tty_struct *tty)
>  {
>        struct hci_uart *hu = (void *)tty->disc_data;
>
> -       BT_DBG("tty %p", tty);
> +       BT_DBG("tty %p\n", tty);
>
>        /* Detach from the tty */
>        tty->disc_data = NULL;
> @@ -331,7 +335,7 @@ static void hci_uart_tty_wakeup(struct tty_struct *tty)
>  {
>        struct hci_uart *hu = (void *)tty->disc_data;
>
> -       BT_DBG("");
> +       BT_DBG("\n");
>
>        if (!hu)
>                return;
> @@ -379,7 +383,7 @@ static int hci_uart_register_dev(struct hci_uart *hu)
>  {
>        struct hci_dev *hdev;
>
> -       BT_DBG("");
> +       BT_DBG("\n");
>
>        /* Initialize and register HCI device */
>        hdev = hci_alloc_dev();
> @@ -456,7 +460,7 @@ static int hci_uart_tty_ioctl(struct tty_struct *tty, struct file * file,
>        struct hci_uart *hu = (void *)tty->disc_data;
>        int err = 0;
>
> -       BT_DBG("");
> +       BT_DBG("\n");
>
>        /* Verify the status of the device */
>        if (!hu)
> @@ -549,6 +553,9 @@ static int __init hci_uart_init(void)
>  #ifdef CONFIG_BT_HCIUART_LL
>        ll_init();
>  #endif
> +#ifdef CONFIG_BT_HCIUART_H5
> +       h5_init();
> +#endif
>
>        return 0;
>  }
> @@ -566,6 +573,9 @@ static void __exit hci_uart_exit(void)
>  #ifdef CONFIG_BT_HCIUART_LL
>        ll_deinit();
>  #endif
> +#ifdef CONFIG_BT_HCIUART_H5
> +       h5_deinit();
> +#endif
>
>        /* Release tty registration of line discipline */
>        if ((err = tty_unregister_ldisc(N_HCI)))
> diff --git a/drivers/bluetooth/hci_uart.h b/drivers/bluetooth/hci_uart.h
> index 50113db..0c40dab 100644
> --- a/drivers/bluetooth/hci_uart.h
> +++ b/drivers/bluetooth/hci_uart.h
> @@ -33,13 +33,14 @@
>  #define HCIUARTGETDEVICE       _IOR('U', 202, int)
>
>  /* UART protocols */
> -#define HCI_UART_MAX_PROTO     5
> +#define HCI_UART_MAX_PROTO     6
>
>  #define HCI_UART_H4    0
>  #define HCI_UART_BCSP  1
>  #define HCI_UART_3WIRE 2
>  #define HCI_UART_H4DS  3
>  #define HCI_UART_LL    4
> +#define HCI_UART_H5    5
>
>  struct hci_uart;
>
> @@ -91,3 +92,8 @@ int bcsp_deinit(void);
>  int ll_init(void);
>  int ll_deinit(void);
>  #endif
> +
> +#ifdef CONFIG_BT_HCIUART_H5
> +int h5_init(void);
> +int h5_deinit(void);
> +#endif
> --
> 1.7.1
>
>
> From d2d680c57f9e09cc574faf0c4153aacc007035bd Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Thu, 11 Aug 2011 10:55:02 -0700
> Subject: [PATCH 02/10] add flow control escape characters
>
> ---
>  drivers/bluetooth/hci_h5.c |   18 +++++++++---------
>  1 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index e2a1f69..a2a97cc 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -434,9 +434,10 @@ static void h5_handle_le_pkt(struct hci_uart *hu)
>        }
>  }
>
> +
>  static inline void h5_unslip_one_byte(struct h5_struct *h5, unsigned char byte)
>  {
> -       const u8 c0 = 0xc0, db = 0xdb;
> +       const u8 c0 = 0xc0, db = 0xdb, oneone = 0x11, onethree = 0x13;
>
>        switch (h5->rx_esc_state) {
>        case H5_ESCSTATE_NOESC:
> @@ -473,20 +474,20 @@ static inline void h5_unslip_one_byte(struct h5_struct *h5, unsigned char byte)
>                        h5->rx_count--;
>                        break;
>
> -               case 0xdf:
> -                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +               case 0xde:
> +                       memcpy(skb_put(h5->rx_skb, 1), &oneone, 1);
>                        if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
>                                        h5->rx_state != H5_W4_CRC)
> -                               h5_crc_update(&h5-> message_crc, 0x13);
> +                               h5_crc_update(&h5-> message_crc, 0x11);
>                        h5->rx_esc_state = H5_ESCSTATE_NOESC;
>                        h5->rx_count--;
>                        break;
>
> -               case 0xde:
> -                       memcpy(skb_put(h5->rx_skb, 1), &db, 1);
> +               case 0xdf:
> +                       memcpy(skb_put(h5->rx_skb, 1), &onethree, 1);
>                        if ((h5->rx_skb-> data[0] & 0x40) != 0 &&
>                                        h5->rx_state != H5_W4_CRC)
> -                               h5_crc_update(&h5-> message_crc, 0x11);
> +                               h5_crc_update(&h5-> message_crc, 0x13);
>                        h5->rx_esc_state = H5_ESCSTATE_NOESC;
>                        h5->rx_count--;
>                        break;
> @@ -577,7 +578,7 @@ static void h5_complete_rx_pkt(struct hci_uart *hu)
>
>  static u16 h5_get_crc(struct h5_struct *h5)
>  {
> -       return get_unaligned_be16(&h5->rx_skb->data[h5->rx_skb->len - 2]);
> +       return(get_unaligned_be16(&h5->rx_skb->data[h5->rx_skb->len - 2]));
>  }
>
>  /* Recv data */
> @@ -668,7 +669,6 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                                h5->rx_count = 0;
>                                continue;
>                        }
> -
>                        skb_trim(h5->rx_skb, h5->rx_skb->len - 2);
>                        h5_complete_rx_pkt(hu);
>                        h5->rx_state = H5_W4_PKT_DELIMITER;
> --
> 1.7.1
>
>
> From 07e40bafb21cc61c59dac90a308639f18487d00f Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Thu, 11 Aug 2011 11:12:03 -0700
> Subject: [PATCH 03/10] add flow control escape characters for tx
>
> ---
>  drivers/bluetooth/hci_h5.c |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index a2a97cc..0168269 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -139,6 +139,8 @@ static void h5_slip_one_byte(struct sk_buff *skb, u8 c)
>  {
>        const char esc_c0[2] = { 0xdb, 0xdc };
>        const char esc_db[2] = { 0xdb, 0xdd };
> +       const char esc_oneone[2] = { 0xdb, 0xde };
> +       const char esc_onethree[2] = { 0xdb, 0xdf };
>
>        switch (c) {
>        case 0xc0:
> @@ -147,6 +149,12 @@ static void h5_slip_one_byte(struct sk_buff *skb, u8 c)
>        case 0xdb:
>                memcpy(skb_put(skb, 2), &esc_db, 2);
>                break;
> +       case 0x11:
> +               memcpy(skb_put(skb, 2), &esc_oneone, 2);
> +               break;
> +       case 0x13:
> +               memcpy(skb_put(skb, 2), &esc_onethree, 2);
> +               break;
>        default:
>                memcpy(skb_put(skb, 1), &c, 1);
>        }
> --
> 1.7.1
>
>
> From c42f8bb513986054504e5c122a0f7597066dc15b Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Wed, 17 Aug 2011 15:12:17 -0700
> Subject: [PATCH 04/10] added sleep mode
>
> ---
>  drivers/bluetooth/hci_h5.c |  101 +++++++++++++++++++++++++++++++++++---------
>  1 files changed, 81 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index 0168269..265a9c5 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -61,6 +61,10 @@ static int h5extn = 1;
>  #define H5_ACK_PKT     0x00
>  #define H5_LE_PKT      0x0f
>
> +static u8 h5_wakeup_msg[] = { 0x05, 0xfa };
> +static u8 h5_woken_msg[] = { 0x06, 0xf9 };
> +static u8 h5_sleep_msg[] = { 0x07, 0x78 };
> +
>  struct h5_struct {
>        struct sk_buff_head unack;      /* Unack'ed packets queue */
>        struct sk_buff_head rel;        /* Reliable packets queue */
> @@ -92,8 +96,53 @@ struct h5_struct {
>
>        /* Reliable packet sequence number - used to assign seq to each rel pkt. */
>        u8      msgq_txseq;
> +
> +       u8 sleep_state;
> +       u8 is_there_activity;
> +       u16 inactive_period;
> +};
> +
> +enum sleep_states {
> +       H5_ASLEEP,
> +       H5_ASLEEP_TO_AWAKE,
> +       H5_AWAKE,
> +       H5_AWAKE_TO_ASLEEP
>  };
>
> +#define TIMER_PERIOD 100
> +#define HOST_CONTROLLER_IDLE_THRSH 4000
> +
> +static struct timer_list sleep_timer;
> +
> +extern struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> +               int len, int pkt_type);
> +
> +//
> +//     Timeout Handler
> +//
> +
> +void sleep_timer_function(unsigned long data)
> +{
> +       struct h5_struct *lh5 = (struct h5_struct *)data;
> +
> +       if (lh5->is_there_activity) {
> +               lh5->is_there_activity = 0;
> +               lh5->inactive_period = 0;
> +       } else if (lh5->sleep_state != H5_ASLEEP) {
> +               lh5->inactive_period += TIMER_PERIOD;
> +
> +               if (lh5->inactive_period >= HOST_CONTROLLER_IDLE_THRSH) {
> +                       BT_DBG("moves to ASLEEP");
> +                       lh5->sleep_state = H5_ASLEEP;
> +                       lh5->inactive_period = 0;
> +                       struct sk_buff *nskb = h5_prepare_pkt(lh5, h5_sleep_msg, sizeof(h5_sleep_msg), 15);
> +                       skb_queue_tail(&lh5->unrel, nskb);
> +               }
> +       }
> +
> +       mod_timer(&sleep_timer, jiffies + TIMER_PERIOD * HZ / 1000);
> +}
> +
>  /* ---- H5 CRC calculation ---- */
>
>  /* Table for calculating CRC for polynomial 0x1021, LSB processed first,
> @@ -170,6 +219,14 @@ static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
>                return 0;
>        }
>
> +       h5->is_there_activity = 1;
> +
> +       if (h5->sleep_state == H5_ASLEEP) {
> +               struct sk_buff *nskb = h5_prepare_pkt(h5, h5_wakeup_msg, sizeof(h5_wakeup_msg), 15);
> +               skb_queue_tail(&h5->unrel, skb);
> +               h5->sleep_state = H5_AWAKE;
> +       }
> +
>        switch (bt_cb(skb)->pkt_type) {
>        case HCI_ACLDATA_PKT:
>        case HCI_COMMAND_PKT:
> @@ -189,7 +246,7 @@ static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
>        return 0;
>  }
>
> -static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> +struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>                int len, int pkt_type)
>  {
>        struct sk_buff *nskb;
> @@ -357,6 +414,7 @@ static struct sk_buff *h5_dequeue(struct hci_uart *hu)
>  static int h5_flush(struct hci_uart *hu)
>  {
>        BT_DBG("hu %p\n", hu);
> +       del_timer_sync(&sleep_timer);
>        return 0;
>  }
>
> @@ -417,28 +475,22 @@ static void h5_pkt_cull(struct h5_struct *h5)
>  static void h5_handle_le_pkt(struct hci_uart *hu)
>  {
>        struct h5_struct *h5 = hu->priv;
> -       u8 conf_pkt[4]     = { 0xad, 0xef, 0xac, 0xed };
> -       u8 conf_rsp_pkt[4] = { 0xde, 0xad, 0xd0, 0xd0 };
> -       u8 sync_pkt[4]     = { 0x00, 0xd0, 0x01, 0x7e };
> -
> -       /* spot "conf" pkts and reply with a "conf rsp" pkt */
> -       if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> -                       !memcmp(&h5->rx_skb->data[4], conf_pkt, 4)) {
> -               struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> -
> -               BT_DBG("Found a LE conf pkt\n");
> -               if (!nskb)
> -                       return;
> -               memcpy(skb_put(nskb, 4), conf_rsp_pkt, 4);
> -               bt_cb(nskb)->pkt_type = H5_LE_PKT;
> +       u8 wakeup_pkt[]     = { 0x05, 0xfa };
> +       u8 woken_pkt[]     = { 0x06, 0xf9 };
> +
> +       struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> +
> +       BT_DBG("Found a LE pkt\n");
>
> +       if (!nskb)
> +               return;
> +
> +       if (!memcmp(&h5->rx_skb->data[4], wakeup_pkt, 2)) {
> +               memcpy(skb_put(nskb, 2), woken_pkt, 2);
> +               bt_cb(nskb)->pkt_type = H5_LE_PKT;
>                skb_queue_head(&h5->unrel, nskb);
>                hci_uart_tx_wakeup(hu);
> -       }
> -       /* Spot "sync" pkts. If we find one...disaster! */
> -       else if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> -                       !memcmp(&h5->rx_skb->data[4], sync_pkt, 4)) {
> -               BT_ERR("Found a LE sync pkt, card has reset");
> +       } else if (!memcmp(&h5->rx_skb->data[4], woken_pkt, 2)) {
>        }
>  }
>
> @@ -772,6 +824,15 @@ static int h5_open(struct hci_uart *hu)
>        if (h5txcrc)
>                h5->use_crc = 1;
>
> +       init_timer(&sleep_timer);
> +
> +       sleep_timer.expires = jiffies + TIMER_PERIOD * HZ / 1000;
> +       sleep_timer.data = (unsigned long)h5;
> +       sleep_timer.function = sleep_timer_function;
> +
> +       add_timer(&sleep_timer);
> +       h5->is_there_activity = 0;
> +
>        return 0;
>  }
>
> --
> 1.7.1
>
>
> From f9915f77dfbcb703ca098a7a0ace91fab3f686e8 Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Mon, 22 Aug 2011 11:42:10 -0700
> Subject: [PATCH 05/10] fixed seqno issues
>
> ---
>  drivers/bluetooth/hci_h5.c |  113 +++++++++++---------------------------------
>  1 files changed, 28 insertions(+), 85 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index 265a9c5..88ad715 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -61,10 +61,6 @@ static int h5extn = 1;
>  #define H5_ACK_PKT     0x00
>  #define H5_LE_PKT      0x0f
>
> -static u8 h5_wakeup_msg[] = { 0x05, 0xfa };
> -static u8 h5_woken_msg[] = { 0x06, 0xf9 };
> -static u8 h5_sleep_msg[] = { 0x07, 0x78 };
> -
>  struct h5_struct {
>        struct sk_buff_head unack;      /* Unack'ed packets queue */
>        struct sk_buff_head rel;        /* Reliable packets queue */
> @@ -96,53 +92,8 @@ struct h5_struct {
>
>        /* Reliable packet sequence number - used to assign seq to each rel pkt. */
>        u8      msgq_txseq;
> -
> -       u8 sleep_state;
> -       u8 is_there_activity;
> -       u16 inactive_period;
> -};
> -
> -enum sleep_states {
> -       H5_ASLEEP,
> -       H5_ASLEEP_TO_AWAKE,
> -       H5_AWAKE,
> -       H5_AWAKE_TO_ASLEEP
>  };
>
> -#define TIMER_PERIOD 100
> -#define HOST_CONTROLLER_IDLE_THRSH 4000
> -
> -static struct timer_list sleep_timer;
> -
> -extern struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> -               int len, int pkt_type);
> -
> -//
> -//     Timeout Handler
> -//
> -
> -void sleep_timer_function(unsigned long data)
> -{
> -       struct h5_struct *lh5 = (struct h5_struct *)data;
> -
> -       if (lh5->is_there_activity) {
> -               lh5->is_there_activity = 0;
> -               lh5->inactive_period = 0;
> -       } else if (lh5->sleep_state != H5_ASLEEP) {
> -               lh5->inactive_period += TIMER_PERIOD;
> -
> -               if (lh5->inactive_period >= HOST_CONTROLLER_IDLE_THRSH) {
> -                       BT_DBG("moves to ASLEEP");
> -                       lh5->sleep_state = H5_ASLEEP;
> -                       lh5->inactive_period = 0;
> -                       struct sk_buff *nskb = h5_prepare_pkt(lh5, h5_sleep_msg, sizeof(h5_sleep_msg), 15);
> -                       skb_queue_tail(&lh5->unrel, nskb);
> -               }
> -       }
> -
> -       mod_timer(&sleep_timer, jiffies + TIMER_PERIOD * HZ / 1000);
> -}
> -
>  /* ---- H5 CRC calculation ---- */
>
>  /* Table for calculating CRC for polynomial 0x1021, LSB processed first,
> @@ -219,14 +170,6 @@ static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
>                return 0;
>        }
>
> -       h5->is_there_activity = 1;
> -
> -       if (h5->sleep_state == H5_ASLEEP) {
> -               struct sk_buff *nskb = h5_prepare_pkt(h5, h5_wakeup_msg, sizeof(h5_wakeup_msg), 15);
> -               skb_queue_tail(&h5->unrel, skb);
> -               h5->sleep_state = H5_AWAKE;
> -       }
> -
>        switch (bt_cb(skb)->pkt_type) {
>        case HCI_ACLDATA_PKT:
>        case HCI_COMMAND_PKT:
> @@ -246,7 +189,7 @@ static int h5_enqueue(struct hci_uart *hu, struct sk_buff *skb)
>        return 0;
>  }
>
> -struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
> +static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>                int len, int pkt_type)
>  {
>        struct sk_buff *nskb;
> @@ -315,6 +258,7 @@ struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>        if (rel) {
>                hdr[0] |= 0x80 + h5->msgq_txseq;
>                BT_DBG("Sending packet with seqno %u\n", h5->msgq_txseq);
> +               h5->msgq_txseq = (h5->msgq_txseq + 1) & 0x07;
>        }
>
>        if (h5->use_crc)
> @@ -414,7 +358,6 @@ static struct sk_buff *h5_dequeue(struct hci_uart *hu)
>  static int h5_flush(struct hci_uart *hu)
>  {
>        BT_DBG("hu %p\n", hu);
> -       del_timer_sync(&sleep_timer);
>        return 0;
>  }
>
> @@ -430,20 +373,23 @@ static void h5_pkt_cull(struct h5_struct *h5)
>
>        pkts_to_be_removed = skb_queue_len(&h5->unack);
>
> +       printk("%s: unack len %d\n", pkts_to_be_removed);
> +
>        seqno = h5->msgq_txseq;
>
>        while (pkts_to_be_removed) {
> -               if (((h5->rxack - 1) & 0x07) == seqno)
> +               printk("ack %d seq %d\n", h5->rxack, seqno);
> +               if (h5->rxack == seqno)
>                        break;
>
>                pkts_to_be_removed--;
> -               seqno = (seqno + 1) & 0x07;
> +               seqno = (seqno - 1) & 0x07;
>        }
>
> -/*
> +       printk("seqno now %d\n", seqno);
> +
>        if (h5->rxack != seqno)
>                BT_ERR("Peer acked invalid packet");
> -*/
>
>        BT_DBG("Removing %u pkts out of %u, up to seqno %u\n",
>               pkts_to_be_removed, skb_queue_len(&h5->unack),
> @@ -475,22 +421,28 @@ static void h5_pkt_cull(struct h5_struct *h5)
>  static void h5_handle_le_pkt(struct hci_uart *hu)
>  {
>        struct h5_struct *h5 = hu->priv;
> -       u8 wakeup_pkt[]     = { 0x05, 0xfa };
> -       u8 woken_pkt[]     = { 0x06, 0xf9 };
> -
> -       struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> -
> -       BT_DBG("Found a LE pkt\n");
> -
> -       if (!nskb)
> -               return;
> -
> -       if (!memcmp(&h5->rx_skb->data[4], wakeup_pkt, 2)) {
> -               memcpy(skb_put(nskb, 2), woken_pkt, 2);
> +       u8 conf_pkt[4]     = { 0xad, 0xef, 0xac, 0xed };
> +       u8 conf_rsp_pkt[4] = { 0xde, 0xad, 0xd0, 0xd0 };
> +       u8 sync_pkt[4]     = { 0x00, 0xd0, 0x01, 0x7e };
> +
> +       /* spot "conf" pkts and reply with a "conf rsp" pkt */
> +       if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], conf_pkt, 4)) {
> +               struct sk_buff *nskb = alloc_skb(4, GFP_ATOMIC);
> +
> +               BT_DBG("Found a LE conf pkt\n");
> +               if (!nskb)
> +                       return;
> +               memcpy(skb_put(nskb, 4), conf_rsp_pkt, 4);
>                bt_cb(nskb)->pkt_type = H5_LE_PKT;
> +
>                skb_queue_head(&h5->unrel, nskb);
>                hci_uart_tx_wakeup(hu);
> -       } else if (!memcmp(&h5->rx_skb->data[4], woken_pkt, 2)) {
> +       }
> +       /* Spot "sync" pkts. If we find one...disaster! */
> +       else if (h5->rx_skb->data[1] >> 4 == 4 && h5->rx_skb->data[2] == 0 &&
> +                       !memcmp(&h5->rx_skb->data[4], sync_pkt, 4)) {
> +               BT_ERR("Found a LE sync pkt, card has reset");
>        }
>  }
>
> @@ -824,15 +776,6 @@ static int h5_open(struct hci_uart *hu)
>        if (h5txcrc)
>                h5->use_crc = 1;
>
> -       init_timer(&sleep_timer);
> -
> -       sleep_timer.expires = jiffies + TIMER_PERIOD * HZ / 1000;
> -       sleep_timer.data = (unsigned long)h5;
> -       sleep_timer.function = sleep_timer_function;
> -
> -       add_timer(&sleep_timer);
> -       h5->is_there_activity = 0;
> -
>        return 0;
>  }
>
> --
> 1.7.1
>
>
> From e0eae63d62db4365130df405bdc3ede6b153008b Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Mon, 12 Sep 2011 11:05:15 -0700
> Subject: [PATCH 06/10] send VSC over packet type 01 instead of packet type 14
>
> ---
>  drivers/bluetooth/hci_h5.c |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index 88ad715..a661ff3 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -197,6 +197,7 @@ static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>        u16 H5_CRC_INIT(h5_txmsg_crc);
>        int rel, i;
>
> +       printk("%s pkt_type %d\n", __func__, pkt_type);
>        print_hex_dump_bytes("h5_prepare_pkt ", DUMP_PREFIX_NONE, data, len);
>
>        switch (pkt_type) {
> @@ -225,11 +226,13 @@ static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>                return NULL;
>        }
>
> -       if (h5extn && chan == 14) {
> +       if (h5extn && chan == 1) {
>                __le16 opcode = ((struct hci_command_hdr *)data)->opcode;
>
>                /* Vendor specific commands */
>                if (hci_opcode_ogf(__le16_to_cpu(opcode)) == 0x3f) {
> +                       chan = 1;
> +
>                        u8 desc = *(data + HCI_COMMAND_HDR_SIZE);
>                        if ((desc & 0xf0) == 0xc0) {
>                                data += HCI_COMMAND_HDR_SIZE + 1;
> @@ -608,6 +611,8 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>        ptr = data;
>
>        while (count) {
> +               // printk("while count %d rx_count %d\n", count, h5->rx_count);
> +
>                if (h5->rx_count) {
>                        if (*ptr == 0xc0) {
>                                BT_ERR("Short H5 packet");
> @@ -621,6 +626,8 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                        continue;
>                }
>
> +               // printk("switch state %d\n", h5->rx_state);
> +
>                switch (h5->rx_state) {
>                case H5_W4_H5_HDR:
>                        if ((0xff & (u8) ~ (h5->rx_skb->data[0] + h5->rx_skb->data[1] +
> --
> 1.7.1
>
>
> From 428d90aef9bca3d50fe273f5f3d8864605bf5e3b Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Fri, 23 Sep 2011 13:01:30 -0700
> Subject: [PATCH 07/10] remove DEBUG for open source release
>
> ---
>  drivers/bluetooth/hci_h5.c |   49 +++++++++++++++++++++++++++++++++++---------
>  1 files changed, 39 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index a661ff3..7a40565 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -77,7 +77,6 @@ struct h5_struct {
>                H5_W4_PKT_START,
>                H5_W4_H5_HDR,
>                H5_W4_DATA,
> -               H5_W4_CRC1,
>                H5_W4_CRC
>        } rx_state;
>
> @@ -197,8 +196,10 @@ static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>        u16 H5_CRC_INIT(h5_txmsg_crc);
>        int rel, i;
>
> +#ifdef DEBUG
>        printk("%s pkt_type %d\n", __func__, pkt_type);
>        print_hex_dump_bytes("h5_prepare_pkt ", DUMP_PREFIX_NONE, data, len);
> +#endif
>
>        switch (pkt_type) {
>        case HCI_ACLDATA_PKT:
> @@ -376,12 +377,16 @@ static void h5_pkt_cull(struct h5_struct *h5)
>
>        pkts_to_be_removed = skb_queue_len(&h5->unack);
>
> +#ifdef DEBUG
>        printk("%s: unack len %d\n", pkts_to_be_removed);
> +#endif
>
>        seqno = h5->msgq_txseq;
>
>        while (pkts_to_be_removed) {
> +#ifdef DEBUG
>                printk("ack %d seq %d\n", h5->rxack, seqno);
> +#endif
>                if (h5->rxack == seqno)
>                        break;
>
> @@ -389,7 +394,9 @@ static void h5_pkt_cull(struct h5_struct *h5)
>                seqno = (seqno - 1) & 0x07;
>        }
>
> +#ifdef DEBUG
>        printk("seqno now %d\n", seqno);
> +#endif
>
>        if (h5->rxack != seqno)
>                BT_ERR("Peer acked invalid packet");
> @@ -529,7 +536,7 @@ static void h5_complete_rx_pkt(struct hci_uart *hu)
>                h5->txack_req    = 1;
>
>                /* If needed, transmit an ack pkt */
> -               hci_uart_tx_wakeup(hu);
> +               // hci_uart_tx_wakeup(hu); // wait till end of buffer
>        }
>
>        h5->rxack = (h5->rx_skb->data[0] >> 3) & 0x07;
> @@ -611,11 +618,17 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>        ptr = data;
>
>        while (count) {
> -               // printk("while count %d rx_count %d\n", count, h5->rx_count);
> +#ifdef DEBUG
> +               printk("while count %d rx_count %d\n", count, h5->rx_count);
> +#endif
>
>                if (h5->rx_count) {
>                        if (*ptr == 0xc0) {
>                                BT_ERR("Short H5 packet");
> +#ifdef DEBUG
> +                               printk("Short H5 %02x count %d rx_count %d state %d\n",
> +                                       *ptr, count, h5->rx_count, h5->rx_state);
> +#endif
>                                kfree_skb(h5->rx_skb);
>                                h5->rx_state = H5_W4_PKT_START;
>                                h5->rx_count = 0;
> @@ -626,7 +639,9 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                        continue;
>                }
>
> -               // printk("switch state %d\n", h5->rx_state);
> +#ifdef DEBUG
> +               printk("switch state %d\n", h5->rx_state);
> +#endif
>
>                switch (h5->rx_state) {
>                case H5_W4_H5_HDR:
> @@ -641,9 +656,11 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>
>
>                        if (h5->rx_skb->data[0] & 0x80) {       /* reliable pkt */
> +#ifdef DEBUG
>                                printk("%0x != %0x ? %d\n", h5->rx_skb->data[0] & 0x07,
>                                        h5->rxseq_txack,
>                                        (h5->rx_skb->data[0] & 0x07) != h5->rxseq_txack);
> +#endif
>
>                                if ((h5->rx_skb->data[0] & 0x07) != h5->rxseq_txack) {
>                                        BT_ERR ("Out-of-order packet arrived, got %u expected %u",
> @@ -672,11 +689,6 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>
>                        continue;
>
> -               case H5_W4_CRC1:
> -                       h5->rx_state = H5_W4_CRC;
> -                       count--;
> -                       continue;
> -
>                case H5_W4_CRC:
>                        if (bitrev16(h5->message_crc) != h5_get_crc(h5)) {
>                                BT_ERR ("Checksum failed: computed %04x received %04x",
> @@ -689,6 +701,7 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                                continue;
>                        }
>                        skb_trim(h5->rx_skb, h5->rx_skb->len - 2);
> +
>                        h5_complete_rx_pkt(hu);
>                        h5->rx_state = H5_W4_PKT_DELIMITER;
>                        count--;
> @@ -699,8 +712,13 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                        case 0xc0:
>                                h5->rx_state = H5_W4_PKT_START;
>                                break;
> +
> +                       case 0x11:
> +                               // hci_uart_tx_wakeup(hu);
> +                               break;
> +
>                        default:
> -                               /*BT_ERR("Ignoring byte %02x", *ptr);*/
> +                               BT_ERR("Ignoring byte %02x", *ptr);
>                                break;
>                        }
>                        ptr++; count--;
> @@ -712,6 +730,11 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                                ptr++; count--;
>                                break;
>
> +                       case 0x11:
> +                               // hci_uart_tx_wakeup(hu);
> +                               ptr++; count--;
> +                               break;
> +
>                        default:
>                                h5->rx_state = H5_W4_H5_HDR;
>                                h5->rx_count = 4;
> @@ -735,6 +758,12 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                        break;
>                }
>        }
> +
> +       if (h5->txack_req) {
> +               /* If needed, transmit an ack pkt */
> +               hci_uart_tx_wakeup(hu);
> +       }
> +
>        return count;
>  }
>
> --
> 1.7.1
>
>
> From c408f919e96cc8ec51956377b1ed1777b21f5885 Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Fri, 30 Sep 2011 13:42:25 -0700
> Subject: [PATCH 08/10] change license per Matt Sant
>
> ---
>  drivers/bluetooth/hci_h5.c |   18 ++++++++++++++++++
>  1 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index 7a40565..957dc3e 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -1,4 +1,22 @@
>  /*
> + *  Copyright 2011 Broadcom Corporation
> + *
> + *  This software is licensed to you under the terms of the GNU General
> + *  Public License version 2 (the "GPL"), available at
> + *  http://www.broadcom.com/licenses/GPLv2.php.
> + *
> + *  This software is based upon Bluetooth HCI UART driver, which was developed
> + *  by Fabrizio Gennari <fabrizio.gennari@xxxxxxxxxxx> (Copyright (C) 2002-2003)
> + *  and Marcel Holtmann <marcel@xxxxxxxxxxxx> (Copyright (C) 2004-2005).
> + *  The original Bluetooth HCI UART driver is available here: http://www.kernel.org
> + *
> + *  This software is distributed WITHOUT ANY WARRANTY; including the implied
> + *  warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> + *
> + *
> + */
> +
> +/*
>  *
>  *  Bluetooth HCI UART driver
>  *
> --
> 1.7.1
>
>
> From 08a6798ee23088a66ec01befece7ec08b7bac14f Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Mon, 3 Oct 2011 12:40:04 -0700
> Subject: [PATCH 09/10] fix count error on multiple packets in one buffer
>
> ---
>  drivers/bluetooth/hci_h5.c |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index 957dc3e..d629d2b 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -637,7 +637,8 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>
>        while (count) {
>  #ifdef DEBUG
> -               printk("while count %d rx_count %d\n", count, h5->rx_count);
> +               printk("while count %d rx_count %d ptr %02x state %d\n", count,
> +                       h5->rx_count, *ptr, h5->rx_state);
>  #endif
>
>                if (h5->rx_count) {
> @@ -722,7 +723,6 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>
>                        h5_complete_rx_pkt(hu);
>                        h5->rx_state = H5_W4_PKT_DELIMITER;
> -                       count--;
>                        continue;
>
>                case H5_W4_PKT_DELIMITER:
> @@ -777,6 +777,9 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                }
>        }
>
> +       printk("leaving h5_recv rx_state %d rx_count %d ptr %02x\n", h5->rx_state,
> +               h5->rx_count, *ptr);
> +
>        if (h5->txack_req) {
>                /* If needed, transmit an ack pkt */
>                hci_uart_tx_wakeup(hu);
> --
> 1.7.1
>
>
> From 2658d101fd3925ed293b30b8f0910561ba12d21a Mon Sep 17 00:00:00 2001
> From: Mark P. Mendelsohn <mpm>
> Date: Mon, 24 Oct 2011 09:57:04 -0700
> Subject: [PATCH 10/10] conditional compile debug statements
>
> ---
>  drivers/bluetooth/hci_h5.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index d629d2b..2effe06 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -317,7 +317,10 @@ static struct sk_buff *h5_prepare_pkt(struct h5_struct *h5, u8 *data,
>
>        h5_slip_msgdelim(nskb);
>
> +#ifdef DEBUG
>        print_hex_dump_bytes("h5_tx ", DUMP_PREFIX_NONE, nskb->data, nskb->len);
> +#endif
> +
>        return nskb;
>  }
>
> @@ -631,7 +634,9 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>        BT_DBG("hu %p count %d rx_state %d rx_count %ld\n",
>                hu, count, h5->rx_state, h5->rx_count);
>
> +#ifdef DEBUG
>        print_hex_dump_bytes("h5_recv ", DUMP_PREFIX_NONE, data, count);
> +#endif
>
>        ptr = data;
>
> @@ -777,8 +782,10 @@ static int h5_recv(struct hci_uart *hu, void *data, int count)
>                }
>        }
>
> +#ifdef DEBUG
>        printk("leaving h5_recv rx_state %d rx_count %d ptr %02x\n", h5->rx_state,
>                h5->rx_count, *ptr);
> +#endif
>
>        if (h5->txack_req) {
>                /* If needed, transmit an ack pkt */
> --
> 1.7.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/