Message ID | 20230301201446.3713334-1-daniel.lezcano@linaro.org (mailing list archive) |
---|---|
Headers | show |
Series | Self-encapsulate the thermal zone device structure | expand |
Hi Rafael, Do we have enough ack to apply this series, is it for you ? Thanks -- Daniel On 01/03/2023 21:14, Daniel Lezcano wrote: > The exported thermal headers expose the thermal core structure while those > should be private to the framework. The initial idea was the thermal sensor > drivers use the thermal zone device structure pointer to pass it around from > the ops to the thermal framework API like a handler. > > Unfortunately, different drivers are using and abusing the internals of this > structure to hook the associated struct device, read the internals values, take > the lock, etc ... > > In order to fix this situation, let's encapsulate the structure leaking the > more in the different drivers: the thermal_zone_device structure. > > This series revisit the existing drivers using the thermal zone private > structure internals to change the access to something else. For instance, the > get_temp() ops is using the tz->dev to write a debug trace. Despite the trace > is not helpful, we can check the return value for the get_temp() ops in the > call site and show the message in this place. > > With this set of changes, the thermal_zone_device is almost self-encapsulated. > As usual, the acpi driver needs a more complex changes, so that will come in a > separate series along with the structure moved the private core headers. > > Changelog: > - V5: > - Dropped patch 19 : "thermal/tegra: Do not enable ... is already enabled" > - Changed the init sequence of the hw channels on tegra3 to close > the race window > - Collected more tags > - V4: > - Collected more tags > - Fixed a typo therma_zone_device_priv() for db8500 > - Remove traces patch [20/20] to be submitted separetely > - V3: > - Split the first patch into three to reduce the number of > recipients per change > - Collected more tags > - Added missing changes for ->devdata in some drivers > - Added a 'type' accessor > - Replaced the 'type' to 'id' changes by the 'type' accessor > - Used the 'type' accessor in the drivers > - V2: > - Collected tags > - Added missing changes for ->devdata for the tsens driver > - Renamed thermal_zone_device_get_data() to thermal_zone_priv() > - Added stubs when CONFIG_THERMAL is not set > - Dropped hwmon change where we remove the tz->lock usage > > Thank you all for your comments > > Cc: "Rafael J. Wysocki" <rafael@kernel.org> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Len Brown <lenb@kernel.org> > Cc: Damien Le Moal <damien.lemoal@opensource.wdc.com> > Cc: Shawn Guo <shawnguo@kernel.org> > Cc: Sascha Hauer <s.hauer@pengutronix.de> > Cc: Pengutronix Kernel Team <kernel@pengutronix.de> > Cc: Fabio Estevam <festevam@gmail.com> > Cc: NXP Linux Team <linux-imx@nxp.com> > Cc: Jean Delvare <jdelvare@suse.com> > Cc: Guenter Roeck <linux@roeck-us.net> > Cc: Jonathan Cameron <jic23@kernel.org> > Cc: Lars-Peter Clausen <lars@metafoo.de> > Cc: Chen-Yu Tsai <wens@csie.org> > Cc: Jernej Skrabec <jernej.skrabec@gmail.com> > Cc: Samuel Holland <samuel@sholland.org> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Eric Dumazet <edumazet@google.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Paolo Abeni <pabeni@redhat.com> > Cc: Ido Schimmel <idosch@nvidia.com> > Cc: Petr Machata <petrm@nvidia.com> > Cc: Gregory Greenman <gregory.greenman@intel.com> > Cc: Kalle Valo <kvalo@kernel.org> > Cc: Sebastian Reichel <sre@kernel.org> > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Miquel Raynal <miquel.raynal@bootlin.com> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Amit Kucheria <amitk@kernel.org> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com> > Cc: Ray Jui <rjui@broadcom.com> > Cc: Scott Branden <sbranden@broadcom.com> > Cc: Markus Mayer <mmayer@broadcom.com> > Cc: Support Opensource <support.opensource@diasemi.com> > Cc: Andy Gross <agross@kernel.org> > Cc: Bjorn Andersson <andersson@kernel.org> > Cc: Konrad Dybcio <konrad.dybcio@linaro.org> > Cc: Thara Gopinath <thara.gopinath@gmail.com> > Cc: "Niklas Söderlund" <niklas.soderlund@ragnatech.se> > Cc: Heiko Stuebner <heiko@sntech.de> > Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Cc: Alim Akhtar <alim.akhtar@samsung.com> > Cc: Orson Zhai <orsonzhai@gmail.com> > Cc: Baolin Wang <baolin.wang@linux.alibaba.com> > Cc: Chunyan Zhang <zhang.lyra@gmail.com> > Cc: Vasily Khoruzhick <anarsoul@gmail.com> > Cc: Yangtao Li <tiny.windzz@gmail.com> > Cc: Thierry Reding <thierry.reding@gmail.com> > Cc: Jonathan Hunter <jonathanh@nvidia.com> > Cc: Talel Shenhar <talel@amazon.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Keerthy <j-keerthy@ti.com> > Cc: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> > Cc: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Matthias Brugger <matthias.bgg@gmail.com> > Cc: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > Cc: Stefan Wahren <stefan.wahren@i2se.com> > Cc: Zheng Yongjun <zhengyongjun3@huawei.com> > Cc: Yang Li <yang.lee@linux.alibaba.com> > Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> > Cc: Daniel Golle <daniel@makrotopia.org> > Cc: Balsam CHIHI <bchihi@baylibre.com> > Cc: Mikko Perttunen <mperttunen@nvidia.com> > Cc: linux-acpi@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Cc: linux-ide@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-hwmon@vger.kernel.org > Cc: linux-iio@vger.kernel.org > Cc: linux-sunxi@lists.linux.dev > Cc: linux-input@vger.kernel.org > Cc: netdev@vger.kernel.org > Cc: linux-wireless@vger.kernel.org > Cc: linux-pm@vger.kernel.org > Cc: linux-rpi-kernel@lists.infradead.org > Cc: linux-arm-msm@vger.kernel.org > Cc: linux-renesas-soc@vger.kernel.org > Cc: linux-rockchip@lists.infradead.org > Cc: linux-samsung-soc@vger.kernel.org > Cc: linux-tegra@vger.kernel.org > Cc: linux-omap@vger.kernel.org > Cc: linux-mediatek@lists.infradead.org > > Daniel Lezcano (18): > thermal/core: Add a thermal zone 'devdata' accessor > thermal/core: Use the thermal zone 'devdata' accessor in thermal > located drivers > thermal/core: Use the thermal zone 'devdata' accessor in hwmon located > drivers > thermal/core: Use the thermal zone 'devdata' accessor in remaining > drivers > thermal/core: Show a debug message when get_temp() fails > thermal: Remove debug or error messages in get_temp() ops > thermal/hwmon: Do not set no_hwmon before calling > thermal_add_hwmon_sysfs() > thermal/hwmon: Use the right device for devm_thermal_add_hwmon_sysfs() > thermal: Don't use 'device' internal thermal zone structure field > thermal/core: Add thermal_zone_device structure 'type' accessor > thermal/drivers/spear: Don't use tz->device but pdev->dev > thermal: Add a thermal zone id accessor > thermal: Use thermal_zone_device_type() accessor > thermal/drivers/da9062: Don't access the thermal zone device fields > thermal/hwmon: Use the thermal_core.h header > thermal/drivers/tegra: Remove unneeded lock when setting a trip point > thermal/drivers/acerhdf: Make interval setting only at module load > time > thermal/drivers/acerhdf: Remove pointless governor test > > drivers/acpi/thermal.c | 18 +++---- > drivers/ata/ahci_imx.c | 2 +- > drivers/hwmon/hwmon.c | 4 +- > drivers/hwmon/pmbus/pmbus_core.c | 2 +- > drivers/hwmon/scmi-hwmon.c | 4 +- > drivers/hwmon/scpi-hwmon.c | 2 +- > drivers/iio/adc/sun4i-gpadc-iio.c | 2 +- > drivers/input/touchscreen/sun4i-ts.c | 2 +- > .../ethernet/chelsio/cxgb4/cxgb4_thermal.c | 2 +- > .../ethernet/mellanox/mlxsw/core_thermal.c | 16 +++---- > drivers/net/wireless/intel/iwlwifi/mvm/tt.c | 4 +- > drivers/platform/x86/acerhdf.c | 19 ++------ > drivers/power/supply/power_supply_core.c | 2 +- > drivers/regulator/max8973-regulator.c | 2 +- > drivers/thermal/amlogic_thermal.c | 4 +- > drivers/thermal/armada_thermal.c | 14 ++---- > drivers/thermal/broadcom/bcm2711_thermal.c | 3 +- > drivers/thermal/broadcom/bcm2835_thermal.c | 3 +- > drivers/thermal/broadcom/brcmstb_thermal.c | 8 ++-- > drivers/thermal/broadcom/ns-thermal.c | 2 +- > drivers/thermal/broadcom/sr-thermal.c | 2 +- > drivers/thermal/da9062-thermal.c | 13 +++-- > drivers/thermal/db8500_thermal.c | 2 +- > drivers/thermal/dove_thermal.c | 7 +-- > drivers/thermal/hisi_thermal.c | 5 +- > drivers/thermal/imx8mm_thermal.c | 4 +- > drivers/thermal/imx_sc_thermal.c | 9 ++-- > drivers/thermal/imx_thermal.c | 47 +++++-------------- > .../intel/int340x_thermal/int3400_thermal.c | 2 +- > .../int340x_thermal/int340x_thermal_zone.c | 4 +- > .../processor_thermal_device_pci.c | 4 +- > drivers/thermal/intel/intel_pch_thermal.c | 2 +- > .../thermal/intel/intel_quark_dts_thermal.c | 6 +-- > drivers/thermal/intel/intel_soc_dts_iosf.c | 13 ++--- > drivers/thermal/intel/x86_pkg_temp_thermal.c | 4 +- > drivers/thermal/k3_bandgap.c | 4 +- > drivers/thermal/k3_j72xx_bandgap.c | 2 +- > drivers/thermal/kirkwood_thermal.c | 7 +-- > drivers/thermal/max77620_thermal.c | 6 +-- > drivers/thermal/mediatek/auxadc_thermal.c | 4 +- > drivers/thermal/mediatek/lvts_thermal.c | 10 ++-- > drivers/thermal/qcom/qcom-spmi-adc-tm5.c | 6 +-- > drivers/thermal/qcom/qcom-spmi-temp-alarm.c | 6 +-- > drivers/thermal/qcom/tsens.c | 6 +-- > drivers/thermal/qoriq_thermal.c | 4 +- > drivers/thermal/rcar_gen3_thermal.c | 5 +- > drivers/thermal/rcar_thermal.c | 8 +--- > drivers/thermal/rockchip_thermal.c | 8 +--- > drivers/thermal/rzg2l_thermal.c | 3 +- > drivers/thermal/samsung/exynos_tmu.c | 4 +- > drivers/thermal/spear_thermal.c | 10 ++-- > drivers/thermal/sprd_thermal.c | 2 +- > drivers/thermal/st/st_thermal.c | 5 +- > drivers/thermal/st/stm_thermal.c | 4 +- > drivers/thermal/sun8i_thermal.c | 4 +- > drivers/thermal/tegra/soctherm.c | 6 +-- > drivers/thermal/tegra/tegra-bpmp-thermal.c | 6 ++- > drivers/thermal/tegra/tegra30-tsensor.c | 31 ++++++------ > drivers/thermal/thermal-generic-adc.c | 7 ++- > drivers/thermal/thermal_core.c | 18 +++++++ > drivers/thermal/thermal_helpers.c | 3 ++ > drivers/thermal/thermal_hwmon.c | 9 ++-- > drivers/thermal/thermal_hwmon.h | 4 +- > drivers/thermal/thermal_mmio.c | 2 +- > .../ti-soc-thermal/ti-thermal-common.c | 10 ++-- > drivers/thermal/uniphier_thermal.c | 2 +- > include/linux/thermal.h | 19 ++++++++ > 67 files changed, 218 insertions(+), 246 deletions(-) >
Hi Daniel, On Fri, Mar 3, 2023 at 10:24 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > > Hi Rafael, > > Do we have enough ack to apply this series, is it for you ? I've just queued it up for 6.4. It will reach linux-next and the thermal branch some time next week, but I will be traveling, so there may be delays. Thanks!