* [PATCH] pwm: lpss: Add ACPI HID for second PWM controller on Cherry Trail devices
@ 2018-08-27 7:52 Hans de Goede
2018-08-29 16:45 ` Andy Shevchenko
0 siblings, 1 reply; 2+ messages in thread
From: Hans de Goede @ 2018-08-27 7:52 UTC (permalink / raw)
To: Thierry Reding, Andy Shevchenko; +Cc: Hans de Goede, linux-pwm
The second PWM controller on Cherry Trail devices uses a separate ACPI
HID: "80862289", add this so that the driver will properly bind to the
second PWM controller.
The second PWM controller is usually not used, the main thing gained by
this is properly putting the PWM controller in D3 on suspend.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/pwm/pwm-lpss-platform.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pwm/pwm-lpss-platform.c b/drivers/pwm/pwm-lpss-platform.c
index 5561b9e190f8..7304f36ee715 100644
--- a/drivers/pwm/pwm-lpss-platform.c
+++ b/drivers/pwm/pwm-lpss-platform.c
@@ -81,6 +81,7 @@ static SIMPLE_DEV_PM_OPS(pwm_lpss_platform_pm_ops,
static const struct acpi_device_id pwm_lpss_acpi_match[] = {
{ "80860F09", (unsigned long)&pwm_lpss_byt_info },
{ "80862288", (unsigned long)&pwm_lpss_bsw_info },
+ { "80862289", (unsigned long)&pwm_lpss_bsw_info },
{ "80865AC8", (unsigned long)&pwm_lpss_bxt_info },
{ },
};
--
2.18.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] pwm: lpss: Add ACPI HID for second PWM controller on Cherry Trail devices
2018-08-27 7:52 [PATCH] pwm: lpss: Add ACPI HID for second PWM controller on Cherry Trail devices Hans de Goede
@ 2018-08-29 16:45 ` Andy Shevchenko
0 siblings, 0 replies; 2+ messages in thread
From: Andy Shevchenko @ 2018-08-29 16:45 UTC (permalink / raw)
To: Hans de Goede; +Cc: Thierry Reding, linux-pwm
On Mon, Aug 27, 2018 at 09:52:33AM +0200, Hans de Goede wrote:
> The second PWM controller on Cherry Trail devices uses a separate ACPI
> HID: "80862289", add this so that the driver will properly bind to the
> second PWM controller.
>
> The second PWM controller is usually not used, the main thing gained by
> this is properly putting the PWM controller in D3 on suspend.
>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> drivers/pwm/pwm-lpss-platform.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/pwm/pwm-lpss-platform.c b/drivers/pwm/pwm-lpss-platform.c
> index 5561b9e190f8..7304f36ee715 100644
> --- a/drivers/pwm/pwm-lpss-platform.c
> +++ b/drivers/pwm/pwm-lpss-platform.c
> @@ -81,6 +81,7 @@ static SIMPLE_DEV_PM_OPS(pwm_lpss_platform_pm_ops,
> static const struct acpi_device_id pwm_lpss_acpi_match[] = {
> { "80860F09", (unsigned long)&pwm_lpss_byt_info },
> { "80862288", (unsigned long)&pwm_lpss_bsw_info },
> + { "80862289", (unsigned long)&pwm_lpss_bsw_info },
> { "80865AC8", (unsigned long)&pwm_lpss_bxt_info },
> { },
> };
> --
> 2.18.0
>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-08-29 16:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-27 7:52 [PATCH] pwm: lpss: Add ACPI HID for second PWM controller on Cherry Trail devices Hans de Goede
2018-08-29 16:45 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).