* [PATCH] thermal/drivers/mediatek/lvts: remove unused lvts_debugfs_exit
@ 2025-05-05 5:24 Arnd Bergmann
2025-05-05 5:35 ` Daniel Lezcano
0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2025-05-05 5:24 UTC (permalink / raw)
To: Rafael J. Wysocki, Daniel Lezcano, Matthias Brugger,
AngeloGioacchino Del Regno, Chen-Yu Tsai
Cc: Arnd Bergmann, Zhang Rui, Lukasz Luba, Julien Panis,
Nícolas F. R. A. Prado, Nicolas Pitre, Colin Ian King,
linux-pm, linux-kernel, linux-arm-kernel, linux-mediatek
From: Arnd Bergmann <arnd@arndb.de>
When debugfs is disabled, the function has no reference any more:
drivers/thermal/mediatek/lvts_thermal.c:266:13: error: 'lvts_debugfs_exit' defined but not used [-Werror=unused-function]
266 | static void lvts_debugfs_exit(struct lvts_domain *lvts_td) { }
| ^~~~~~~~~~~~~~~~~
Fixes: ef280c17a840 ("thermal/drivers/mediatek/lvts: Fix debugfs unregister on failure")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/thermal/mediatek/lvts_thermal.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/thermal/mediatek/lvts_thermal.c b/drivers/thermal/mediatek/lvts_thermal.c
index c0be4ca55c7b..985925147ac0 100644
--- a/drivers/thermal/mediatek/lvts_thermal.c
+++ b/drivers/thermal/mediatek/lvts_thermal.c
@@ -263,8 +263,6 @@ static inline int lvts_debugfs_init(struct device *dev,
return 0;
}
-static void lvts_debugfs_exit(struct lvts_domain *lvts_td) { }
-
#endif
static int lvts_raw_to_temp(u32 raw_temp, int temp_factor)
--
2.39.5
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] thermal/drivers/mediatek/lvts: remove unused lvts_debugfs_exit
2025-05-05 5:24 [PATCH] thermal/drivers/mediatek/lvts: remove unused lvts_debugfs_exit Arnd Bergmann
@ 2025-05-05 5:35 ` Daniel Lezcano
0 siblings, 0 replies; 2+ messages in thread
From: Daniel Lezcano @ 2025-05-05 5:35 UTC (permalink / raw)
To: Arnd Bergmann, Rafael J. Wysocki, Matthias Brugger,
AngeloGioacchino Del Regno, Chen-Yu Tsai
Cc: Arnd Bergmann, Zhang Rui, Lukasz Luba, Julien Panis,
Nícolas F. R. A. Prado, Nicolas Pitre, Colin Ian King,
linux-pm, linux-kernel, linux-arm-kernel, linux-mediatek
On 05/05/2025 07:24, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> When debugfs is disabled, the function has no reference any more:
>
> drivers/thermal/mediatek/lvts_thermal.c:266:13: error: 'lvts_debugfs_exit' defined but not used [-Werror=unused-function]
> 266 | static void lvts_debugfs_exit(struct lvts_domain *lvts_td) { }
> | ^~~~~~~~~~~~~~~~~
>
> Fixes: ef280c17a840 ("thermal/drivers/mediatek/lvts: Fix debugfs unregister on failure")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
Applied, thanks
--
<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] 2+ messages in thread
end of thread, other threads:[~2025-05-05 5:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-05 5:24 [PATCH] thermal/drivers/mediatek/lvts: remove unused lvts_debugfs_exit Arnd Bergmann
2025-05-05 5:35 ` Daniel Lezcano
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox