Re: [PATCH v2 1/2] Bluetooth: hci_bcm: Add support for MINIX Z83-4 based devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ian,

> The MINIX NEO Z83-4 and MINIX NEO Z83-4 Pro devices use an AP6255 chip
> for wifi and bluetooth. Bluetooth requires an ACPI device id of BCM2EA4
> with BCM4345 rev C0 firmware.
> 
> This patch adds the device id and to use trigger type IRQF_TRIGGER_FALLING.
> 
> Signed-off-by: ianwmorrison <ianwmorrison@xxxxxxxxx>
> 
> ---
> drivers/bluetooth/hci_bcm.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/bluetooth/hci_bcm.c b/drivers/bluetooth/hci_bcm.c
> index ab1455e63b92..77326eeb6146 100644
> --- a/drivers/bluetooth/hci_bcm.c
> +++ b/drivers/bluetooth/hci_bcm.c
> @@ -726,6 +726,13 @@ static const struct dmi_system_id bcm_active_low_irq_dmi_table[] = {
> 			DMI_EXACT_MATCH(DMI_PRODUCT_VERSION, "ThinkPad 8"),
> 		},
> 	},
> +	{
> +		.ident = "MINIX Z83-4",
> +		.matches = {
> +			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "MINIX"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "Z83-4"),
> +		},
> +	},
> 	{ }
> };
> 
> @@ -934,6 +941,7 @@ static const struct acpi_device_id bcm_acpi_match[] = {
> 	{ "BCM2E7C", (kernel_ulong_t)&acpi_bcm_int_last_gpios },
> 	{ "BCM2E95", (kernel_ulong_t)&acpi_bcm_int_first_gpios },
> 	{ "BCM2E96", (kernel_ulong_t)&acpi_bcm_int_first_gpios },
> +	{ "BCM2EA4", (kernel_ulong_t)&acpi_bcm_int_first_gpios },
> 	{ },
> };
> MODULE_DEVICE_TABLE(acpi, bcm_acpi_match);

can you actually add an extract of the ACPI table where the IRQ polarity is described wrongly. I like to have that in the commit message so that this becomes some sort of record.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux