Tiger Lake's new unique ACPI device IDs for DPTF and fan drivers are not
valid as the IDs are missing 'C'. Fix the IDs by updating them.
After the update, the new IDs should now look like
INT1047 --> INTC1047
INT1040 --> INTC1040
INT1043 --> INTC1043
INT1044 --> INTC1044
Fixes: 55cfe6a5c582 ("ACPI: DPTF: Add Tiger Lake ACPI device IDs")
Fixes: c248dfe7e0ca ("ACPI: fan: Add Tiger Lake ACPI device ID")
Cc: 5.6+ <[email protected]> # 5.6+
Cc: Andy Shevchenko <[email protected]>
Cc: Srinivas Pandruvada <[email protected]>
Cc: Rafael J. Wysocki <[email protected]>
Suggested-by: Srinivas Pandruvada <[email protected]>
Signed-off-by: Gayatri Kammela <[email protected]>
---
drivers/acpi/device_pm.c | 2 +-
drivers/acpi/dptf/dptf_power.c | 2 +-
drivers/acpi/dptf/int340x_thermal.c | 8 ++++----
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c
index b64c62bfcea5..80dae3b3c36a 100644
--- a/drivers/acpi/device_pm.c
+++ b/drivers/acpi/device_pm.c
@@ -1321,7 +1321,7 @@ int acpi_dev_pm_attach(struct device *dev, bool power_on)
*/
static const struct acpi_device_id special_pm_ids[] = {
{"PNP0C0B", }, /* Generic ACPI fan */
- {"INT1044", }, /* Fan for Tiger Lake generation */
+ {"INTC1044", }, /* Fan for Tiger Lake generation */
{"INT3404", }, /* Fan */
{}
};
diff --git a/drivers/acpi/dptf/dptf_power.c b/drivers/acpi/dptf/dptf_power.c
index 387f27ef3368..e5fb34bfa52c 100644
--- a/drivers/acpi/dptf/dptf_power.c
+++ b/drivers/acpi/dptf/dptf_power.c
@@ -97,7 +97,7 @@ static int dptf_power_remove(struct platform_device *pdev)
}
static const struct acpi_device_id int3407_device_ids[] = {
- {"INT1047", 0},
+ {"INTC1047", 0},
{"INT3407", 0},
{"", 0},
};
diff --git a/drivers/acpi/dptf/int340x_thermal.c b/drivers/acpi/dptf/int340x_thermal.c
index 1ec7b6900662..29b5c77256dd 100644
--- a/drivers/acpi/dptf/int340x_thermal.c
+++ b/drivers/acpi/dptf/int340x_thermal.c
@@ -13,10 +13,10 @@
#define INT3401_DEVICE 0X01
static const struct acpi_device_id int340x_thermal_device_ids[] = {
- {"INT1040"},
- {"INT1043"},
- {"INT1044"},
- {"INT1047"},
+ {"INTC1040"},
+ {"INTC1043"},
+ {"INTC1044"},
+ {"INTC1047"},
{"INT3400"},
{"INT3401", INT3401_DEVICE},
{"INT3402"},
--
2.17.1
On Fri, Mar 27, 2020 at 10:34 PM Gayatri Kammela
<[email protected]> wrote:
>
> Tiger Lake's new unique ACPI device IDs for DPTF and fan drivers are not
> valid as the IDs are missing 'C'. Fix the IDs by updating them.
>
> After the update, the new IDs should now look like
> INT1047 --> INTC1047
> INT1040 --> INTC1040
> INT1043 --> INTC1043
> INT1044 --> INTC1044
>
> Fixes: 55cfe6a5c582 ("ACPI: DPTF: Add Tiger Lake ACPI device IDs")
> Fixes: c248dfe7e0ca ("ACPI: fan: Add Tiger Lake ACPI device ID")
> Cc: 5.6+ <[email protected]> # 5.6+
> Cc: Andy Shevchenko <[email protected]>
> Cc: Srinivas Pandruvada <[email protected]>
> Cc: Rafael J. Wysocki <[email protected]>
> Suggested-by: Srinivas Pandruvada <[email protected]>
> Signed-off-by: Gayatri Kammela <[email protected]>
> ---
> drivers/acpi/device_pm.c | 2 +-
> drivers/acpi/dptf/dptf_power.c | 2 +-
> drivers/acpi/dptf/int340x_thermal.c | 8 ++++----
> 3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c
> index b64c62bfcea5..80dae3b3c36a 100644
> --- a/drivers/acpi/device_pm.c
> +++ b/drivers/acpi/device_pm.c
> @@ -1321,7 +1321,7 @@ int acpi_dev_pm_attach(struct device *dev, bool power_on)
> */
> static const struct acpi_device_id special_pm_ids[] = {
> {"PNP0C0B", }, /* Generic ACPI fan */
> - {"INT1044", }, /* Fan for Tiger Lake generation */
> + {"INTC1044", }, /* Fan for Tiger Lake generation */
> {"INT3404", }, /* Fan */
> {}
> };
> diff --git a/drivers/acpi/dptf/dptf_power.c b/drivers/acpi/dptf/dptf_power.c
> index 387f27ef3368..e5fb34bfa52c 100644
> --- a/drivers/acpi/dptf/dptf_power.c
> +++ b/drivers/acpi/dptf/dptf_power.c
> @@ -97,7 +97,7 @@ static int dptf_power_remove(struct platform_device *pdev)
> }
>
> static const struct acpi_device_id int3407_device_ids[] = {
> - {"INT1047", 0},
> + {"INTC1047", 0},
> {"INT3407", 0},
> {"", 0},
> };
> diff --git a/drivers/acpi/dptf/int340x_thermal.c b/drivers/acpi/dptf/int340x_thermal.c
> index 1ec7b6900662..29b5c77256dd 100644
> --- a/drivers/acpi/dptf/int340x_thermal.c
> +++ b/drivers/acpi/dptf/int340x_thermal.c
> @@ -13,10 +13,10 @@
>
> #define INT3401_DEVICE 0X01
> static const struct acpi_device_id int340x_thermal_device_ids[] = {
> - {"INT1040"},
> - {"INT1043"},
> - {"INT1044"},
> - {"INT1047"},
> + {"INTC1040"},
> + {"INTC1043"},
> + {"INTC1044"},
> + {"INTC1047"},
> {"INT3400"},
> {"INT3401", INT3401_DEVICE},
> {"INT3402"},
> --
I can take this along with the other two patches in the series if that
is fine by Andy and Rui.
Thanks!
On Mon, Mar 30, 2020 at 06:43:35PM +0200, Rafael J. Wysocki wrote:
> On Fri, Mar 27, 2020 at 10:34 PM Gayatri Kammela
> <[email protected]> wrote:
> > - {"INT1044"},
> > - {"INT1047"},
> > + {"INTC1040"},
> > + {"INTC1043"},
> > + {"INTC1044"},
> > + {"INTC1047"},
> > {"INT3400"},
> > {"INT3401", INT3401_DEVICE},
> > {"INT3402"},
> > --
>
> I can take this along with the other two patches in the series if that
> is fine by Andy and Rui.
One nit is to fix the ordering to be alphanumeric or close enough
(I admit in some cases it might require unneeded churn) to that.
Otherwise,
Reviewed-by: Andy Shevchenko <[email protected]>
--
With Best Regards,
Andy Shevchenko
> -----Original Message-----
> From: Andy Shevchenko <[email protected]>
> Sent: Monday, March 30, 2020 10:25 AM
> To: Rafael J. Wysocki <[email protected]>
> Cc: Kammela, Gayatri <[email protected]>; Zhang, Rui
> <[email protected]>; Linux PM <[email protected]>; Platform
> Driver <[email protected]>; Linux Kernel Mailing List
> <[email protected]>; Len Brown <[email protected]>; Darren Hart
> <[email protected]>; Alex Hung <[email protected]>; Daniel
> Lezcano <[email protected]>; Amit Kucheria
> <[email protected]>; Westerberg, Mika
> <[email protected]>; Peter Zijlstra <[email protected]>;
> Prestopine, Charles D <[email protected]>; 5 . 6+
> <[email protected]>; Pandruvada, Srinivas
> <[email protected]>; Wysocki, Rafael J
> <[email protected]>
> Subject: Re: [PATCH v2 1/3] ACPI: fix: Update Tiger Lake ACPI device IDs
>
> On Mon, Mar 30, 2020 at 06:43:35PM +0200, Rafael J. Wysocki wrote:
> > On Fri, Mar 27, 2020 at 10:34 PM Gayatri Kammela
> > <[email protected]> wrote:
>
> > > - {"INT1044"},
> > > - {"INT1047"},
> > > + {"INTC1040"},
> > > + {"INTC1043"},
> > > + {"INTC1044"},
> > > + {"INTC1047"},
> > > {"INT3400"},
> > > {"INT3401", INT3401_DEVICE},
> > > {"INT3402"},
> > > --
> >
> > I can take this along with the other two patches in the series if that
> > is fine by Andy and Rui.
>
> One nit is to fix the ordering to be alphanumeric or close enough (I admit in
> some cases it might require unneeded churn) to that.
Thanks Andy and Rafael! Should I send v3 for this series with right ordering this time?
>
> Otherwise,
> Reviewed-by: Andy Shevchenko <[email protected]>
>
> --
> With Best Regards,
> Andy Shevchenko
>
On Tue, Mar 31, 2020 at 1:22 AM Kammela, Gayatri
<[email protected]> wrote:
>
> > -----Original Message-----
> > From: Andy Shevchenko <[email protected]>
> > Sent: Monday, March 30, 2020 10:25 AM
> > To: Rafael J. Wysocki <[email protected]>
> > Cc: Kammela, Gayatri <[email protected]>; Zhang, Rui
> > <[email protected]>; Linux PM <[email protected]>; Platform
> > Driver <[email protected]>; Linux Kernel Mailing List
> > <[email protected]>; Len Brown <[email protected]>; Darren Hart
> > <[email protected]>; Alex Hung <[email protected]>; Daniel
> > Lezcano <[email protected]>; Amit Kucheria
> > <[email protected]>; Westerberg, Mika
> > <[email protected]>; Peter Zijlstra <[email protected]>;
> > Prestopine, Charles D <[email protected]>; 5 . 6+
> > <[email protected]>; Pandruvada, Srinivas
> > <[email protected]>; Wysocki, Rafael J
> > <[email protected]>
> > Subject: Re: [PATCH v2 1/3] ACPI: fix: Update Tiger Lake ACPI device IDs
> >
> > On Mon, Mar 30, 2020 at 06:43:35PM +0200, Rafael J. Wysocki wrote:
> > > On Fri, Mar 27, 2020 at 10:34 PM Gayatri Kammela
> > > <[email protected]> wrote:
> >
> > > > - {"INT1044"},
> > > > - {"INT1047"},
> > > > + {"INTC1040"},
> > > > + {"INTC1043"},
> > > > + {"INTC1044"},
> > > > + {"INTC1047"},
> > > > {"INT3400"},
> > > > {"INT3401", INT3401_DEVICE},
> > > > {"INT3402"},
> > > > --
> > >
> > > I can take this along with the other two patches in the series if that
> > > is fine by Andy and Rui.
> >
> > One nit is to fix the ordering to be alphanumeric or close enough (I admit in
> > some cases it might require unneeded churn) to that.
> Thanks Andy and Rafael! Should I send v3 for this series with right ordering this time?
No need, I can fix up the ordering just fine.
> > Otherwise,
> > Reviewed-by: Andy Shevchenko <[email protected]>
Thanks!
On Tue, Mar 31, 2020 at 8:19 AM Rafael J. Wysocki <[email protected]> wrote:
>
> On Tue, Mar 31, 2020 at 1:22 AM Kammela, Gayatri
> <[email protected]> wrote:
> >
> > > -----Original Message-----
> > > From: Andy Shevchenko <[email protected]>
> > > Sent: Monday, March 30, 2020 10:25 AM
> > > To: Rafael J. Wysocki <[email protected]>
> > > Cc: Kammela, Gayatri <[email protected]>; Zhang, Rui
> > > <[email protected]>; Linux PM <[email protected]>; Platform
> > > Driver <[email protected]>; Linux Kernel Mailing List
> > > <[email protected]>; Len Brown <[email protected]>; Darren Hart
> > > <[email protected]>; Alex Hung <[email protected]>; Daniel
> > > Lezcano <[email protected]>; Amit Kucheria
> > > <[email protected]>; Westerberg, Mika
> > > <[email protected]>; Peter Zijlstra <[email protected]>;
> > > Prestopine, Charles D <[email protected]>; 5 . 6+
> > > <[email protected]>; Pandruvada, Srinivas
> > > <[email protected]>; Wysocki, Rafael J
> > > <[email protected]>
> > > Subject: Re: [PATCH v2 1/3] ACPI: fix: Update Tiger Lake ACPI device IDs
> > >
> > > On Mon, Mar 30, 2020 at 06:43:35PM +0200, Rafael J. Wysocki wrote:
> > > > On Fri, Mar 27, 2020 at 10:34 PM Gayatri Kammela
> > > > <[email protected]> wrote:
> > >
> > > > > - {"INT1044"},
> > > > > - {"INT1047"},
> > > > > + {"INTC1040"},
> > > > > + {"INTC1043"},
> > > > > + {"INTC1044"},
> > > > > + {"INTC1047"},
> > > > > {"INT3400"},
> > > > > {"INT3401", INT3401_DEVICE},
> > > > > {"INT3402"},
> > > > > --
> > > >
> > > > I can take this along with the other two patches in the series if that
> > > > is fine by Andy and Rui.
> > >
> > > One nit is to fix the ordering to be alphanumeric or close enough (I admit in
> > > some cases it might require unneeded churn) to that.
> > Thanks Andy and Rafael! Should I send v3 for this series with right ordering this time?
>
> No need, I can fix up the ordering just fine.
>
> > > Otherwise,
> > > Reviewed-by: Andy Shevchenko <[email protected]>
>
> Thanks!
Now applied as 5.7 material with the changes mentioned above.
Please check the bleeding-edge branch in my tree.