Message ID | 20220629225843.332453-3-willmcvicker@google.com (mailing list archive) |
---|---|
State | Rejected |
Headers | show |
Series | Fixes for thermal hwmon registration | expand |
On 6/29/22 15:58, Will McVicker wrote: > From: Guenter Roeck <linux@roeck-us.net> > > [ upstream commit 87743bcf08072b3e1952a0bf5524b2833e667b4c ] > > The thermal subsystem registers a hwmon device without providing chip > information or sysfs attribute groups. While undesirable, it would be > difficult to change. On the other side, it abuses the > hwmon_device_register_with_info API by not providing that information. > Use new API specifically created for the thermal subsystem instead to > let us enforce the 'chip' parameter for other callers of > hwmon_device_register_with_info(). > > Acked-by: Rafael J . Wysocki <rafael@kernel.org> > Signed-off-by: Guenter Roeck <linux@roeck-us.net> NACK. Same reason as 1/2. Guenter > --- > drivers/thermal/thermal_hwmon.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/thermal/thermal_hwmon.c b/drivers/thermal/thermal_hwmon.c > index dd5d8ee37928..b3b229421936 100644 > --- a/drivers/thermal/thermal_hwmon.c > +++ b/drivers/thermal/thermal_hwmon.c > @@ -147,8 +147,8 @@ int thermal_add_hwmon_sysfs(struct thermal_zone_device *tz) > INIT_LIST_HEAD(&hwmon->tz_list); > strlcpy(hwmon->type, tz->type, THERMAL_NAME_LENGTH); > strreplace(hwmon->type, '-', '_'); > - hwmon->device = hwmon_device_register_with_info(&tz->device, hwmon->type, > - hwmon, NULL, NULL); > + hwmon->device = hwmon_device_register_for_thermal(&tz->device, > + hwmon->type, hwmon); > if (IS_ERR(hwmon->device)) { > result = PTR_ERR(hwmon->device); > goto free_mem;
diff --git a/drivers/thermal/thermal_hwmon.c b/drivers/thermal/thermal_hwmon.c index dd5d8ee37928..b3b229421936 100644 --- a/drivers/thermal/thermal_hwmon.c +++ b/drivers/thermal/thermal_hwmon.c @@ -147,8 +147,8 @@ int thermal_add_hwmon_sysfs(struct thermal_zone_device *tz) INIT_LIST_HEAD(&hwmon->tz_list); strlcpy(hwmon->type, tz->type, THERMAL_NAME_LENGTH); strreplace(hwmon->type, '-', '_'); - hwmon->device = hwmon_device_register_with_info(&tz->device, hwmon->type, - hwmon, NULL, NULL); + hwmon->device = hwmon_device_register_for_thermal(&tz->device, + hwmon->type, hwmon); if (IS_ERR(hwmon->device)) { result = PTR_ERR(hwmon->device); goto free_mem;