All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Bluetooth: hci_qca: resolve various warnings
@ 2020-11-01  2:52 ` Nigel Christian
  0 siblings, 0 replies; 6+ messages in thread
From: Nigel Christian @ 2020-11-01  2:52 UTC (permalink / raw)
  To: marcel, johan.hedberg; +Cc: kernel-janitors, linux-bluetooth

Use template string as opposed to function name. Add a blank
line after declaration to improve readability. Replace symbolic
permissions with octal values. Use usleep_range for small msec
values due to the fact that msleep() less than 20ms may have
unexpected behavior/sleep longer. This patch removes all
checkpatch warnings.

- https://lkml.org/lkml/2016/8/2/1945
- Documentation/timers/timers-howto.rst

Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
---
 drivers/bluetooth/hci_qca.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index 2d3f1f179a1e..22da59d8f471 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
 	struct qca_serdev *qcadev;
 	struct qca_data *qca;
 
-	BT_DBG("hu %p qca_open", hu);
+	BT_DBG("hu %p %s", hu, __func__);
 
 	if (!hci_uart_has_flow_control(hu))
 		return -EOPNOTSUPP;
@@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
 
 	/* read only */
-	mode = S_IRUGO;
+	mode = 0444;
 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
@@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
 
 	/* read/write */
-	mode = S_IRUGO | S_IWUSR;
+	mode = 0644;
 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
 			   &qca->tx_idle_delay);
@@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
 	if (IS_ERR(qca->rx_skb)) {
 		int err = PTR_ERR(qca->rx_skb);
+
 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
 		qca->rx_skb = NULL;
 		return err;
@@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
 
 	/* Give the controller time to process the request */
 	if (qca_is_wcn399x(qca_soc_type(hu)))
-		msleep(10);
+		usleep_range(1000, 10000);
 	else
 		msleep(300);
 
@@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
 	if (on)
 		msleep(100);
 	else
-		msleep(10);
+		usleep_range(1000, 10000);
 
 	return 0;
 }
-- 
2.28.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] Bluetooth: hci_qca: resolve various warnings
@ 2020-11-01  2:52 ` Nigel Christian
  0 siblings, 0 replies; 6+ messages in thread
From: Nigel Christian @ 2020-11-01  2:52 UTC (permalink / raw)
  To: marcel, johan.hedberg; +Cc: kernel-janitors, linux-bluetooth

Use template string as opposed to function name. Add a blank
line after declaration to improve readability. Replace symbolic
permissions with octal values. Use usleep_range for small msec
values due to the fact that msleep() less than 20ms may have
unexpected behavior/sleep longer. This patch removes all
checkpatch warnings.

- https://lkml.org/lkml/2016/8/2/1945
- Documentation/timers/timers-howto.rst

Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
---
 drivers/bluetooth/hci_qca.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
index 2d3f1f179a1e..22da59d8f471 100644
--- a/drivers/bluetooth/hci_qca.c
+++ b/drivers/bluetooth/hci_qca.c
@@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
 	struct qca_serdev *qcadev;
 	struct qca_data *qca;
 
-	BT_DBG("hu %p qca_open", hu);
+	BT_DBG("hu %p %s", hu, __func__);
 
 	if (!hci_uart_has_flow_control(hu))
 		return -EOPNOTSUPP;
@@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
 
 	/* read only */
-	mode = S_IRUGO;
+	mode = 0444;
 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
@@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
 
 	/* read/write */
-	mode = S_IRUGO | S_IWUSR;
+	mode = 0644;
 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
 			   &qca->tx_idle_delay);
@@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
 	if (IS_ERR(qca->rx_skb)) {
 		int err = PTR_ERR(qca->rx_skb);
+
 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
 		qca->rx_skb = NULL;
 		return err;
@@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
 
 	/* Give the controller time to process the request */
 	if (qca_is_wcn399x(qca_soc_type(hu)))
-		msleep(10);
+		usleep_range(1000, 10000);
 	else
 		msleep(300);
 
@@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
 	if (on)
 		msleep(100);
 	else
-		msleep(10);
+		usleep_range(1000, 10000);
 
 	return 0;
 }
