linux/Documentation/driver-api
Daniel Lezcano a7d6ba14ef thermal/core: Remove the 'forced_passive' option
The code was reorganized in 2012 with the commit 0c01ebbfd3.

The main change is a loop on the trip points array and a unconditional
call to the throttle() ops of the governors for each of them even if
the trip temperature is not reached yet.

With this change, the 'forced_passive' is no longer checked in the
thermal_zone_device_update() function but in the step wise governor's
throttle() callback.

As the force_passive does no belong to the trip point array, the
thermal_zone_device_update() can not compare with the specified
passive temperature, thus does not detect the passive limit has been
crossed. Consequently, throttle() is never called and the
'forced_passive' branch is unreached.

In addition, the default processor cooling device is not automatically
bound to the thermal zone if there is not passive trip point, thus the
'forced_passive' can not operate.

If there is an active trip point, then the throttle function will be
called to mitigate at this temperature and the 'forced_passive' will
override the mitigation of the active trip point in this case but with
the default cooling device bound to the thermal zone, so usually a
fan, and that is not a passive cooling effect.

Given the regression exists since more than 8 years, nobody complained
and at the best of my knowledge there is no bug open in
https://bugzilla.kernel.org, it is reasonable to say it is unused.

Remove the 'forced_passive' related code.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Thara Gopinath <thara.gopinath@linaro.org>
Link: https://lore.kernel.org/r/20201214233811.485669-1-daniel.lezcano@linaro.org
2021-01-19 22:22:45 +01:00
..
80211
acpi
backlight
dmaengine dmaengine: doc: client: Update for dmaengine_get_dma_device() usage 2020-12-11 21:20:08 +05:30
driver-model rtc: rework rtc_register_device() resource management 2020-11-19 12:50:12 +01:00
early-userspace
firmware
fpga
gpio Documentation: gpio: fix typo and unclear legacy API section 2020-12-05 20:23:34 +01:00
i3c
iio
md
media media: Documentation: ccs: Reorder SPDX and copyright notice lines 2020-12-03 10:39:24 +01:00
mei docs updates for v5.10-rc1 2020-10-16 15:02:21 -07:00
memory-devices
mmc
mtd MTD core: 2020-12-16 14:58:35 -08:00
nfc
nvdimm
pci
phy
pldmfw
pm
rapidio
serial
soundwire
thermal thermal/core: Remove the 'forced_passive' option 2021-01-19 22:22:45 +01:00
usb
xilinx
auxiliary_bus.rst driver core: auxiliary bus: make remove function return void 2020-12-04 13:30:48 +01:00
basics.rst
clk.rst
component.rst
connector.rst net/connector: Add const qualifier to cb_id 2020-12-16 11:06:49 -08:00
console.rst
dcdbas.rst
devfreq.rst
device-io.rst
device_link.rst
dma-buf.rst dma-buf: Fix kerneldoc formatting 2020-12-15 10:57:29 +01:00
edac.rst
eisa.rst
firewire.rst
frame-buffer.rst
generic-counter.rst
hsi.rst
i2c.rst
index.rst Networking updates for 5.11 2020-12-15 13:22:29 -08:00
infiniband.rst
infrastructure.rst
input.rst Input: remove input_polled_dev implementation 2020-12-02 12:35:14 -08:00
interconnect.rst
io-mapping.rst io-mapping: Provide iomap_local variant 2020-11-24 14:42:09 +01:00
io_ordering.rst
ioctl.rst
ipmb.rst
ipmi.rst
isa.rst
isapnp.rst
libata.rst
lightnvm-pblk.rst
mailbox.rst
men-chameleon-bus.rst
message-based.rst
misc_devices.rst
miscellaneous.rst
mtdnand.rst docs: mtd: Avoid htmldocs warnings 2020-12-10 22:37:32 +01:00
ntb.rst
nvmem.rst
parport-lowlevel.rst
pinctl.rst
pps.rst
pti_intel_mid.rst
ptp.rst
pwm.rst
regulator.rst
reset.rst docs: add a reset controller chapter to the driver API docs 2020-12-08 10:48:04 -07:00
rfkill.rst
s390-drivers.rst
scsi.rst
slimbus.rst
sm501.rst
spi.rst
switchtec.rst
sync_file.rst
target.rst
uio-howto.rst
vfio-mediated-device.rst
vfio.rst
vme.rst
w1.rst
xillybus.rst
zorro.rst