* [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF
@ 2022-07-10 17:36 Hans de Goede
2022-07-10 17:49 ` Hans de Goede
2022-07-10 18:44 ` Andy Shevchenko
0 siblings, 2 replies; 4+ messages in thread
From: Hans de Goede @ 2022-07-10 17:36 UTC (permalink / raw)
To: Mark Gross, Andy Shevchenko; +Cc: Hans de Goede, platform-driver-x86
Like the Asus T100TA the Asus T100TAF has a camera LED which is always
on by default and both also use the same GPIO for the LED.
Relax the DMI match for the Asus T100TA so that it also matches
the T100TAF.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/platform/x86/intel/atomisp2/led.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/x86/intel/atomisp2/led.c b/drivers/platform/x86/intel/atomisp2/led.c
index 5935dfca166f..10077a61d8c5 100644
--- a/drivers/platform/x86/intel/atomisp2/led.c
+++ b/drivers/platform/x86/intel/atomisp2/led.c
@@ -50,7 +50,8 @@ static const struct dmi_system_id atomisp2_led_systems[] __initconst = {
{
.matches = {
DMI_EXACT_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
- DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "T100TA"),
+ /* Non exact match to also match T100TAF */
+ DMI_MATCH(DMI_PRODUCT_NAME, "T100TA"),
},
.driver_data = &asus_t100ta_lookup,
},
--
2.36.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF
2022-07-10 17:36 [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF Hans de Goede
@ 2022-07-10 17:49 ` Hans de Goede
2022-07-10 18:44 ` Andy Shevchenko
1 sibling, 0 replies; 4+ messages in thread
From: Hans de Goede @ 2022-07-10 17:49 UTC (permalink / raw)
To: Mark Gross, Andy Shevchenko; +Cc: platform-driver-x86
Hi,
On 7/10/22 19:36, Hans de Goede wrote:
> Like the Asus T100TA the Asus T100TAF has a camera LED which is always
> on by default and both also use the same GPIO for the LED.
>
> Relax the DMI match for the Asus T100TA so that it also matches
> the T100TAF.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
I've added this to my review-hans (soon to be for-next) branch now.
Regards,
Hans
> ---
> drivers/platform/x86/intel/atomisp2/led.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/intel/atomisp2/led.c b/drivers/platform/x86/intel/atomisp2/led.c
> index 5935dfca166f..10077a61d8c5 100644
> --- a/drivers/platform/x86/intel/atomisp2/led.c
> +++ b/drivers/platform/x86/intel/atomisp2/led.c
> @@ -50,7 +50,8 @@ static const struct dmi_system_id atomisp2_led_systems[] __initconst = {
> {
> .matches = {
> DMI_EXACT_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> - DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "T100TA"),
> + /* Non exact match to also match T100TAF */
> + DMI_MATCH(DMI_PRODUCT_NAME, "T100TA"),
> },
> .driver_data = &asus_t100ta_lookup,
> },
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF
2022-07-10 17:36 [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF Hans de Goede
2022-07-10 17:49 ` Hans de Goede
@ 2022-07-10 18:44 ` Andy Shevchenko
2022-07-10 18:53 ` Hans de Goede
1 sibling, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2022-07-10 18:44 UTC (permalink / raw)
To: Hans de Goede; +Cc: Mark Gross, Andy Shevchenko, platform-driver-x86
On Sun, Jul 10, 2022 at 07:36:58PM +0200, Hans de Goede wrote:
> Like the Asus T100TA the Asus T100TAF has a camera LED which is always
> on by default and both also use the same GPIO for the LED.
>
> Relax the DMI match for the Asus T100TA so that it also matches
> the T100TAF.
Shouldn't atomisp driver take care about this (presumable in the future)?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF
2022-07-10 18:44 ` Andy Shevchenko
@ 2022-07-10 18:53 ` Hans de Goede
0 siblings, 0 replies; 4+ messages in thread
From: Hans de Goede @ 2022-07-10 18:53 UTC (permalink / raw)
To: Andy Shevchenko; +Cc: Mark Gross, Andy Shevchenko, platform-driver-x86
Hi,
On 7/10/22 20:44, Andy Shevchenko wrote:
> On Sun, Jul 10, 2022 at 07:36:58PM +0200, Hans de Goede wrote:
>> Like the Asus T100TA the Asus T100TAF has a camera LED which is always
>> on by default and both also use the same GPIO for the LED.
>>
>> Relax the DMI match for the Asus T100TA so that it also matches
>> the T100TAF.
>
> Shouldn't atomisp driver take care about this (presumable in the future)?
Eventually yes, for now we have a small driver to at least turn of
the LED on devices where it is on at boot (or to be more precise
gets turned on when running the ACPI INIT method of the asus-wmi
WMI device).
Regards,
Hans
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-07-10 18:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-10 17:36 [PATCH] platform/x86: intel_atomisp2_led: Also turn off the always-on camera LED on the Asus T100TAF Hans de Goede
2022-07-10 17:49 ` Hans de Goede
2022-07-10 18:44 ` Andy Shevchenko
2022-07-10 18:53 ` Hans de Goede
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.