Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1571503
| From | João Paulo Rechi Vita <jprvita@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 8/8] asus-wmi: Don't register rfkill if ASHS and user bit are present |
| Date | 2017-02-01 13:20 +0100 |
| Message-ID | <t623g-5RJ-7@gated-at.bofh.it> (permalink) |
| References | <t3Ujv-Eh-15@gated-at.bofh.it> <t3Ujx-Eh-51@gated-at.bofh.it> <t41bk-4KV-19@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 26 January 2017 at 17:45, Joe Perches <joe@perches.com> wrote:
> On Thu, 2017-01-26 at 10:30 -0500, João Paulo Rechi Vita wrote:
>> Some Asus laptops that have an airplane-mode indicator LED, also have
>> the WMI WLAN user bit set, and the following bits in their DSDT:
>
> style trivia:
>
>> diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c
> []
>> @@ -2051,6 +2052,16 @@ static int asus_wmi_fan_init(struct asus_wmi *asus)
>> return 0;
>> }
>>
>> +static bool ashs_present(void)
>> +{
>> + int i;
>> +
>> + for (i = 0; strcmp(asus_wireless_ids[i].id, ""); i++)
>> + if (acpi_dev_found(asus_wireless_ids[i].id))
>> + return true;
>
> The for loop would commonly have braces because of the
> internal if.
>
Thanks, I'm fixing it for the next revision.
--
João Paulo Rechi Vita
http://about.me/jprvita
Back to linux.kernel | Previous | Next | Find similar | Unroll thread
Re: [PATCH 8/8] asus-wmi: Don't register rfkill if ASHS and user bit are present João Paulo Rechi Vita <jprvita@gmail.com> - 2017-02-01 13:20 +0100
csiph-web