Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1302820 > unrolled thread

[PATCH 1/2] Bluetooth: btmrvl: max out host sleep parameter 'gap'

Started byAmitkumar Karwar <akarwar@marvell.com>
First post2016-01-06 15:40 +0100
Last post2016-01-06 16:30 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/2] Bluetooth: btmrvl: max out host sleep parameter 'gap' Amitkumar Karwar <akarwar@marvell.com> - 2016-01-06 15:40 +0100
    Re: [PATCH 1/2] Bluetooth: btmrvl: max out host sleep parameter 'gap' Marcel Holtmann <marcel@holtmann.org> - 2016-01-06 16:30 +0100

#1302820 — [PATCH 1/2] Bluetooth: btmrvl: max out host sleep parameter 'gap'

FromAmitkumar Karwar <akarwar@marvell.com>
Date2016-01-06 15:40 +0100
Subject[PATCH 1/2] Bluetooth: btmrvl: max out host sleep parameter 'gap'
Message-ID<qNXpM-2lM-29@gated-at.bofh.it>
From: Chin-Ran Lo <crlo@marvell.com>

For gpio=0xff (wake up host through SDIO interface) case,
gap=0xff means no delay (same as gap=0) for incoming data packet
to be sent to host after host sleep is activated.
Change it to the maximum delay to reduce the chance that RX
interrupt could be delivered while host controller suspends.

Signed-off-by: Chin-Ran Lo <crlo@marvell.com>
Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
---
 drivers/bluetooth/btmrvl_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/bluetooth/btmrvl_main.c b/drivers/bluetooth/btmrvl_main.c
index f2b38c8..b2a567b 100644
--- a/drivers/bluetooth/btmrvl_main.c
+++ b/drivers/bluetooth/btmrvl_main.c
@@ -543,7 +543,7 @@ static int btmrvl_setup(struct hci_dev *hdev)
 	if (ret)
 		return ret;
 
-	priv->btmrvl_dev.gpio_gap = 0xffff;
+	priv->btmrvl_dev.gpio_gap = 0xfffe;
 
 	btmrvl_check_device_tree(priv);
 
-- 
1.8.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1302850

FromMarcel Holtmann <marcel@holtmann.org>
Date2016-01-06 16:30 +0100
Message-ID<qNYca-2Y2-23@gated-at.bofh.it>
In reply to#1302820
Hi Amitkumar,

> For gpio=0xff (wake up host through SDIO interface) case,
> gap=0xff means no delay (same as gap=0) for incoming data packet
> to be sent to host after host sleep is activated.
> Change it to the maximum delay to reduce the chance that RX
> interrupt could be delivered while host controller suspends.
> 
> Signed-off-by: Chin-Ran Lo <crlo@marvell.com>
> Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> ---
> drivers/bluetooth/btmrvl_main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web