hwmon: (bt1-pvt) Define Temp- and Volt-to-N poly as maybe-unused
authorSerge Semin <Sergey.Semin@baikalelectronics.ru>
Wed, 3 Jun 2020 00:07:53 +0000 (03:07 +0300)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 23 Jun 2020 13:29:32 +0000 (06:29 -0700)
commit332516947ef83343b9bcc7275ff99db392592c94
tree37ac00941300679a35ecdbe4483f58690bd571f3
parent48778464bb7d346b47157d21ffde2af6b2d39110
hwmon: (bt1-pvt) Define Temp- and Volt-to-N poly as maybe-unused

Clang-based kernel building with W=1 warns that some static const
variables are unused:

drivers/hwmon/bt1-pvt.c:67:30: warning: unused variable 'poly_temp_to_N' [-Wunused-const-variable]
static const struct pvt_poly poly_temp_to_N = {
                             ^
drivers/hwmon/bt1-pvt.c:99:30: warning: unused variable 'poly_volt_to_N' [-Wunused-const-variable]
static const struct pvt_poly poly_volt_to_N = {
                             ^

Indeed these polynomials are utilized only when the PVT sensor alarms are
enabled. In that case they are used to convert the temperature and
voltage alarm limits from normal quantities (Volts and degree Celsius) to
the sensor data representation N = [0, 1023]. Otherwise when alarms are
disabled the driver only does the detected data conversion to the human
readable form and doesn't need that polynomials defined. So let's mark the
Temp-to-N and Volt-to-N polynomials with __maybe_unused attribute.

Note gcc with W=1 doesn't notice the problem.

Fixes: 87976ce2825d ("hwmon: Add Baikal-T1 PVT sensor driver")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru>
Cc: Maxim Kaurkin <Maxim.Kaurkin@baikalelectronics.ru>
Cc: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru>
Link: https://lore.kernel.org/r/20200603000753.391-1-Sergey.Semin@baikalelectronics.ru
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/bt1-pvt.c