Hi Fred, > Signed-off-by: Frédéric Danis <frederic.danis.oss@xxxxxxxxx> > --- > drivers/bluetooth/hci_bcm.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/drivers/bluetooth/hci_bcm.c b/drivers/bluetooth/hci_bcm.c > index 2e358cc..b1cf07e 100644 > --- a/drivers/bluetooth/hci_bcm.c > +++ b/drivers/bluetooth/hci_bcm.c > @@ -922,7 +922,6 @@ static const struct hci_uart_proto bcm_proto = { > #ifdef CONFIG_ACPI > static const struct acpi_device_id bcm_acpi_match[] = { > { "BCM2E1A", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > - { "BCM2E39", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > { "BCM2E3A", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > { "BCM2E3D", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > { "BCM2E3F", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > @@ -942,6 +941,14 @@ static const struct acpi_device_id bcm_acpi_match[] = { > MODULE_DEVICE_TABLE(acpi, bcm_acpi_match); > #endif > > +#ifdef CONFIG_ACPI > +static const struct acpi_device_id bcm_serdev_acpi_match[] = { > + { "BCM2E39", (kernel_ulong_t)&acpi_bcm_int_last_gpios }, > + { }, > +}; > +MODULE_DEVICE_TABLE(acpi, bcm_serdev_acpi_match); > +#endif > + > /* Platform suspend and resume callbacks */ > static const struct dev_pm_ops bcm_pm_ops = { > SET_SYSTEM_SLEEP_PM_OPS(bcm_suspend, bcm_resume) > @@ -999,6 +1006,7 @@ static struct serdev_device_driver bcm_serdev_driver = { > .driver = { > .name = "hci_uart_bcm", > .of_match_table = of_match_ptr(bcm_bluetooth_of_match), > + .acpi_match_table = ACPI_PTR(bcm_serdev_acpi_match), > }, > }; I think doing this one device at a time is actually fine. However please add a proper commit message for it explaining it. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html