Commit 294d749b authored by Kiran K's avatar Kiran K Committed by Luiz Augusto von Dentz
Browse files

Bluetooth: btintel: Iterate only bluetooth device ACPI entries 



Current flow interates over entire ACPI table entries looking for
Bluetooth Per Platform Antenna Gain(PPAG) entry. This patch iterates
over ACPI entries relvant to Bluetooth device only.

Fixes: c585a92b ("Bluetooth: btintel: Set Per Platform Antenna Gain(PPAG)")
Signed-off-by: default avatarKiran K <kiran.k@intel.com>
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent 2f10e40a

drivers/bluetooth/btintel.c

+26 −18
Original line number Diff line number Diff line
@@ -26,7 +26,14 @@ 
#define ECDSA_HEADER_LEN	320



#define BTINTEL_PPAG_NAME   "PPAG"

#define BTINTEL_PPAG_PREFIX "\\_SB_.PCI0.XHCI.RHUB"



/* structure to store the PPAG data read from ACPI table */

struct btintel_ppag {

	u32	domain;

	u32     mode;

	acpi_status status;

	struct hci_dev *hdev;

};



#define CMD_WRITE_BOOT_PARAMS	0xfc0e

struct cmd_write_boot_params {
@@ -1295,17 +1302,16 @@ static acpi_status btintel_ppag_callback(acpi_handle handle, u32 lvl, void *data 


	status = acpi_get_name(handle, ACPI_FULL_PATHNAME, &string);

	if (ACPI_FAILURE(status)) {

		bt_dev_warn(hdev, "ACPI Failure: %s", acpi_format_exception(status));

		bt_dev_warn(hdev, "PPAG-BT: ACPI Failure: %s", acpi_format_exception(status));

		return status;

	}



	if (strncmp(BTINTEL_PPAG_PREFIX, string.pointer,

		    strlen(BTINTEL_PPAG_PREFIX))) {

	len = strlen(string.pointer);

	if (len < strlen(BTINTEL_PPAG_NAME)) {

		kfree(string.pointer);

		return AE_OK;

	}



	len = strlen(string.pointer);

	if (strncmp((char *)string.pointer + len - 4, BTINTEL_PPAG_NAME, 4)) {

		kfree(string.pointer);

		return AE_OK;
@@ -1314,7 +1320,8 @@ static acpi_status btintel_ppag_callback(acpi_handle handle, u32 lvl, void *data 


	status = acpi_evaluate_object(handle, NULL, NULL, &buffer);

	if (ACPI_FAILURE(status)) {

		bt_dev_warn(hdev, "ACPI Failure: %s", acpi_format_exception(status));

		ppag->status = status;

		bt_dev_warn(hdev, "PPAG-BT: ACPI Failure: %s", acpi_format_exception(status));

		return status;

	}
@@ -1323,8 +1330,9 @@ static acpi_status btintel_ppag_callback(acpi_handle handle, u32 lvl, void *data 


	if (p->type != ACPI_TYPE_PACKAGE || p->package.count != 2) {

		kfree(buffer.pointer);

		bt_dev_warn(hdev, "Invalid object type: %d or package count: %d",

		bt_dev_warn(hdev, "PPAG-BT: Invalid object type: %d or package count: %d",

			    p->type, p->package.count);

		ppag->status = AE_ERROR;

		return AE_ERROR;

	}
@@ -1335,6 +1343,7 @@ static acpi_status btintel_ppag_callback(acpi_handle handle, u32 lvl, void *data 


	ppag->domain = (u32)p->package.elements[0].integer.value;

	ppag->mode = (u32)p->package.elements[1].integer.value;

	ppag->status = AE_OK;

	kfree(buffer.pointer);

	return AE_CTRL_TERMINATE;

}
@@ -2314,7 +2323,6 @@ static int btintel_configure_offload(struct hci_dev *hdev) 


static void btintel_set_ppag(struct hci_dev *hdev, struct intel_version_tlv *ver)

{

	acpi_status status;

	struct btintel_ppag ppag;

	struct sk_buff *skb;

	struct btintel_loc_aware_reg ppag_cmd;
@@ -2330,26 +2338,26 @@ static void btintel_set_ppag(struct hci_dev *hdev, struct intel_version_tlv *ver 
	memset(&ppag, 0, sizeof(ppag));



	ppag.hdev = hdev;

	status = acpi_walk_namespace(ACPI_TYPE_ANY, ACPI_ROOT_OBJECT,

				     ACPI_UINT32_MAX, NULL,

				     btintel_ppag_callback, &ppag, NULL);

	ppag.status = AE_NOT_FOUND;

	acpi_walk_namespace(ACPI_TYPE_PACKAGE, ACPI_HANDLE(GET_HCIDEV_DEV(hdev)),

			    1, NULL, btintel_ppag_callback, &ppag, NULL);



	if (ACPI_FAILURE(status)) {

		/* Do not log warning message if ACPI entry is not found */

		if (status == AE_NOT_FOUND)

	if (ACPI_FAILURE(ppag.status)) {

		if (ppag.status == AE_NOT_FOUND) {

			bt_dev_dbg(hdev, "PPAG-BT: ACPI entry not found");

			return;

		bt_dev_warn(hdev, "PPAG: ACPI Failure: %s", acpi_format_exception(status));

		}

		return;

	}



	if (ppag.domain != 0x12) {

		bt_dev_warn(hdev, "PPAG-BT Domain disabled");

		bt_dev_warn(hdev, "PPAG-BT: domain is not bluetooth");

		return;

	}



	/* PPAG mode, BIT0 = 0 Disabled, BIT0 = 1 Enabled */

	if (!(ppag.mode & BIT(0))) {

		bt_dev_dbg(hdev, "PPAG disabled");

		bt_dev_dbg(hdev, "PPAG-BT: disabled");

		return;

	}

drivers/bluetooth/btintel.h

+0 −7
Original line number Diff line number Diff line
@@ -137,13 +137,6 @@ struct intel_offload_use_cases { 
	__u8	preset[8];

} __packed;



/* structure to store the PPAG data read from ACPI table */

struct btintel_ppag {

	u32	domain;

	u32     mode;

	struct hci_dev *hdev;

};



struct btintel_loc_aware_reg {

	__le32 mcc;

	__le32 sel;

include/net/bluetooth/hci_core.h

+1 −0
Original line number Diff line number Diff line
@@ -1613,6 +1613,7 @@ void hci_conn_add_sysfs(struct hci_conn *conn); 
void hci_conn_del_sysfs(struct hci_conn *conn);



#define SET_HCIDEV_DEV(hdev, pdev) ((hdev)->dev.parent = (pdev))

#define GET_HCIDEV_DEV(hdev) ((hdev)->dev.parent)



/* ----- LMP capabilities ----- */

#define lmp_encrypt_capable(dev)   ((dev)->features[0][0] & LMP_ENCRYPT)