public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: Johan Hedberg <johan.hedberg@gmail.com>,
	Linux Bluetooth mailing list <linux-bluetooth@vger.kernel.org>,
	linux-kernel@vger.kernel.org,
	Ivan Levshin <ivan.levshin@microfocus.com>
Subject: Re: Atheros 1525/QCA6174 BT issue
Date: Tue, 13 Mar 2018 09:50:41 +0100	[thread overview]
Message-ID: <s5ha7vcfjfi.wl-tiwai@suse.de> (raw)
In-Reply-To: <D8E4DA1B-060E-45F4-9B26-E3FAEE652D03@holtmann.org>

On Tue, 13 Mar 2018 09:38:30 +0100,
Marcel Holtmann wrote:
> 
> Hi Takashi,
> 
> >>>>>> we've got a but report about the broken Atheros BT on the recent
> >>>>>> kernels:
> >>>>>> http://bugzilla.opensuse.org/show_bug.cgi?id=1082504
> >>>>>> 
> >>>>>> In short, btusb can't load the patch ar3k/AthrBT_0x00000200.dfu, and
> >>>>>> this could be worked around by the patch to move 0cf3:3004 blacklist
> >>>>>> entry to use BTUSB_QCA_ROM instead of BTUSB_ATH3012.
> >>>>>> 
> >>>>>> And this looks like a long-standing problem, at least for over two
> >>>>>> years.  Many web pages suggest the same patch, but it's never merged
> >>>>>> to upstream.
> >>>>>> 
> >>>>>> So this made me wonder what's going on.  I see that the BTUSB_ATH3012
> >>>>>> quirk was originally introduced just for this chip id (0cf3:3004).
> >>>>>> Is it a different variant from the original chip that causes a
> >>>>>> problem?
> >>>>> 
> >>>>> not all patches from distro kernel are sent upstream. I have not heard of this specific issues, but happy to accept patches to get it fixed.
> >>>> 
> >>>> OK, basically it's like below.
> >>>> But, as mentioned, this made me wonder whether it's the right fix.
> >>>> The BTUSB_ATH3012 quirk was introduced exactly for this chip ID
> >>>> (0cf3:3004), and now this chip is moved to another quirk...
> >>>> 
> >>>> If this is the right move, I can re-submit via git-send-email, too.
> >>>> Just let me know.
> >>> 
> >>> Marcel, could you take a look at this?
> >>> If it sucks, let's seek for a better solution.
> >> 
> >> wasn’t the confusion that this is fixed with a recent kernel? I am lost in this thread. I mean if people add Tested-by, then I can take this as well. Otherwise we might need someone from Qualcomm to shed some light into these.
> > 
> > Well, *this* thread is likely different from the recent other
> > threads.
> > 
> > Isn't 4.15.7 recent enough?  At least, it already contains the
> > backport of relevant fixes:
> >    Revert "Bluetooth: btusb: fix QCA Rome suspend/resume"
> >    Bluetooth: btusb: Restore QCA Rome suspend/resume fix with a
> >      "rewritten" version
> > 
> > (And it's not Yoga but MSI GS40 laptop, so DMI doesn't matter.)
> > According to Ivan, the reporter of the bug (now Cc'ed), 4.15.7 didn't
> > work without the patch, so the problem is still there, as it seems.
> > 
> > In anyway, I'm going to build a kernel with my patch on top of 4.15.9
> > for testing again.  Maybe also a patched 4.16-rc5 kernel, too.  If
> > it's confirmed, will report back with tested-by tag.
> 
> I think there are two patches that are not yet in Linus’ tree and waiting in Dave’s net tree. We actually removed the Yoga DMI entry again since it was found that it is not needed. However there is a Dell OptiPlex entry that was needed.
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git/commit/?id=0c6e526646c04ce31d4aaa280ed2237dd1cd774c

In our case, the target machine is a MSI laptop, so these changes
should be irrelevant.  Or do you suggest to try the same DMI reset
quirk matching with the MSI machine?


thanks,

Takashi

  reply	other threads:[~2018-03-13  8:50 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-08  8:27 Atheros 1525/QCA6174 BT issue Takashi Iwai
2018-03-08  9:06 ` Marcel Holtmann
2018-03-08  9:16   ` Takashi Iwai
2018-03-13  8:05     ` Takashi Iwai
2018-03-13  8:10       ` Marcel Holtmann
2018-03-13  8:32         ` Takashi Iwai
2018-03-13  8:38           ` Marcel Holtmann
2018-03-13  8:50             ` Takashi Iwai [this message]
2018-03-13  9:15               ` Marcel Holtmann
2018-03-15 15:56                 ` Takashi Iwai
2018-03-15 18:32                   ` Marcel Holtmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5ha7vcfjfi.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=ivan.levshin@microfocus.com \
    --cc=johan.hedberg@gmail.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcel@holtmann.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox