Re: [PATCH v4 6/6] thermal/drivers/intel_menlow: Make additionnal sysfs information optional

From: Daniel Lezcano
Date: Thu Apr 20 2023 - 17:40:11 EST


On 20/04/2023 19:24, Rafael J. Wysocki wrote:

[ ... ]

After looking at it once more, I think that this driver isn't really
functional without the additional sysfs attributes, so if
CONFIG_INTEL_MENLOW_SYSFS_ADDON is set, the driver effectively becomes
dead code.

That's rather unfortunate and I'm not sure how to deal with it ATM.

I can queue up the rest of the series for 6.4-rc1 (in which case it
will be pushed in the second half of the merge window), but this
particular patch requires more thought IMV.

I'm fine if you drop this one from the series.

Thanks for looking at it more deeply


--
<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