-- 
2.28.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH] Bluetooth: hci_qca: resolve various warnings
  2020-11-01  2:52 ` Nigel Christian
@ 2020-11-09 12:45   ` Marcel Holtmann
  -1 siblings, 0 replies; 6+ messages in thread
From: Marcel Holtmann @ 2020-11-09 12:45 UTC (permalink / raw)
  To: Nigel Christian; +Cc: Johan Hedberg, kernel-janitors, linux-bluetooth

Hi Nigel,

> Use template string as opposed to function name. Add a blank
> line after declaration to improve readability. Replace symbolic
> permissions with octal values. Use usleep_range for small msec
> values due to the fact that msleep() less than 20ms may have
> unexpected behavior/sleep longer. This patch removes all
> checkpatch warnings.
> 
> - https://lkml.org/lkml/2016/8/2/1945
> - Documentation/timers/timers-howto.rst
> 
> Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
> ---
> drivers/bluetooth/hci_qca.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
> index 2d3f1f179a1e..22da59d8f471 100644
> --- a/drivers/bluetooth/hci_qca.c
> +++ b/drivers/bluetooth/hci_qca.c
> @@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
> 	struct qca_serdev *qcadev;
> 	struct qca_data *qca;
> 
> -	BT_DBG("hu %p qca_open", hu);
> +	BT_DBG("hu %p %s", hu, __func__);

please clean up your patch before submitting it. This change does not belong
here.

> 	if (!hci_uart_has_flow_control(hu))
> 		return -EOPNOTSUPP;
> @@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
> 
> 	/* read only */
> -	mode = S_IRUGO;
> +	mode = 0444;
> 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
> 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
> 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
> @@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
> 
> 	/* read/write */
> -	mode = S_IRUGO | S_IWUSR;
> +	mode = 0644;
> 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
> 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
> 			   &qca->tx_idle_delay);
> @@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
> 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
> 	if (IS_ERR(qca->rx_skb)) {
> 		int err = PTR_ERR(qca->rx_skb);
> +

This omitted empty line was intentional btw.

> 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
> 		qca->rx_skb = NULL;
> 		return err;
> @@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
> 
> 	/* Give the controller time to process the request */
> 	if (qca_is_wcn399x(qca_soc_type(hu)))
> -		msleep(10);
> +		usleep_range(1000, 10000);
> 	else
> 		msleep(300);
> 
> @@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
> 	if (on)
> 		msleep(100);
> 	else
> -		msleep(10);
> +		usleep_range(1000, 10000);

Regards

Marcel


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] Bluetooth: hci_qca: resolve various warnings
@ 2020-11-09 12:45   ` Marcel Holtmann
  0 siblings, 0 replies; 6+ messages in thread
From: Marcel Holtmann @ 2020-11-09 12:45 UTC (permalink / raw)
  To: Nigel Christian; +Cc: Johan Hedberg, kernel-janitors, linux-bluetooth

Hi Nigel,

> Use template string as opposed to function name. Add a blank
> line after declaration to improve readability. Replace symbolic
> permissions with octal values. Use usleep_range for small msec
> values due to the fact that msleep() less than 20ms may have
> unexpected behavior/sleep longer. This patch removes all
> checkpatch warnings.
> 
> - https://lkml.org/lkml/2016/8/2/1945
> - Documentation/timers/timers-howto.rst
> 
> Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
> ---
> drivers/bluetooth/hci_qca.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
> index 2d3f1f179a1e..22da59d8f471 100644
> --- a/drivers/bluetooth/hci_qca.c
> +++ b/drivers/bluetooth/hci_qca.c
> @@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
> 	struct qca_serdev *qcadev;
> 	struct qca_data *qca;
> 
> -	BT_DBG("hu %p qca_open", hu);
> +	BT_DBG("hu %p %s", hu, __func__);

please clean up your patch before submitting it. This change does not belong
here.

> 	if (!hci_uart_has_flow_control(hu))
> 		return -EOPNOTSUPP;
> @@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
> 
> 	/* read only */
> -	mode = S_IRUGO;
> +	mode = 0444;
> 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
> 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
> 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
> @@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
> 
> 	/* read/write */
> -	mode = S_IRUGO | S_IWUSR;
> +	mode = 0644;
> 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
> 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
> 			   &qca->tx_idle_delay);
> @@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
> 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
> 	if (IS_ERR(qca->rx_skb)) {
> 		int err = PTR_ERR(qca->rx_skb);
> +

This omitted empty line was intentional btw.

> 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
> 		qca->rx_skb = NULL;
> 		return err;
> @@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
> 
> 	/* Give the controller time to process the request */
> 	if (qca_is_wcn399x(qca_soc_type(hu)))
> -		msleep(10);
> +		usleep_range(1000, 10000);
> 	else
> 		msleep(300);
> 
> @@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
> 	if (on)
> 		msleep(100);
> 	else
> -		msleep(10);
> +		usleep_range(1000, 10000);

Regards

Marcel

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] Bluetooth: hci_qca: resolve various warnings
  2020-11-09 12:45   ` Marcel Holtmann
@ 2020-11-10 13:25     ` Nigel Christian
  -1 siblings, 0 replies; 6+ messages in thread
From: Nigel Christian @ 2020-11-10 13:25 UTC (permalink / raw)
  To: Marcel Holtmann; +Cc: johan.hedberg, kernel-janitors, linux-bluetooth

On Mon, Nov 09, 2020 at 01:45:34PM +0100, Marcel Holtmann wrote:
> Hi Nigel,
> 
> > Use template string as opposed to function name. Add a blank
> > line after declaration to improve readability. Replace symbolic
> > permissions with octal values. Use usleep_range for small msec
> > values due to the fact that msleep() less than 20ms may have
> > unexpected behavior/sleep longer. This patch removes all
> > checkpatch warnings.
> > 
> > - https://lkml.org/lkml/2016/8/2/1945
> > - Documentation/timers/timers-howto.rst
> > 
> > Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
> > ---
> > drivers/bluetooth/hci_qca.c | 11 ++++++-----
> > 1 file changed, 6 insertions(+), 5 deletions(-)
> > 
> > diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
> > index 2d3f1f179a1e..22da59d8f471 100644
> > --- a/drivers/bluetooth/hci_qca.c
> > +++ b/drivers/bluetooth/hci_qca.c
> > @@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
> > 	struct qca_serdev *qcadev;
> > 	struct qca_data *qca;
> > 
> > -	BT_DBG("hu %p qca_open", hu);
> > +	BT_DBG("hu %p %s", hu, __func__);
> 
> please clean up your patch before submitting it. This change does not belong
> here.
> 
> > 	if (!hci_uart_has_flow_control(hu))
> > 		return -EOPNOTSUPP;
> > @@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> > 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
> > 
> > 	/* read only */
> > -	mode = S_IRUGO;
> > +	mode = 0444;
> > 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
> > 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
> > 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
> > @@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> > 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
> > 
> > 	/* read/write */
> > -	mode = S_IRUGO | S_IWUSR;
> > +	mode = 0644;
> > 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
> > 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
> > 			   &qca->tx_idle_delay);
> > @@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
> > 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
> > 	if (IS_ERR(qca->rx_skb)) {
> > 		int err = PTR_ERR(qca->rx_skb);
> > +
> 
> This omitted empty line was intentional btw.
> 
> > 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
> > 		qca->rx_skb = NULL;
> > 		return err;
> > @@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
> > 
> > 	/* Give the controller time to process the request */
> > 	if (qca_is_wcn399x(qca_soc_type(hu)))
> > -		msleep(10);
> > +		usleep_range(1000, 10000);
> > 	else
> > 		msleep(300);
> > 
> > @@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
> > 	if (on)
> > 		msleep(100);
> > 	else
> > -		msleep(10);
> > +		usleep_range(1000, 10000);
> 
> Regards
> 
> Marcel
> 

Sure, will clean up and resend. Thanks for checking on this!

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] Bluetooth: hci_qca: resolve various warnings
@ 2020-11-10 13:25     ` Nigel Christian
  0 siblings, 0 replies; 6+ messages in thread
From: Nigel Christian @ 2020-11-10 13:25 UTC (permalink / raw)
  To: Marcel Holtmann; +Cc: johan.hedberg, kernel-janitors, linux-bluetooth

