From: Rafael J. Wysocki <[email protected]>
Use capitals in the names of the board ID symbols and add the PCH_
prefix to each of them for consistency.
Also rename the board_ids enum accordingly.
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <[email protected]>
---
drivers/thermal/intel/intel_pch_thermal.c | 54 +++++++++++++++---------------
1 file changed, 27 insertions(+), 27 deletions(-)
Index: linux-pm/drivers/thermal/intel/intel_pch_thermal.c
===================================================================
--- linux-pm.orig/drivers/thermal/intel/intel_pch_thermal.c
+++ linux-pm/drivers/thermal/intel/intel_pch_thermal.c
@@ -135,38 +135,38 @@ static struct thermal_zone_device_ops tz
.critical = pch_critical,
};
-enum board_ids {
- board_hsw,
- board_wpt,
- board_skl,
- board_cnl,
- board_cml,
- board_lwb,
- board_wbg,
+enum pch_board_ids {
+ PCH_BOARD_HSW = 0,
+ PCH_BOARD_WPT,
+ PCH_BOARD_SKL,
+ PCH_BOARD_CNL,
+ PCH_BOARD_CML,
+ PCH_BOARD_LWB,
+ PCH_BOARD_WBG,
};
static const struct board_info {
const char *name;
} board_info[] = {
- [board_hsw] = {
+ [PCH_BOARD_HSW] = {
.name = "pch_haswell",
},
- [board_wpt] = {
+ [PCH_BOARD_WPT] = {
.name = "pch_wildcat_point",
},
- [board_skl] = {
+ [PCH_BOARD_SKL] = {
.name = "pch_skylake",
},
- [board_cnl] = {
+ [PCH_BOARD_CNL] = {
.name = "pch_cannonlake",
},
- [board_cml] = {
+ [PCH_BOARD_CML] = {
.name = "pch_cometlake",
},
- [board_lwb] = {
+ [PCH_BOARD_LWB] = {
.name = "pch_lewisburg",
},
- [board_wbg] = {
+ [PCH_BOARD_WBG] = {
.name = "pch_wellsburg",
},
};
@@ -174,7 +174,7 @@ static const struct board_info {
static int intel_pch_thermal_probe(struct pci_dev *pdev,
const struct pci_device_id *id)
{
- enum board_ids board_id = id->driver_data;
+ enum pch_board_ids board_id = id->driver_data;
const struct board_info *bi = &board_info[board_id];
struct pch_thermal_device *ptd;
u16 trip_temp;
@@ -372,27 +372,27 @@ static int intel_pch_thermal_resume(stru
static const struct pci_device_id intel_pch_thermal_id[] = {
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_HSW_1),
- .driver_data = board_hsw, },
+ .driver_data = PCH_BOARD_HSW, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_HSW_2),
- .driver_data = board_hsw, },
+ .driver_data = PCH_BOARD_HSW, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_WPT),
- .driver_data = board_wpt, },
+ .driver_data = PCH_BOARD_WPT, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_SKL),
- .driver_data = board_skl, },
+ .driver_data = PCH_BOARD_SKL, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_SKL_H),
- .driver_data = board_skl, },
+ .driver_data = PCH_BOARD_SKL, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL),
- .driver_data = board_cnl, },
+ .driver_data = PCH_BOARD_CNL, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL_H),
- .driver_data = board_cnl, },
+ .driver_data = PCH_BOARD_CNL, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL_LP),
- .driver_data = board_cnl, },
+ .driver_data = PCH_BOARD_CNL, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CML_H),
- .driver_data = board_cml, },
+ .driver_data = PCH_BOARD_CML, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_LWB),
- .driver_data = board_lwb, },
+ .driver_data = PCH_BOARD_LWB, },
{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_WBG),
- .driver_data = board_wbg, },
+ .driver_data = PCH_BOARD_WBG, },
{ 0, },
};
MODULE_DEVICE_TABLE(pci, intel_pch_thermal_id);
On Mon, 2023-01-30 at 20:04 +0100, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <[email protected]>
>
> Use capitals in the names of the board ID symbols and add the PCH_
> prefix to each of them for consistency.
>
> Also rename the board_ids enum accordingly.
>
> No intentional functional impact.
>
> Signed-off-by: Rafael J. Wysocki <[email protected]>
> ---
> drivers/thermal/intel/intel_pch_thermal.c | 54 +++++++++++++++--
> -------------
> 1 file changed, 27 insertions(+), 27 deletions(-)
>
> Index: linux-pm/drivers/thermal/intel/intel_pch_thermal.c
> ===================================================================
> --- linux-pm.orig/drivers/thermal/intel/intel_pch_thermal.c
> +++ linux-pm/drivers/thermal/intel/intel_pch_thermal.c
> @@ -135,38 +135,38 @@ static struct thermal_zone_device_ops tz
> .critical = pch_critical,
> };
>
> -enum board_ids {
> - board_hsw,
> - board_wpt,
> - board_skl,
> - board_cnl,
> - board_cml,
> - board_lwb,
> - board_wbg,
> +enum pch_board_ids {
> + PCH_BOARD_HSW = 0,
> + PCH_BOARD_WPT,
> + PCH_BOARD_SKL,
> + PCH_BOARD_CNL,
> + PCH_BOARD_CML,
> + PCH_BOARD_LWB,
> + PCH_BOARD_WBG,
> };
>
> static const struct board_info {
> const char *name;
> } board_info[] = {
Now struct board_info has "name" field only, so maybe we can remove
struct board_info, and use a "static const char *" array instead?
BTW, I'm building a kernel with this patch series as well as
https://patchwork.kernel.org/project/linux-pm/list/?series=717084,
will update the test result later.
thanks,
rui
> - [board_hsw] = {
> + [PCH_BOARD_HSW] = {
> .name = "pch_haswell",
> },
> - [board_wpt] = {
> + [PCH_BOARD_WPT] = {
> .name = "pch_wildcat_point",
> },
> - [board_skl] = {
> + [PCH_BOARD_SKL] = {
> .name = "pch_skylake",
> },
> - [board_cnl] = {
> + [PCH_BOARD_CNL] = {
> .name = "pch_cannonlake",
> },
> - [board_cml] = {
> + [PCH_BOARD_CML] = {
> .name = "pch_cometlake",
> },
> - [board_lwb] = {
> + [PCH_BOARD_LWB] = {
> .name = "pch_lewisburg",
> },
> - [board_wbg] = {
> + [PCH_BOARD_WBG] = {
> .name = "pch_wellsburg",
> },
> };
> @@ -174,7 +174,7 @@ static const struct board_info {
> static int intel_pch_thermal_probe(struct pci_dev *pdev,
> const struct pci_device_id *id)
> {
> - enum board_ids board_id = id->driver_data;
> + enum pch_board_ids board_id = id->driver_data;
> const struct board_info *bi = &board_info[board_id];
> struct pch_thermal_device *ptd;
> u16 trip_temp;
> @@ -372,27 +372,27 @@ static int intel_pch_thermal_resume(stru
>
> static const struct pci_device_id intel_pch_thermal_id[] = {
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_HSW_1),
> - .driver_data = board_hsw, },
> + .driver_data = PCH_BOARD_HSW, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_HSW_2),
> - .driver_data = board_hsw, },
> + .driver_data = PCH_BOARD_HSW, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_WPT),
> - .driver_data = board_wpt, },
> + .driver_data = PCH_BOARD_WPT, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_SKL),
> - .driver_data = board_skl, },
> + .driver_data = PCH_BOARD_SKL, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_SKL_H),
> - .driver_data = board_skl, },
> + .driver_data = PCH_BOARD_SKL, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL),
> - .driver_data = board_cnl, },
> + .driver_data = PCH_BOARD_CNL, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL_H),
> - .driver_data = board_cnl, },
> + .driver_data = PCH_BOARD_CNL, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CNL_LP),
> - .driver_data = board_cnl, },
> + .driver_data = PCH_BOARD_CNL, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_CML_H),
> - .driver_data = board_cml, },
> + .driver_data = PCH_BOARD_CML, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_LWB),
> - .driver_data = board_lwb, },
> + .driver_data = PCH_BOARD_LWB, },
> { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCH_THERMAL_DID_WBG),
> - .driver_data = board_wbg, },
> + .driver_data = PCH_BOARD_WBG, },
> { 0, },
> };
> MODULE_DEVICE_TABLE(pci, intel_pch_thermal_id);
>
>
>
On Tuesday, January 31, 2023 12:17:55 PM CET Zhang, Rui wrote:
> On Mon, 2023-01-30 at 20:04 +0100, Rafael J. Wysocki wrote:
> > From: Rafael J. Wysocki <[email protected]>
> >
> > Use capitals in the names of the board ID symbols and add the PCH_
> > prefix to each of them for consistency.
> >
> > Also rename the board_ids enum accordingly.
> >
> > No intentional functional impact.
> >
> > Signed-off-by: Rafael J. Wysocki <[email protected]>
> > ---
> > drivers/thermal/intel/intel_pch_thermal.c | 54 +++++++++++++++--
> > -------------
> > 1 file changed, 27 insertions(+), 27 deletions(-)
> >
> > Index: linux-pm/drivers/thermal/intel/intel_pch_thermal.c
> > ===================================================================
> > --- linux-pm.orig/drivers/thermal/intel/intel_pch_thermal.c
> > +++ linux-pm/drivers/thermal/intel/intel_pch_thermal.c
> > @@ -135,38 +135,38 @@ static struct thermal_zone_device_ops tz
> > .critical = pch_critical,
> > };
> >
> > -enum board_ids {
> > - board_hsw,
> > - board_wpt,
> > - board_skl,
> > - board_cnl,
> > - board_cml,
> > - board_lwb,
> > - board_wbg,
> > +enum pch_board_ids {
> > + PCH_BOARD_HSW = 0,
> > + PCH_BOARD_WPT,
> > + PCH_BOARD_SKL,
> > + PCH_BOARD_CNL,
> > + PCH_BOARD_CML,
> > + PCH_BOARD_LWB,
> > + PCH_BOARD_WBG,
> > };
> >
> > static const struct board_info {
> > const char *name;
> > } board_info[] = {
>
> Now struct board_info has "name" field only, so maybe we can remove
> struct board_info, and use a "static const char *" array instead?
Good point.
I think that the last patch in the series can be replaced with the
appended one.
> BTW, I'm building a kernel with this patch series as well as
> https://patchwork.kernel.org/project/linux-pm/list/?series=717084,
> will update the test result later.
Thank you!
---
From: Rafael J. Wysocki <[email protected]>
Subject: [PATCH] thermal: intel: intel_pch: Drop struct board_info
Because the only member of struct board_info is the name, the
board_info[] array of struct board_info elements can be replaced with
an array of strings.
Modify the code accordingly and drop struct board_info.
No intentional functional impact.
Suggested-by: Zhang Rui <[email protected]>
Signed-off-by: Rafael J. Wysocki <[email protected]>
---
drivers/thermal/intel/intel_pch_thermal.c | 42 +++++++++---------------------
1 file changed, 13 insertions(+), 29 deletions(-)
Index: linux-pm/drivers/thermal/intel/intel_pch_thermal.c
===================================================================
--- linux-pm.orig/drivers/thermal/intel/intel_pch_thermal.c
+++ linux-pm/drivers/thermal/intel/intel_pch_thermal.c
@@ -145,37 +145,20 @@ enum pch_board_ids {
PCH_BOARD_WBG,
};
-static const struct board_info {
- const char *name;
-} board_info[] = {
- [PCH_BOARD_HSW] = {
- .name = "pch_haswell",
- },
- [PCH_BOARD_WPT] = {
- .name = "pch_wildcat_point",
- },
- [PCH_BOARD_SKL] = {
- .name = "pch_skylake",
- },
- [PCH_BOARD_CNL] = {
- .name = "pch_cannonlake",
- },
- [PCH_BOARD_CML] = {
- .name = "pch_cometlake",
- },
- [PCH_BOARD_LWB] = {
- .name = "pch_lewisburg",
- },
- [PCH_BOARD_WBG] = {
- .name = "pch_wellsburg",
- },
+static const char *board_names[] = {
+ [PCH_BOARD_HSW] = "pch_haswell",
+ [PCH_BOARD_WPT] = "pch_wildcat_point",
+ [PCH_BOARD_SKL] = "pch_skylake",
+ [PCH_BOARD_CNL] = "pch_cannonlake",
+ [PCH_BOARD_CML] = "pch_cometlake",
+ [PCH_BOARD_LWB] = "pch_lewisburg",
+ [PCH_BOARD_WBG] = "pch_wellsburg",
};
static int intel_pch_thermal_probe(struct pci_dev *pdev,
const struct pci_device_id *id)
{
enum pch_board_ids board_id = id->driver_data;
- const struct board_info *bi = &board_info[board_id];
struct pch_thermal_device *ptd;
u16 trip_temp;
int nr_trips;
@@ -249,12 +232,13 @@ read_trips:
nr_trips += pch_wpt_add_acpi_psv_trip(ptd, nr_trips);
- ptd->tzd = thermal_zone_device_register_with_trips(bi->name, ptd->trips,
- nr_trips, 0, ptd,
- &tzd_ops, NULL, 0, 0);
+ ptd->tzd = thermal_zone_device_register_with_trips(board_names[board_id],
+ ptd->trips, nr_trips,
+ 0, ptd, &tzd_ops,
+ NULL, 0, 0);
if (IS_ERR(ptd->tzd)) {
dev_err(&pdev->dev, "Failed to register thermal zone %s\n",
- bi->name);
+ board_names[board_id]);
err = PTR_ERR(ptd->tzd);
goto error_cleanup;
}
On 31/01/2023 14:08, Rafael J. Wysocki wrote:
> On Tuesday, January 31, 2023 12:17:55 PM CET Zhang, Rui wrote:
>> On Mon, 2023-01-30 at 20:04 +0100, Rafael J. Wysocki wrote:
>>> From: Rafael J. Wysocki <[email protected]>
>>>
>>> Use capitals in the names of the board ID symbols and add the PCH_
>>> prefix to each of them for consistency.
>>>
>>> Also rename the board_ids enum accordingly.
>>>
>>> No intentional functional impact.
>>>
>>> Signed-off-by: Rafael J. Wysocki <[email protected]>
>>> ---
Acked-by: Daniel Lezcano <[email protected]>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog