netdev - Re: [PATCH] net: rfkill: gpio: Add OBDA8723 ACPI HID

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d63b1718-910a-09e2-7a2a-f34c08c11ce6@lwfinger.net>
Date:   Sun, 9 Apr 2017 10:50:30 -0500
From:   Larry Finger <Larry.Finger@...inger.net>
To:     Marcel Holtmann <marcel@...tmann.org>,
        Hans de Goede <hdegoede@...hat.com>
Cc:     Johannes Berg <johannes@...solutions.net>,
        "David S. Miller" <davem@...emloft.net>,
        linux-wireless <linux-wireless@...r.kernel.org>,
        Network Development <netdev@...r.kernel.org>,
        russianneuromancer@...ru,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: Re: [PATCH] net: rfkill: gpio: Add OBDA8723 ACPI HID

On 04/09/2017 07:11 AM, Marcel Holtmann wrote:
> Hi Hans,
>
>>>> The OBDA8723 ACPI HID is used on quite a few Bay Trail based tablets
>>>> for bluetooth rfkill functionality.
>>>>
>>>> Tested-by: russianneuromancer@...ru <russianneuromancer@...ru>
>>>> Signed-off-by: Hans de Goede <hdegoede@...hat.com>
>>>> ---
>>>> net/rfkill/rfkill-gpio.c | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c
>>>> index 76c01cb..50ca65e 100644
>>>> --- a/net/rfkill/rfkill-gpio.c
>>>> +++ b/net/rfkill/rfkill-gpio.c
>>>> @@ -163,6 +163,7 @@ static int rfkill_gpio_remove(struct platform_device *pdev)
>>>> static const struct acpi_device_id rfkill_acpi_match[] = {
>>>> 	{ "BCM4752", RFKILL_TYPE_GPS },
>>>> 	{ "LNV4752", RFKILL_TYPE_GPS },
>>>> +	{ "OBDA8723", RFKILL_TYPE_BLUETOOTH },
>>>> 	{ },
>>>> };
>>>
>>> NAK. We are integrating these with hci_bcm.c or hci_intel.c drivers.
>>
>> This is for the bluetooth side of the rtl8723bs driver which recently
>> (yesterday) got merged in into drivers/staging. Which still needs
>> hciattach from userspace. I completely agree that eventually we should
>> fix that. In the mean time it would be nice if we could carry this
>> one line patch to give people using the staging driver working bluetooth.
>
> why are Bluetooth drivers in staging? I objected to them before. The only reason to have them in staging would be people being to lazy to clean things up.

The Bluetooth driver is not in staging. It is only the wifi part; however, 
having a driver for that hardware in the kernel makes it easier for users to 
handle the BT part of the device. As Hans stated, the current BT driver runs in 
userspace. Until we actually produce hci_rtk, having this one line patch will be 
helpful.

Larry



-- 
If I was stranded on an island and the only way to get off
the island was to make a pretty UI, I’d die there.

Linus Torvalds