linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ulf.hansson@linaro.org (Ulf Hansson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] arm64: hi6220-hikey: Fix Bluetooth support
Date: Wed,  7 Jun 2017 11:08:19 +0200	[thread overview]
Message-ID: <1496826502-1339-1-git-send-email-ulf.hansson@linaro.org> (raw)

The commit ea452678734e ("arm64: dts: hikey: Fix WiFi support") indeed managed
to fix the WiFi support for the Hikey board. However, it also caused the
Bluetooth to break.

The reason to the problem is because part of the fixes for WiFi, involved
explicitly to start manage the external clock for the TI WiLink chip, as it's
needed to properly power on/off WiFi. Then, because the Bluetooth driver,
hci_ll, don't manage the clock, it fails to initialize Bluetooth.

This small series add support and DT bindings for the external clock, now
properly managed by the hci_ll driver during power on/off. On top of that, the
external clock is added to the Bluetooth DT node in the Hikey DTS file.

The regression has been reported by John Stultz, running a Hikey board with
UEFI based boot. I have confirmed that there is no regressions with u-boot as
we still don't have bluetooth working on that set up. I intend to fix that as
well, although in second step and not being part of a fixup series.

John, can you please confirm/test this on your Hikey board using UEFI!?

Perhaps this can go in via the bluetooth git, when respective patches has got
an ack. Or whatever you think is best.

Ulf Hansson (3):
  dt-bindings: net: Add binding for the external clock for TI WiLink
  Bluetooth: hci_ll: Add support for the external clock
  arm64: dts: hikey: Add external clock to the Bluetooth node

 Documentation/devicetree/bindings/net/ti,wilink-st.txt |  6 ++++++
 arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts         |  2 ++
 drivers/bluetooth/hci_ll.c                             | 14 +++++++++++++-
 3 files changed, 21 insertions(+), 1 deletion(-)

-- 
2.7.4

             reply	other threads:[~2017-06-07  9:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-07  9:08 Ulf Hansson [this message]
2017-06-07  9:08 ` [PATCH 1/3] dt-bindings: net: Add binding for the external clock for TI WiLink Ulf Hansson
2017-06-07  9:08 ` [PATCH 2/3] Bluetooth: hci_ll: Add support for the external clock Ulf Hansson
2017-06-07  9:08 ` [PATCH 3/3] arm64: dts: hikey: Add external clock to the Bluetooth node Ulf Hansson
2017-06-07 19:07 ` [PATCH 0/3] arm64: hi6220-hikey: Fix Bluetooth support John Stultz
2017-06-07 19:14   ` Daniel Lezcano
2017-06-09  5:49 ` 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=1496826502-1339-1-git-send-email-ulf.hansson@linaro.org \
    --to=ulf.hansson@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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).