diff mbox series

[v1] Bluetooth: support download nvm with different board id for wcn6855

Message ID 1606791564-2443-1-git-send-email-zijuhu@codeaurora.org
State New
Headers show
Series [v1] Bluetooth: support download nvm with different board id for wcn6855 | expand

Commit Message

Zijun Hu Dec. 1, 2020, 2:59 a.m. UTC
From: Tim Jiang <tjiang@codeaurora.org>

we define many nvm files for wcn6855 btsoc and host driver
should find the correct nvm file based on board ID and then
download it.

Signed-off-by: Tim Jiang <tjiang@codeaurora.org>
---
 drivers/bluetooth/btusb.c | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

Comments

bluez.test.bot@gmail.com Dec. 1, 2020, 4:03 a.m. UTC | #1
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=393653

---Test result---

##############################
Test: CheckPatch - PASS

##############################
Test: CheckGitLint - PASS

##############################
Test: CheckBuildK - PASS



---
Regards,
Linux Bluetooth
Bjorn Andersson Dec. 2, 2020, 1:51 a.m. UTC | #2
On Mon 30 Nov 20:59 CST 2020, Zijun Hu wrote:

> From: Tim Jiang <tjiang@codeaurora.org>

> 

> we define many nvm files for wcn6855 btsoc and host driver

> should find the correct nvm file based on board ID and then

> download it.

> 

> Signed-off-by: Tim Jiang <tjiang@codeaurora.org>

> ---

>  drivers/bluetooth/btusb.c | 17 ++++++++++++++---

>  1 file changed, 14 insertions(+), 3 deletions(-)

> 

> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c

> index 3bbe8f43e7fa..66e19085e0fa 100644

> --- a/drivers/bluetooth/btusb.c

> +++ b/drivers/bluetooth/btusb.c

> @@ -3474,7 +3474,8 @@ struct qca_version {

>  	__le32	rom_version;

>  	__le32	patch_version;

>  	__le32	ram_version;

> -	__le32	ref_clock;

> +	__u16	board_id;


You should follow the scheme of describing the endianess of the fields.

> +	__u8	flag[2];


It seems more reasonable to make this a 16-bit flags. And either way
there are more than one of these, so plural "flags" seems appropriate.


PS. Can you confirm that no firmware actually used these 16 bits of the
"ref_clock"? Why wasn't the reserved bytes used to add the new
properties?

>  	__u8	reserved[4];

>  } __packed;

>  

> @@ -3657,8 +3658,13 @@ static int btusb_setup_qca_load_nvm(struct hci_dev *hdev,

>  	char fwname[64];

>  	int err;

>  

> -	snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x.bin",

> -		 le32_to_cpu(ver->rom_version));

> +	if (ver->flag[1] == 0x80) { //check board id for wcn6855


Is this BIT(7) in the second byte of the flags, or is the second flag
0x80?

Based on the comment you should be able to provide a

#define QCA_VERSION_SECOND_FLAG_IS_WCN6855 0x80

to use instead of this magic number.

Regards,
Bjorn

> +		snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x_%04x.bin",

> +			le32_to_cpu(ver->rom_version), le16_to_cpu(ver->board_id));

> +	} else {

> +		snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x.bin",

> +			le32_to_cpu(ver->rom_version));

> +	}

>  

>  	err = request_firmware(&fw, fwname, &hdev->dev);

>  	if (err) {

> @@ -3725,6 +3731,11 @@ static int btusb_setup_qca(struct hci_dev *hdev)

>  			return err;

>  	}

>  

> +	err = btusb_qca_send_vendor_req(udev, QCA_GET_TARGET_VERSION, &ver,

> +					sizeof(ver));

> +	if (err < 0)

> +		return err;

> +

>  	if (!(status & QCA_SYSCFG_UPDATED)) {

>  		err = btusb_setup_qca_load_nvm(hdev, &ver, info);

>  		if (err < 0)

> -- 

> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

>
diff mbox series

Patch

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 3bbe8f43e7fa..66e19085e0fa 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -3474,7 +3474,8 @@  struct qca_version {
 	__le32	rom_version;
 	__le32	patch_version;
 	__le32	ram_version;
-	__le32	ref_clock;
+	__u16	board_id;
+	__u8	flag[2];
 	__u8	reserved[4];
 } __packed;
 
@@ -3657,8 +3658,13 @@  static int btusb_setup_qca_load_nvm(struct hci_dev *hdev,
 	char fwname[64];
 	int err;
 
-	snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x.bin",
-		 le32_to_cpu(ver->rom_version));
+	if (ver->flag[1] == 0x80) { //check board id for wcn6855
+		snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x_%04x.bin",
+			le32_to_cpu(ver->rom_version), le16_to_cpu(ver->board_id));
+	} else {
+		snprintf(fwname, sizeof(fwname), "qca/nvm_usb_%08x.bin",
+			le32_to_cpu(ver->rom_version));
+	}
 
 	err = request_firmware(&fw, fwname, &hdev->dev);
 	if (err) {
@@ -3725,6 +3731,11 @@  static int btusb_setup_qca(struct hci_dev *hdev)
 			return err;
 	}
 
+	err = btusb_qca_send_vendor_req(udev, QCA_GET_TARGET_VERSION, &ver,
+					sizeof(ver));
+	if (err < 0)
+		return err;
+
 	if (!(status & QCA_SYSCFG_UPDATED)) {
 		err = btusb_setup_qca_load_nvm(hdev, &ver, info);
 		if (err < 0)