public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
From: Johan Hovold <johan+linaro@kernel.org>
To: Marcel Holtmann <marcel@holtmann.org>,
	Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: Doug Anderson <dianders@chromium.org>,
	Janaki Ramaiah Thota <quic_janathot@quicinc.com>,
	linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org,
	Johan Hovold <johan+linaro@kernel.org>
Subject: [PATCH v2 0/3] Bluetooth: qca: generalise device address check
Date: Tue, 30 Apr 2024 19:07:38 +0200	[thread overview]
Message-ID: <20240430170741.15742-1-johan+linaro@kernel.org> (raw)

The QCA default device address apparently comes from the NVM
configuration file and can differ quite a bit between controllers.

This series adds support for storing the default address when parsing
the configuration file and using it to determine whether the controller
has been provisioned with an address.
    
This makes sure that devices without a unique address start as
unconfigured unless a valid address has been provided in the devicetree.

Included in v2 are two preparatory but otherwise independent fixes that
adds the missing sanity checks when parsing the firmware files and makes
sure that the parser can handle configuration files for WCN3xxx.

Johan


Changes in v2:
 - add missing firmware sanity checks (new patch)
 - fix nvm configuration parsing (new patch)
 - make sure to set the BD_ADDR quirk flag also when the controller
   returns BDADDR_ANY


Johan Hovold (3):
  Bluetooth: qca: add missing firmware sanity checks
  Bluetooth: qca: fix NVM configuration parsing
  Bluetooth: qca: generalise device address check

 drivers/bluetooth/btqca.c | 83 +++++++++++++++++++++++++++++++--------
 drivers/bluetooth/btqca.h |  2 +
 2 files changed, 68 insertions(+), 17 deletions(-)

-- 
2.43.2


             reply	other threads:[~2024-04-30 17:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-30 17:07 Johan Hovold [this message]
2024-04-30 17:07 ` [PATCH v2 1/3] Bluetooth: qca: add missing firmware sanity checks Johan Hovold
2024-04-30 17:43   ` Bluetooth: qca: generalise device address check bluez.test.bot
2024-05-01  6:27   ` [PATCH v2 1/3] Bluetooth: qca: add missing firmware sanity checks Johan Hovold
2024-04-30 17:07 ` [PATCH v2 2/3] Bluetooth: qca: fix NVM configuration parsing Johan Hovold
2024-04-30 17:07 ` [PATCH v2 3/3] Bluetooth: qca: generalise device address check Johan Hovold
2024-04-30 21:21   ` Doug Anderson
2024-05-01  6:31     ` Johan Hovold
2024-04-30 21:51 ` [PATCH v2 0/3] " patchwork-bot+bluetooth

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=20240430170741.15742-1-johan+linaro@kernel.org \
    --to=johan+linaro@kernel.org \
    --cc=dianders@chromium.org \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luiz.dentz@gmail.com \
    --cc=marcel@holtmann.org \
    --cc=quic_janathot@quicinc.com \
    /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