On Wed, Dec 23, 2015 at 04:28:59PM +0100, Marcel Holtmann wrote: > Hi Heikki, > > > These are used with BCM43241 Wi-Fi/Bluetooth Combo Device. > > > > Signed-off-by: Heikki Krogerus <[email protected]> > > --- > > net/rfkill/rfkill-gpio.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c > > index 9312722..ba6d61d 100644 > > --- a/net/rfkill/rfkill-gpio.c > > +++ b/net/rfkill/rfkill-gpio.c > > @@ -164,9 +164,12 @@ static int rfkill_gpio_remove(struct platform_device > > *pdev) > > #ifdef CONFIG_ACPI > > static const struct acpi_device_id rfkill_acpi_match[] = { > > { "BCM2E1A", RFKILL_TYPE_BLUETOOTH }, > > + { "BCM2E3A", RFKILL_TYPE_BLUETOOTH }, > > { "BCM2E3D", RFKILL_TYPE_BLUETOOTH }, > > + { "BCM2E3F", RFKILL_TYPE_BLUETOOTH }, > > { "BCM2E40", RFKILL_TYPE_BLUETOOTH }, > > { "BCM2E64", RFKILL_TYPE_BLUETOOTH }, > > + { "BCM2E7B", RFKILL_TYPE_BLUETOOTH }, > > { "BCM4752", RFKILL_TYPE_GPS }, > > { "LNV4752", RFKILL_TYPE_GPS }, > > { }, > > I am actually not convinced that these should be added here. We are driving > the Broadcom chips via drivers/bluetooth/hci_bcm.c and I pretty sure that > these devices should be added there. We moved a few of the existing ones > already over, but I have the feeling that in the end all should be moved.
I totally agree! That is so much better! Frederic did tell me about adding ACPI support to that driver, but I forgot. Sorry about that. So Johannes, forget about this patch. I'll create a patch where I'll propose these ID's to that driver instead. I can also prepare patch where we move the other BCM2Exx ACPI IDs to drivers/bluetooth/hci_bcm.c at the same time if you like. Thanks, -- heikki -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
