Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1567536 > unrolled thread
| Started by | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| First post | 2017-01-26 16:40 +0100 |
| Last post | 2017-01-27 16:40 +0100 |
| Articles | 10 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH 0/8] asus-wmi: Get rid of quirk_no_rfkill{,_wapf4} "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
[PATCH 4/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus U303LB" "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
[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-01-26 16:40 +0100
Re: [PATCH 8/8] asus-wmi: Don't register rfkill if ASHS and user bit are present Joe Perches <joe@perches.com> - 2017-01-27 00:00 +0100
[PATCH 2/8] Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UF" "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
[PATCH 5/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus N552VW" "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
[PATCH 7/8] asus-wireless: Export ids list "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
Re: [PATCH 7/8] asus-wireless: Export ids list Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-27 16:40 +0100
[PATCH 3/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus Z550MA" "João Paulo Rechi Vita" <jprvita@gmail.com> - 2017-01-26 16:40 +0100
Re: [PATCH 0/8] asus-wmi: Get rid of quirk_no_rfkill{,_wapf4} Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-27 16:40 +0100
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 0/8] asus-wmi: Get rid of quirk_no_rfkill{,_wapf4} |
| Message-ID | <t3Ujv-Eh-15@gated-at.bofh.it> |
This quirk avoids the conflicting usage of ASUS_WMI_DEVID_WLAN_LED (0x00010002) by both asus-wireless and asus-wmi on machines where the BIOS can't set and record the wlan status (wlan_ctrl_by_user in asus-wmi.c). At the moment we have six models using that quirk upstream, but there are another fifteen for which we have downstream patches, and will send upstream unless we find a more dynamic solution. We also expect this list to continue growing. This series makes use of ASUS_WMI_DSTS_USER_BIT plus the status of the ASHS device to skip asus_wmi_rfkill_init(). Since asus-wmi now needs to know the ASHS _HIDs, I have moved device_ids[] to a new header file, asus-wireless.h. I'm not sure if there are any best practices in the kernel community against having one module including headers from a different module, but I wanted to avoid duplicating this list or creating an ordering dependency between the modules. Any advice on alternatives here are welcome. PS: I don't think asus-wireless.h needs to also be listed in MAINTAINERS, please correct me if I'm wrong. João Paulo Rechi Vita (8): Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UA" Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UF" Revert "asus-wmi: Add quirk_no_rfkill for the Asus Z550MA" Revert "asus-wmi: Add quirk_no_rfkill for the Asus U303LB" Revert "asus-wmi: Add quirk_no_rfkill for the Asus N552VW" Revert "asus-wmi: Create quirk for airplane_mode LED" asus-wireless: Export ids list asus-wmi: Don't register rfkill if ASHS and user bit are present drivers/platform/x86/asus-nb-wmi.c | 49 ++---------------------------------- drivers/platform/x86/asus-wireless.c | 15 +++++------ drivers/platform/x86/asus-wireless.h | 10 ++++++++ drivers/platform/x86/asus-wmi.c | 21 ++++++++++++---- drivers/platform/x86/asus-wmi.h | 1 - 5 files changed, 34 insertions(+), 62 deletions(-) create mode 100644 drivers/platform/x86/asus-wireless.h -- 2.11.0
[toc] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 4/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus U303LB" |
| Message-ID | <t3Ujx-Eh-45@gated-at.bofh.it> |
| In reply to | #1567536 |
This reverts commit 02db9ff7af18f7ace60f430cbbaa1d846b350916.
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-nb-wmi.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c
index 9d686d642934..5e088fbef5ad 100644
--- a/drivers/platform/x86/asus-nb-wmi.c
+++ b/drivers/platform/x86/asus-nb-wmi.c
@@ -382,15 +382,6 @@ static const struct dmi_system_id asus_quirks[] = {
},
{
.callback = dmi_matched,
- .ident = "ASUSTeK COMPUTER INC. U303LB",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
- DMI_MATCH(DMI_PRODUCT_NAME, "U303LB"),
- },
- .driver_data = &quirk_no_rfkill,
- },
- {
- .callback = dmi_matched,
.ident = "ASUSTeK COMPUTER INC. UX303UB",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 8/8] asus-wmi: Don't register rfkill if ASHS and user bit are present |
| Message-ID | <t3Ujx-Eh-51@gated-at.bofh.it> |
| In reply to | #1567536 |
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:
Scope (_SB)
{
(...)
Device (ATKD)
{
(...)
Method (WMNB, 3, Serialized)
{
(...)
If (LEqual (IIA0, 0x00010002))
{
OWGD (IIA1)
Return (One)
}
}
}
}
So when asus-wmi uses ASUS_WMI_DEVID_WLAN_LED (0x00010002) to store the
wlan state, it drives the airplane-mode indicator LED (through the call
to OWGD) in an inverted fashion: the LED is ON when airplane mode is OFF
(since wlan is ON), and vice-versa.
This skips registering RFKill switches at all for these laptops, to
allow the asus-wireless driver to drive the airplane mode LED correctly
through the ASHS ACPI device. Relying on the presence of ASHS and
ASUS_WMI_DSTS_USER_BIT avoids adding DMI-based quirks for at least 21
different laptops.
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-wmi.c | 27 ++++++++++++++++++++-------
1 file changed, 20 insertions(+), 7 deletions(-)
diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c
index da3b12c131ce..645204865a3f 100644
--- a/drivers/platform/x86/asus-wmi.c
+++ b/drivers/platform/x86/asus-wmi.c
@@ -49,6 +49,7 @@
#include <linux/dmi.h>
#include <acpi/video.h>
+#include "asus-wireless.h"
#include "asus-wmi.h"
MODULE_AUTHOR("Corentin Chary <corentin.chary@gmail.com>, "
@@ -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;
+ return false;
+}
+
/*
* WMI Driver
*/
@@ -2095,9 +2106,15 @@ static int asus_wmi_add(struct platform_device *pdev)
if (err)
goto fail_leds;
- err = asus_wmi_rfkill_init(asus);
- if (err)
- goto fail_rfkill;
+ asus_wmi_get_devstate(asus, ASUS_WMI_DEVID_WLAN, &result);
+ if (result & (ASUS_WMI_DSTS_PRESENCE_BIT | ASUS_WMI_DSTS_USER_BIT))
+ asus->driver->wlan_ctrl_by_user = 1;
+
+ if (!(asus->driver->wlan_ctrl_by_user && ashs_present())) {
+ err = asus_wmi_rfkill_init(asus);
+ if (err)
+ goto fail_rfkill;
+ }
/* Some Asus desktop boards export an acpi-video backlight interface,
stop this from showing up */
@@ -2132,10 +2149,6 @@ static int asus_wmi_add(struct platform_device *pdev)
if (err)
goto fail_debugfs;
- asus_wmi_get_devstate(asus, ASUS_WMI_DEVID_WLAN, &result);
- if (result & (ASUS_WMI_DSTS_PRESENCE_BIT | ASUS_WMI_DSTS_USER_BIT))
- asus->driver->wlan_ctrl_by_user = 1;
-
return 0;
fail_debugfs:
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2017-01-27 00:00 +0100 |
| Subject | Re: [PATCH 8/8] asus-wmi: Don't register rfkill if ASHS and user bit are present |
| Message-ID | <t41bk-4KV-19@gated-at.bofh.it> |
| In reply to | #1567541 |
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.
[toc] | [prev] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 2/8] Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UF" |
| Message-ID | <t3Ujx-Eh-67@gated-at.bofh.it> |
| In reply to | #1567536 |
This reverts commit a961a285b479977fa21f12f71cd62f28adaba17c.
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-nb-wmi.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c
index 6079510eaa29..5bf1dd225788 100644
--- a/drivers/platform/x86/asus-nb-wmi.c
+++ b/drivers/platform/x86/asus-nb-wmi.c
@@ -107,11 +107,6 @@ static struct quirk_entry quirk_no_rfkill = {
.no_rfkill = true,
};
-static struct quirk_entry quirk_no_rfkill_wapf4 = {
- .wapf = 4,
- .no_rfkill = true,
-};
-
static struct quirk_entry quirk_asus_ux303ub = {
.wmi_backlight_native = true,
};
@@ -203,7 +198,7 @@ static const struct dmi_system_id asus_quirks[] = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
DMI_MATCH(DMI_PRODUCT_NAME, "X456UF"),
},
- .driver_data = &quirk_no_rfkill_wapf4,
+ .driver_data = &quirk_asus_wapf4,
},
{
.callback = dmi_matched,
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 5/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus N552VW" |
| Message-ID | <t3Ujx-Eh-59@gated-at.bofh.it> |
| In reply to | #1567536 |
This reverts commit 2d735244b798f0c8bf93ace1facfafdc1f7a4e6e.
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-nb-wmi.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c
index 5e088fbef5ad..3a2fb29b1bc4 100644
--- a/drivers/platform/x86/asus-nb-wmi.c
+++ b/drivers/platform/x86/asus-nb-wmi.c
@@ -373,15 +373,6 @@ static const struct dmi_system_id asus_quirks[] = {
},
{
.callback = dmi_matched,
- .ident = "ASUSTeK COMPUTER INC. N552VW",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
- DMI_MATCH(DMI_PRODUCT_NAME, "N552VW"),
- },
- .driver_data = &quirk_no_rfkill,
- },
- {
- .callback = dmi_matched,
.ident = "ASUSTeK COMPUTER INC. UX303UB",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 7/8] asus-wireless: Export ids list |
| Message-ID | <t3Ujx-Eh-61@gated-at.bofh.it> |
| In reply to | #1567536 |
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-wireless.c | 15 ++++++---------
drivers/platform/x86/asus-wireless.h | 10 ++++++++++
2 files changed, 16 insertions(+), 9 deletions(-)
create mode 100644 drivers/platform/x86/asus-wireless.h
diff --git a/drivers/platform/x86/asus-wireless.c b/drivers/platform/x86/asus-wireless.c
index 5a238fad35fb..fa28613688b4 100644
--- a/drivers/platform/x86/asus-wireless.c
+++ b/drivers/platform/x86/asus-wireless.c
@@ -17,6 +17,8 @@
#include <linux/pci_ids.h>
#include <linux/leds.h>
+#include "asus-wireless.h"
+
#define ASUS_WIRELESS_LED_STATUS 0x2
struct hswc_params {
@@ -40,12 +42,7 @@ static const struct hswc_params id_params[] = {
{ 0x5, 0x4 },
};
-static const struct acpi_device_id device_ids[] = {
- {"ATK4001", 0},
- {"ATK4002", 0},
- {"", 0},
-};
-MODULE_DEVICE_TABLE(acpi, device_ids);
+MODULE_DEVICE_TABLE(acpi, asus_wireless_ids);
static u64 asus_wireless_method(acpi_handle handle, const char *method,
int param)
@@ -130,8 +127,8 @@ static int asus_wireless_add(struct acpi_device *adev)
adev->driver_data = data;
hid = acpi_device_hid(adev);
- for (i = 0; strcmp(device_ids[i].id, ""); i++) {
- if (!strcmp(device_ids[i].id, hid)) {
+ for (i = 0; strcmp(asus_wireless_ids[i].id, ""); i++) {
+ if (!strcmp(asus_wireless_ids[i].id, hid)) {
data->hswc_params = &id_params[i];
break;
}
@@ -179,7 +176,7 @@ static int asus_wireless_remove(struct acpi_device *adev)
static struct acpi_driver asus_wireless_driver = {
.name = "Asus Wireless Radio Control Driver",
.class = "hotkey",
- .ids = device_ids,
+ .ids = asus_wireless_ids,
.ops = {
.add = asus_wireless_add,
.remove = asus_wireless_remove,
diff --git a/drivers/platform/x86/asus-wireless.h b/drivers/platform/x86/asus-wireless.h
new file mode 100644
index 000000000000..10a345863da6
--- /dev/null
+++ b/drivers/platform/x86/asus-wireless.h
@@ -0,0 +1,10 @@
+#ifndef _ASUS_WIRELESS_H_
+#define _ASUS_WIRELESS_H_
+
+const struct acpi_device_id asus_wireless_ids[] = {
+ {"ATK4001", 0},
+ {"ATK4002", 0},
+ {"", 0},
+};
+
+#endif /* !_ASUS_WIRELESS_H_ */
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | Andy Shevchenko <andy.shevchenko@gmail.com> |
|---|---|
| Date | 2017-01-27 16:40 +0100 |
| Subject | Re: [PATCH 7/8] asus-wireless: Export ids list |
| Message-ID | <t4gN5-5TS-29@gated-at.bofh.it> |
| In reply to | #1567548 |
On Thu, Jan 26, 2017 at 5:30 PM, João Paulo Rechi Vita
<jprvita@gmail.com> wrote:
> Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
> ---
> drivers/platform/x86/asus-wireless.c | 15 ++++++---------
> drivers/platform/x86/asus-wireless.h | 10 ++++++++++
> 2 files changed, 16 insertions(+), 9 deletions(-)
> create mode 100644 drivers/platform/x86/asus-wireless.h
>
> diff --git a/drivers/platform/x86/asus-wireless.c b/drivers/platform/x86/asus-wireless.c
> index 5a238fad35fb..fa28613688b4 100644
> --- a/drivers/platform/x86/asus-wireless.c
> +++ b/drivers/platform/x86/asus-wireless.c
> @@ -17,6 +17,8 @@
> #include <linux/pci_ids.h>
> #include <linux/leds.h>
>
> +#include "asus-wireless.h"
> +
> #define ASUS_WIRELESS_LED_STATUS 0x2
>
> struct hswc_params {
> @@ -40,12 +42,7 @@ static const struct hswc_params id_params[] = {
> { 0x5, 0x4 },
> };
>
> -static const struct acpi_device_id device_ids[] = {
> - {"ATK4001", 0},
> - {"ATK4002", 0},
> - {"", 0},
> -};
> -MODULE_DEVICE_TABLE(acpi, device_ids);
> +MODULE_DEVICE_TABLE(acpi, asus_wireless_ids);
>
No, Don't do this.
> static u64 asus_wireless_method(acpi_handle handle, const char *method,
> int param)
> @@ -130,8 +127,8 @@ static int asus_wireless_add(struct acpi_device *adev)
> adev->driver_data = data;
>
> hid = acpi_device_hid(adev);
> - for (i = 0; strcmp(device_ids[i].id, ""); i++) {
This is wrong.
> - if (!strcmp(device_ids[i].id, hid)) {
> + for (i = 0; strcmp(asus_wireless_ids[i].id, ""); i++) {
This is too.
Potential infinite loop.
On top of that seems you just introduced this by previous patches and
changing here.
Often it means you need to reconsider how you actually split the
series on logical pieces.
> + if (!strcmp(asus_wireless_ids[i].id, hid)) {
> data->hswc_params = &id_params[i];
> break;
> }
> @@ -179,7 +176,7 @@ static int asus_wireless_remove(struct acpi_device *adev)
> static struct acpi_driver asus_wireless_driver = {
> .name = "Asus Wireless Radio Control Driver",
> .class = "hotkey",
> - .ids = device_ids,
> + .ids = asus_wireless_ids,
> .ops = {
> .add = asus_wireless_add,
> .remove = asus_wireless_remove,
> diff --git a/drivers/platform/x86/asus-wireless.h b/drivers/platform/x86/asus-wireless.h
> new file mode 100644
> index 000000000000..10a345863da6
> --- /dev/null
> +++ b/drivers/platform/x86/asus-wireless.h
> @@ -0,0 +1,10 @@
> +#ifndef _ASUS_WIRELESS_H_
> +#define _ASUS_WIRELESS_H_
> +
> +const struct acpi_device_id asus_wireless_ids[] = {
> + {"ATK4001", 0},
> + {"ATK4002", 0},
> + {"", 0},
> +};
> +
> +#endif /* !_ASUS_WIRELESS_H_ */
> --
> 2.11.0
>
--
With Best Regards,
Andy Shevchenko
[toc] | [prev] | [next] | [standalone]
| From | "João Paulo Rechi Vita" <jprvita@gmail.com> |
|---|---|
| Date | 2017-01-26 16:40 +0100 |
| Subject | [PATCH 3/8] Revert "asus-wmi: Add quirk_no_rfkill for the Asus Z550MA" |
| Message-ID | <t3Ujx-Eh-69@gated-at.bofh.it> |
| In reply to | #1567536 |
This reverts commit 6b7ff2af5286a8c6bec7ff5f4df62e3506c1674e.
Signed-off-by: João Paulo Rechi Vita <jprvita@endlessm.com>
---
drivers/platform/x86/asus-nb-wmi.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c
index 5bf1dd225788..9d686d642934 100644
--- a/drivers/platform/x86/asus-nb-wmi.c
+++ b/drivers/platform/x86/asus-nb-wmi.c
@@ -391,15 +391,6 @@ static const struct dmi_system_id asus_quirks[] = {
},
{
.callback = dmi_matched,
- .ident = "ASUSTeK COMPUTER INC. Z550MA",
- .matches = {
- DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
- DMI_MATCH(DMI_PRODUCT_NAME, "Z550MA"),
- },
- .driver_data = &quirk_no_rfkill,
- },
- {
- .callback = dmi_matched,
.ident = "ASUSTeK COMPUTER INC. UX303UB",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
--
2.11.0
[toc] | [prev] | [next] | [standalone]
| From | Andy Shevchenko <andy.shevchenko@gmail.com> |
|---|---|
| Date | 2017-01-27 16:40 +0100 |
| Message-ID | <t4gN4-5TS-11@gated-at.bofh.it> |
| In reply to | #1567536 |
On Thu, Jan 26, 2017 at 5:30 PM, João Paulo Rechi Vita <jprvita@gmail.com> wrote: > This quirk avoids the conflicting usage of ASUS_WMI_DEVID_WLAN_LED (0x00010002) > by both asus-wireless and asus-wmi on machines where the BIOS can't set and > record the wlan status (wlan_ctrl_by_user in asus-wmi.c). At the moment we have > six models using that quirk upstream, but there are another fifteen for which > we have downstream patches, and will send upstream unless we find a more > dynamic solution. We also expect this list to continue growing. > > This series makes use of ASUS_WMI_DSTS_USER_BIT plus the status of the ASHS > device to skip asus_wmi_rfkill_init(). Since asus-wmi now needs to know the ASHS > _HIDs, I have moved device_ids[] to a new header file, asus-wireless.h. I'm not > sure if there are any best practices in the kernel community against having one > module including headers from a different module, but I wanted to avoid > duplicating this list or creating an ordering dependency between the modules. > Any advice on alternatives here are welcome. > > PS: I don't think asus-wireless.h needs to also be listed in MAINTAINERS, > please correct me if I'm wrong. > > João Paulo Rechi Vita (8): > Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UA" > Revert "asus-wmi: Add quirk_no_rfkill_wapf4 for the Asus X456UF" > Revert "asus-wmi: Add quirk_no_rfkill for the Asus Z550MA" > Revert "asus-wmi: Add quirk_no_rfkill for the Asus U303LB" > Revert "asus-wmi: Add quirk_no_rfkill for the Asus N552VW" > Revert "asus-wmi: Create quirk for airplane_mode LED" Don't do this. Each of them will break the working case -> means regression on user experience. > asus-wireless: Export ids list > asus-wmi: Don't register rfkill if ASHS and user bit are present > > drivers/platform/x86/asus-nb-wmi.c | 49 ++---------------------------------- > drivers/platform/x86/asus-wireless.c | 15 +++++------ > drivers/platform/x86/asus-wireless.h | 10 ++++++++ > drivers/platform/x86/asus-wmi.c | 21 ++++++++++++---- > drivers/platform/x86/asus-wmi.h | 1 - > 5 files changed, 34 insertions(+), 62 deletions(-) > create mode 100644 drivers/platform/x86/asus-wireless.h > > -- > 2.11.0 > -- With Best Regards, Andy Shevchenko
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web