linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Loic Poulain <loic.poulain@intel.com>
To: "Jérôme de Bretagne" <jerome.debretagne@gmail.com>,
	"Ilya Faenson" <ilya.faenson@broadcom.com>,
	linux-bluetooth@vger.kernel.org,
	"Marcel Holtmann" <marcel@holtmann.org>
Subject: Re: Broadcom BCM4324(1) rev B5 chipset - Bluetooth support in Linux Kernel 4.x ?
Date: Mon, 07 Mar 2016 10:42:07 +0100	[thread overview]
Message-ID: <56DD4CEF.3070300@intel.com> (raw)
In-Reply-To: <CA+kEDGHz=V+BBxbnQDDKwNgLUFjj_9p85DepRSqArRJUzbyusQ@mail.gmail.com>

Hi Jerome,

> Maybe this is a naive question about ACPI ID, and not specific to
> Broadcom by the way: could there be a more generic way to gather ACPI
> IDs used by new devices and add them to the kernel on a regular basis?
> They seem to be added almost one by one mainly upon user reports when
> looking at the mailing list. So Bluetooth doesn't work out-of-the-box
> on some systems which would be compatible otherwise, especially the
> less popular ones. Is there no existing driver list to re-use perhaps?

New ACPI ID means that this is a different hardware. So it needs to be
tested before being added into the driver supported list. An already
supported hardware should keep the same ACPI ID, making it work out-of
the-box.

Since this driver is pretty new, this is not surprising that some IDs
are regularly added. This is a good thing. Meaning that people use this
driver and that Broadcom controllers keep good compatibility across
hardware versions.

> Indeed, it is working fine with ttyS1 and the correct binary firmware
> stored at /lib/firmware/brcm/BCM.hcd:
>
>     # btattach -B /dev/ttyS1 -P bcm
>     Attaching BR/EDR controller to /dev/ttyS1
>     Switched line discipline from 0 to 15
>     Device index 0 attached
>
> Finally, is btattach the only command recommended and supported moving
> forward, or should hciattach work as well with that specific
> chipset/tablet (as a transition until most Linux distros switch to
> btattach)?

Absolutely, think btattach is the only recommended way.
Even if you can make it work with hciattach by adding device entry with
correct protocol ID, it is not recommended since hciattach is dedicated
to user-space initialisation (old-way) and code is pretty deprecated.

Regards,
Loic

-- 
Intel Open Source Technology Center
http://oss.intel.com/

  reply	other threads:[~2016-03-07  9:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CA+kEDGEHfsODon6aOdLU86E9c0Lfs-4bo=ycEZz3MdEa1qLrKw@mail.gmail.com>
2016-02-22  9:17 ` Broadcom BCM4324(1) rev B5 chipset - Bluetooth support in Linux Kernel 4.x ? Arend Van Spriel
2016-02-28 17:56   ` Jérôme de Bretagne
2016-02-28 18:23   ` Jérôme de Bretagne
2016-02-28 19:54     ` Loic Poulain
     [not found]       ` <CA+kEDGHbzH9VPRUaz5-JPsU5kHTDk8UwGGmt94rGVXiG1KY-Ow@mail.gmail.com>
2016-02-28 22:58         ` Loic Poulain
2016-03-06 13:22           ` Jérôme de Bretagne
2016-03-07  9:42             ` Loic Poulain [this message]
2016-03-07 10:48               ` Jérôme de Bretagne
2016-03-07 15:06                 ` Loic Poulain

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=56DD4CEF.3070300@intel.com \
    --to=loic.poulain@intel.com \
    --cc=ilya.faenson@broadcom.com \
    --cc=jerome.debretagne@gmail.com \
    --cc=linux-bluetooth@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;
as well as URLs for NNTP newsgroup(s).