[SRU][Bionic0[PATCH 0/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[SRU][Bionic0[PATCH 0/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table

Joseph Salisbury-3
BugLink: http://bugs.launchpad.net/bugs/1775217

== SRU Justification ==
This is a regression introduced in v4.15 and Bionic.  The bluetooth
controller fails after suspend with USB autosuspend on an XPS 13 9360.

This regression is fixed by mainline commit 596b07a9a226.  This commit
was also cc'd to upstream stable, but it has not landed in Bionic via
stable updates as of yet.

== Fix ==
596b07a9a226 ("Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table")

== Regression Potential ==
Low.  This patch has also been cc'd to upstream stable, so it recieved
additonal upstream review.

== Test Case ==
A test kernel was built with this patch and tested by the original bug reporter.
The bug reporter states the test kernel resolved the bug.


Hans de Goede (1):
  Bluetooth: btusb: Add Dell XPS 13 9360 to
    btusb_needs_reset_resume_table

 drivers/bluetooth/btusb.c | 7 +++++++
 1 file changed, 7 insertions(+)

--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[SRU][Bionic0[PATCH 1/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table

Joseph Salisbury-3
From: Hans de Goede <[hidden email]>

BugLink: http://bugs.launchpad.net/bugs/1775217

The Dell XPS 13 9360 uses a QCA Rome chip which needs to be reset
(and have its firmware reloaded) for bluetooth to work after
suspend/resume.

BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1514836
Cc: [hidden email]
Cc: Garrett LeSage <[hidden email]>
Reported-and-tested-by: Garrett LeSage <[hidden email]>
Signed-off-by: Hans de Goede <[hidden email]>
Signed-off-by: Marcel Holtmann <[hidden email]>
(cherry picked from commit 596b07a9a22656493726edf1739569102bd3e136)
Signed-off-by: Joseph Salisbury <[hidden email]>
---
 drivers/bluetooth/btusb.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index e6e66dd..3b30224 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -391,6 +391,13 @@ static const struct dmi_system_id btusb_needs_reset_resume_table[] = {
  DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 3060"),
  },
  },
+ {
+ /* Dell XPS 9360 (QCA ROME device 0cf3:e300) */
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9360"),
+ },
+ },
  {}
 };
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [[SCRIPT=remove_re|Re: [SRU][Bionic0[PATCH 1/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table]]

Stefan Bader-2
On 08.06.2018 10:56, Joseph Salisbury wrote:

> From: Hans de Goede <[hidden email]>
>
> BugLink: http://bugs.launchpad.net/bugs/1775217
>
> The Dell XPS 13 9360 uses a QCA Rome chip which needs to be reset
> (and have its firmware reloaded) for bluetooth to work after
> suspend/resume.
>
> BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1514836
> Cc: [hidden email]
> Cc: Garrett LeSage <[hidden email]>
> Reported-and-tested-by: Garrett LeSage <[hidden email]>
> Signed-off-by: Hans de Goede <[hidden email]>
> Signed-off-by: Marcel Holtmann <[hidden email]>
> (cherry picked from commit 596b07a9a22656493726edf1739569102bd3e136)
> Signed-off-by: Joseph Salisbury <[hidden email]>
Acked-by: Stefan Bader <[hidden email]>

> ---
>  drivers/bluetooth/btusb.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index e6e66dd..3b30224 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -391,6 +391,13 @@ static const struct dmi_system_id btusb_needs_reset_resume_table[] = {
>   DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 3060"),
>   },
>   },
> + {
> + /* Dell XPS 9360 (QCA ROME device 0cf3:e300) */
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9360"),
> + },
> + },
>   {}
>  };
>  
>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

ACK: [SRU][Bionic0[PATCH 0/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table

Khaled Elmously
In reply to this post by Joseph Salisbury-3
On 2018-06-08 13:56:23 , Joseph Salisbury wrote:

> BugLink: http://bugs.launchpad.net/bugs/1775217
>
> == SRU Justification ==
> This is a regression introduced in v4.15 and Bionic.  The bluetooth
> controller fails after suspend with USB autosuspend on an XPS 13 9360.
>
> This regression is fixed by mainline commit 596b07a9a226.  This commit
> was also cc'd to upstream stable, but it has not landed in Bionic via
> stable updates as of yet.
>
> == Fix ==
> 596b07a9a226 ("Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table")
>
> == Regression Potential ==
> Low.  This patch has also been cc'd to upstream stable, so it recieved
> additonal upstream review.
>
> == Test Case ==
> A test kernel was built with this patch and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
>
>
> Hans de Goede (1):
>   Bluetooth: btusb: Add Dell XPS 13 9360 to
>     btusb_needs_reset_resume_table
>
>  drivers/bluetooth/btusb.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
Acked-by: Khalid Elmously <[hidden email]>
 

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED: [SRU][Bionic0[PATCH 0/1] Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table

Khaled Elmously
In reply to this post by Joseph Salisbury-3
Applied to Bionic

On 2018-06-08 13:56:23 , Joseph Salisbury wrote:

> BugLink: http://bugs.launchpad.net/bugs/1775217
>
> == SRU Justification ==
> This is a regression introduced in v4.15 and Bionic.  The bluetooth
> controller fails after suspend with USB autosuspend on an XPS 13 9360.
>
> This regression is fixed by mainline commit 596b07a9a226.  This commit
> was also cc'd to upstream stable, but it has not landed in Bionic via
> stable updates as of yet.
>
> == Fix ==
> 596b07a9a226 ("Bluetooth: btusb: Add Dell XPS 13 9360 to btusb_needs_reset_resume_table")
>
> == Regression Potential ==
> Low.  This patch has also been cc'd to upstream stable, so it recieved
> additonal upstream review.
>
> == Test Case ==
> A test kernel was built with this patch and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
>
>
> Hans de Goede (1):
>   Bluetooth: btusb: Add Dell XPS 13 9360 to
>     btusb_needs_reset_resume_table
>
>  drivers/bluetooth/btusb.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> --
> 2.7.4
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team