diff mbox series

[v2,2/4] acpi: fan: Add new Tiger Lake hardware ID to support fan driver in acpi

Message ID 3c1ed2c9964c63fc956ec249d7bfce8d49f53fee.1576260216.git.gayatri.kammela@intel.com (mailing list archive)
State Superseded, archived
Headers show
Series drivers: Add Tiger Lake hardware IDs to support acpi, | expand

Commit Message

Kammela, Gayatri Dec. 13, 2019, 6:14 p.m. UTC
Tiger Lake has a new unique hardware ID to support fan driver. Hence,
add it.

Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@intel.com>
Signed-off-by: Gayatri Kammela <gayatri.kammela@intel.com>
---
 drivers/acpi/fan.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Rafael J. Wysocki Dec. 16, 2019, 8:58 a.m. UTC | #1
On Fri, Dec 13, 2019 at 9:40 PM Gayatri Kammela
<gayatri.kammela@intel.com> wrote:
>
> Tiger Lake has a new unique hardware ID to support fan driver. Hence,
> add it.
>
> Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Srinivas Pandruvada <srinivas.pandruvada@intel.com>
> Signed-off-by: Gayatri Kammela <gayatri.kammela@intel.com>
> ---
>  drivers/acpi/fan.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
> index 816b0803f7fb..13708fb48038 100644
> --- a/drivers/acpi/fan.c
> +++ b/drivers/acpi/fan.c
> @@ -25,6 +25,7 @@ static int acpi_fan_remove(struct platform_device *pdev);
>
>  static const struct acpi_device_id fan_device_ids[] = {
>         {"PNP0C0B", 0},
> +       {"INT1044", 0},

This has to go into device_pm.c:acpi_dev_pm_attach() too, see commit

b9ea0bae260f ACPI: PM: Avoid attaching ACPI PM domain to certain devices

>         {"INT3404", 0},
>         {"", 0},
>  };
> --
Kammela, Gayatri Dec. 16, 2019, 5:13 p.m. UTC | #2
> > Tiger Lake has a new unique hardware ID to support fan driver. Hence,
> > add it.
> >
> > Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > Cc: Zhang Rui <rui.zhang@intel.com>
> > Cc: Srinivas Pandruvada <srinivas.pandruvada@intel.com>
> > Signed-off-by: Gayatri Kammela <gayatri.kammela@intel.com>
> > ---
> >  drivers/acpi/fan.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c index
> > 816b0803f7fb..13708fb48038 100644
> > --- a/drivers/acpi/fan.c
> > +++ b/drivers/acpi/fan.c
> > @@ -25,6 +25,7 @@ static int acpi_fan_remove(struct platform_device
> > *pdev);
> >
> >  static const struct acpi_device_id fan_device_ids[] = {
> >         {"PNP0C0B", 0},
> > +       {"INT1044", 0},
> 
> This has to go into device_pm.c:acpi_dev_pm_attach() too, see commit
> 
> b9ea0bae260f ACPI: PM: Avoid attaching ACPI PM domain to certain devices
Thanks Rafael! I will add it and send v3.
> 
> >         {"INT3404", 0},
> >         {"", 0},
> >  };
> > --
diff mbox series

Patch

diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
index 816b0803f7fb..13708fb48038 100644
--- a/drivers/acpi/fan.c
+++ b/drivers/acpi/fan.c
@@ -25,6 +25,7 @@  static int acpi_fan_remove(struct platform_device *pdev);
 
 static const struct acpi_device_id fan_device_ids[] = {
 	{"PNP0C0B", 0},
+	{"INT1044", 0},
 	{"INT3404", 0},
 	{"", 0},
 };