On Mon, Nov 09, 2020 at 01:45:34PM +0100, Marcel Holtmann wrote:
> Hi Nigel,
> 
> > Use template string as opposed to function name. Add a blank
> > line after declaration to improve readability. Replace symbolic
> > permissions with octal values. Use usleep_range for small msec
> > values due to the fact that msleep() less than 20ms may have
> > unexpected behavior/sleep longer. This patch removes all
> > checkpatch warnings.
> > 
> > - https://lkml.org/lkml/2016/8/2/1945
> > - Documentation/timers/timers-howto.rst
> > 
> > Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com>
> > ---
> > drivers/bluetooth/hci_qca.c | 11 ++++++-----
> > 1 file changed, 6 insertions(+), 5 deletions(-)
> > 
> > diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c
> > index 2d3f1f179a1e..22da59d8f471 100644
> > --- a/drivers/bluetooth/hci_qca.c
> > +++ b/drivers/bluetooth/hci_qca.c
> > @@ -555,7 +555,7 @@ static int qca_open(struct hci_uart *hu)
> > 	struct qca_serdev *qcadev;
> > 	struct qca_data *qca;
> > 
> > -	BT_DBG("hu %p qca_open", hu);
> > +	BT_DBG("hu %p %s", hu, __func__);
> 
> please clean up your patch before submitting it. This change does not belong
> here.
> 
> > 	if (!hci_uart_has_flow_control(hu))
> > 		return -EOPNOTSUPP;
> > @@ -631,7 +631,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> > 	ibs_dir = debugfs_create_dir("ibs", hdev->debugfs);
> > 
> > 	/* read only */
> > -	mode = S_IRUGO;
> > +	mode = 0444;
> > 	debugfs_create_u8("tx_ibs_state", mode, ibs_dir, &qca->tx_ibs_state);
> > 	debugfs_create_u8("rx_ibs_state", mode, ibs_dir, &qca->rx_ibs_state);
> > 	debugfs_create_u64("ibs_sent_sleeps", mode, ibs_dir,
> > @@ -658,7 +658,7 @@ static void qca_debugfs_init(struct hci_dev *hdev)
> > 	debugfs_create_u32("vote_off_ms", mode, ibs_dir, &qca->vote_off_ms);
> > 
> > 	/* read/write */
> > -	mode = S_IRUGO | S_IWUSR;
> > +	mode = 0644;
> > 	debugfs_create_u32("wake_retrans", mode, ibs_dir, &qca->wake_retrans);
> > 	debugfs_create_u32("tx_idle_delay", mode, ibs_dir,
> > 			   &qca->tx_idle_delay);
> > @@ -1215,6 +1215,7 @@ static int qca_recv(struct hci_uart *hu, const void *data, int count)
> > 				  qca_recv_pkts, ARRAY_SIZE(qca_recv_pkts));
> > 	if (IS_ERR(qca->rx_skb)) {
> > 		int err = PTR_ERR(qca->rx_skb);
> > +
> 
> This omitted empty line was intentional btw.
> 
> > 		bt_dev_err(hu->hdev, "Frame reassembly failed (%d)", err);
> > 		qca->rx_skb = NULL;
> > 		return err;
> > @@ -1302,7 +1303,7 @@ static int qca_set_baudrate(struct hci_dev *hdev, uint8_t baudrate)
> > 
> > 	/* Give the controller time to process the request */
> > 	if (qca_is_wcn399x(qca_soc_type(hu)))
> > -		msleep(10);
> > +		usleep_range(1000, 10000);
> > 	else
> > 		msleep(300);
> > 
> > @@ -1350,7 +1351,7 @@ static int qca_send_power_pulse(struct hci_uart *hu, bool on)
> > 	if (on)
> > 		msleep(100);
> > 	else
> > -		msleep(10);
> > +		usleep_range(1000, 10000);
> 
> Regards
> 
> Marcel
> 

Sure, will clean up and resend. Thanks for checking on this!

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-11-10 13:25 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-01  2:52 [PATCH] Bluetooth: hci_qca: resolve various warnings Nigel Christian
2020-11-01  2:52 ` Nigel Christian
2020-11-09 12:45 ` Marcel Holtmann
2020-11-09 12:45   ` Marcel Holtmann
2020-11-10 13:25   ` Nigel Christian
2020-11-10 13:25     ` Nigel Christian

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.