* [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding [not found] <20201214233811.485669-1-daniel.lezcano@linaro.org> @ 2020-12-14 23:38 ` Daniel Lezcano 2021-01-05 15:44 ` Daniel Lezcano 0 siblings, 1 reply; 4+ messages in thread From: Daniel Lezcano @ 2020-12-14 23:38 UTC (permalink / raw) To: daniel.lezcano, rui.zhang Cc: mjg59, linux-pm, linux-kernel, amitk, thara.gopinath, Rafael J. Wysocki, Len Brown, open list:ACPI THERMAL DRIVER The loop is here to create default cooling device binding on the THERMAL_TRIPS_NONE number which is used to be the 'forced_passive' feature. However, we removed all code dealing with that in the thermal core, thus this binding does no longer make sense. Remove it. Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> --- drivers/acpi/thermal.c | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c index b5e4bc9e3282..26a89ff80a0e 100644 --- a/drivers/acpi/thermal.c +++ b/drivers/acpi/thermal.c @@ -764,25 +764,6 @@ static int acpi_thermal_cooling_device_cb(struct thermal_zone_device *thermal, } } - for (i = 0; i < tz->devices.count; i++) { - handle = tz->devices.handles[i]; - status = acpi_bus_get_device(handle, &dev); - if (ACPI_SUCCESS(status) && (dev == device)) { - if (bind) - result = thermal_zone_bind_cooling_device - (thermal, THERMAL_TRIPS_NONE, - cdev, THERMAL_NO_LIMIT, - THERMAL_NO_LIMIT, - THERMAL_WEIGHT_DEFAULT); - else - result = thermal_zone_unbind_cooling_device - (thermal, THERMAL_TRIPS_NONE, - cdev); - if (result) - goto failed; - } - } - failed: return result; } -- 2.25.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding 2020-12-14 23:38 ` [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding Daniel Lezcano @ 2021-01-05 15:44 ` Daniel Lezcano 2021-01-07 5:10 ` Zhang, Rui 0 siblings, 1 reply; 4+ messages in thread From: Daniel Lezcano @ 2021-01-05 15:44 UTC (permalink / raw) To: rui.zhang Cc: mjg59, linux-pm, linux-kernel, amitk, thara.gopinath, Rafael J. Wysocki, Len Brown, open list:ACPI THERMAL DRIVER Hi Rui, On 15/12/2020 00:38, Daniel Lezcano wrote: > The loop is here to create default cooling device binding on the > THERMAL_TRIPS_NONE number which is used to be the 'forced_passive' > feature. However, we removed all code dealing with that in the thermal > core, thus this binding does no longer make sense. > > Remove it. > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> Are you fine with this change? Thanks -- Daniel > --- > drivers/acpi/thermal.c | 19 ------------------- > 1 file changed, 19 deletions(-) > > diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c > index b5e4bc9e3282..26a89ff80a0e 100644 > --- a/drivers/acpi/thermal.c > +++ b/drivers/acpi/thermal.c > @@ -764,25 +764,6 @@ static int acpi_thermal_cooling_device_cb(struct thermal_zone_device *thermal, > } > } > > - for (i = 0; i < tz->devices.count; i++) { > - handle = tz->devices.handles[i]; > - status = acpi_bus_get_device(handle, &dev); > - if (ACPI_SUCCESS(status) && (dev == device)) { > - if (bind) > - result = thermal_zone_bind_cooling_device > - (thermal, THERMAL_TRIPS_NONE, > - cdev, THERMAL_NO_LIMIT, > - THERMAL_NO_LIMIT, > - THERMAL_WEIGHT_DEFAULT); > - else > - result = thermal_zone_unbind_cooling_device > - (thermal, THERMAL_TRIPS_NONE, > - cdev); > - if (result) > - goto failed; > - } > - } > - > failed: > return result; > } > -- <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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding 2021-01-05 15:44 ` Daniel Lezcano @ 2021-01-07 5:10 ` Zhang, Rui 2021-01-07 16:46 ` Daniel Lezcano 0 siblings, 1 reply; 4+ messages in thread From: Zhang, Rui @ 2021-01-07 5:10 UTC (permalink / raw) To: Daniel Lezcano Cc: mjg59@codon.org.uk, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, amitk@kernel.org, thara.gopinath@linaro.org, Rafael J. Wysocki, Len Brown, open list:ACPI THERMAL DRIVER ACPI thermal driver binds the devices listed in _TZD method with THERMAL_TRIPS_NONE. Now given that 1. THERMAL_TRIPS_NONE is removed from thermal framework 2. _TZP is rarely supported. I searched ~500 acpidumps from different platforms reported by end users in kernel Bugzilla, there is only one platform with _TZP implemented, and it was almost 10 years ago. So, I think it is safe to remove this piece of code. > -----Original Message----- > From: Daniel Lezcano <daniel.lezcano@linaro.org> > Sent: Tuesday, January 05, 2021 11:44 PM > To: Zhang, Rui <rui.zhang@intel.com> > Cc: mjg59@codon.org.uk; linux-pm@vger.kernel.org; linux- > kernel@vger.kernel.org; amitk@kernel.org; thara.gopinath@linaro.org; > Rafael J. Wysocki <rjw@rjwysocki.net>; Len Brown <lenb@kernel.org>; open > list:ACPI THERMAL DRIVER <linux-acpi@vger.kernel.org> > Subject: Re: [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling > device binding > Importance: High > > Hi Rui, > > > On 15/12/2020 00:38, Daniel Lezcano wrote: > > The loop is here to create default cooling device binding on the > > THERMAL_TRIPS_NONE number which is used to be the 'forced_passive' > > feature. However, we removed all code dealing with that in the thermal > > core, thus this binding does no longer make sense. > > > > Remove it. > > > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> Acked-by: Zhang Rui <rui.zhang@intel.com> Thanks, rui > > Are you fine with this change? > > Thanks > > -- Daniel > > > --- > > drivers/acpi/thermal.c | 19 ------------------- > > 1 file changed, 19 deletions(-) > > > > diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c index > > b5e4bc9e3282..26a89ff80a0e 100644 > > --- a/drivers/acpi/thermal.c > > +++ b/drivers/acpi/thermal.c > > @@ -764,25 +764,6 @@ static int acpi_thermal_cooling_device_cb(struct > thermal_zone_device *thermal, > > } > > } > > > > - for (i = 0; i < tz->devices.count; i++) { > > - handle = tz->devices.handles[i]; > > - status = acpi_bus_get_device(handle, &dev); > > - if (ACPI_SUCCESS(status) && (dev == device)) { > > - if (bind) > > - result = thermal_zone_bind_cooling_device > > - (thermal, > THERMAL_TRIPS_NONE, > > - cdev, THERMAL_NO_LIMIT, > > - THERMAL_NO_LIMIT, > > - > THERMAL_WEIGHT_DEFAULT); > > - else > > - result = > thermal_zone_unbind_cooling_device > > - (thermal, > THERMAL_TRIPS_NONE, > > - cdev); > > - if (result) > > - goto failed; > > - } > > - } > > - > > failed: > > return result; > > } > > > > > -- > <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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding 2021-01-07 5:10 ` Zhang, Rui @ 2021-01-07 16:46 ` Daniel Lezcano 0 siblings, 0 replies; 4+ messages in thread From: Daniel Lezcano @ 2021-01-07 16:46 UTC (permalink / raw) To: Zhang, Rui Cc: mjg59@codon.org.uk, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, amitk@kernel.org, thara.gopinath@linaro.org, Rafael J. Wysocki, Len Brown, open list:ACPI THERMAL DRIVER On 07/01/2021 06:10, Zhang, Rui wrote: > ACPI thermal driver binds the devices listed in _TZD method with THERMAL_TRIPS_NONE. > Now given that > 1. THERMAL_TRIPS_NONE is removed from thermal framework > 2. _TZP is rarely supported. I searched ~500 acpidumps from different platforms reported by end users in kernel Bugzilla, there is only one platform with _TZP implemented, and it was almost 10 years ago. > > So, I think it is safe to remove this piece of code. Thanks Rui for digging into -- <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 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-01-07 16:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20201214233811.485669-1-daniel.lezcano@linaro.org>
2020-12-14 23:38 ` [PATCH 4/6] acpi/drivers/thermal: Remove TRIPS_NONE cooling device binding Daniel Lezcano
2021-01-05 15:44 ` Daniel Lezcano
2021-01-07 5:10 ` Zhang, Rui
2021-01-07 16:46 ` Daniel Lezcano
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).