| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20250515081332.151250-7-asoponar@taladin.ro>
Date: Thu, 15 May 2025 11:13:22 +0300
From: Alexandru Soponar <asoponar@...adin.ro>
To: linux-kernel@...r.kernel.org,
linux-hwmon@...r.kernel.org,
linux-iio@...r.kernel.org,
linux-leds@...r.kernel.org,
linux-watchdog@...r.kernel.org
Cc: jdelvare@...e.com,
linux@...ck-us.net,
jic23@...nel.org,
pavel@....cz,
lee@...nel.org,
baocheng.su@...mens.com,
wim@...ux-watchdog.org,
tobias.schaffner@...mens.com,
angelogioacchino.delregno@...labora.com,
benedikt.niedermayr@...mens.com,
matthias.bgg@...il.com,
aardelean@...libre.com,
contact@...y.one,
Alexandru Soponar <asoponar@...adin.ro>
Subject: [PATCH 06/16] hwmon: lm75: Fix type incompatibility with non-macro find_closest
The default_sample_time and sample_times members of lm75_params were
previously declared as unsigned int but used withfind_closest(). With
find_closest() now implemented as a function taking signed int parameters
instead of a macro, passing unsigned arrays causes type incompatibility
errors. This patch changes the types from unsigned int and unsigned int*
to int respectively int* to ensure compatibility with the function
signature and prevent compilation errors.
Signed-off-by: Alexandru Soponar <asoponar@...adin.ro>
---
drivers/hwmon/lm75.c | 42 +++++++++++++++++++++---------------------
1 file changed, 21 insertions(+), 21 deletions(-)
diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
index d95a3c6c245c..5c835ee189d4 100644
--- a/drivers/hwmon/lm75.c
+++ b/drivers/hwmon/lm75.c
@@ -89,9 +89,9 @@ struct lm75_params {
u8 default_resolution;
u8 resolution_limits;
const u8 *resolutions;
- unsigned int default_sample_time;
+ int default_sample_time;
u8 num_sample_times;
- const unsigned int *sample_times;
+ const int *sample_times;
bool alarm;
};
@@ -110,7 +110,7 @@ struct lm75_data {
struct regmap *regmap;
u16 orig_conf;
u8 resolution; /* In bits, 9 to 16 */
- unsigned int sample_time; /* In ms */
+ int sample_time; /* In ms */
enum lm75_type kind;
const struct lm75_params *params;
u8 reg_buf[1];
@@ -139,7 +139,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 125,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 125, 250, 1000, 4000 },
+ .sample_times = (int []){ 125, 250, 1000, 4000 },
.alarm = true,
},
[at30ts74] = {
@@ -147,7 +147,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 200,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 25, 50, 100, 200 },
+ .sample_times = (int []){ 25, 50, 100, 200 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[ds1775] = {
@@ -156,7 +156,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 11,
.default_sample_time = 500,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 125, 250, 500, 1000 },
+ .sample_times = (int []){ 125, 250, 500, 1000 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[ds75] = {
@@ -165,7 +165,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 11,
.default_sample_time = 600,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 150, 300, 600, 1200 },
+ .sample_times = (int []){ 150, 300, 600, 1200 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[stds75] = {
@@ -174,7 +174,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 11,
.default_sample_time = 600,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 150, 300, 600, 1200 },
+ .sample_times = (int []){ 150, 300, 600, 1200 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[stlm75] = {
@@ -186,7 +186,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 200,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 25, 50, 100, 200 },
+ .sample_times = (int []){ 25, 50, 100, 200 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[g751] = {
@@ -227,13 +227,13 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 55,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
},
[pct2075] = {
.default_resolution = 11,
.default_sample_time = MSEC_PER_SEC / 10,
.num_sample_times = 31,
- .sample_times = (unsigned int []){ 100, 200, 300, 400, 500, 600,
+ .sample_times = (int []){ 100, 200, 300, 400, 500, 600,
700, 800, 900, 1000, 1100, 1200, 1300, 1400, 1500, 1600, 1700,
1800, 1900, 2000, 2100, 2200, 2300, 2400, 2500, 2600, 2700,
2800, 2900, 3000, 3100 },
@@ -245,7 +245,7 @@ static const struct lm75_params device_params[] = {
.resolution_limits = 9,
.default_sample_time = 240,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 30, 60, 120, 240 },
+ .sample_times = (int []){ 30, 60, 120, 240 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp100] = {
@@ -254,7 +254,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 320,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 40, 80, 160, 320 },
+ .sample_times = (int []){ 40, 80, 160, 320 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp101] = {
@@ -263,7 +263,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 320,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 40, 80, 160, 320 },
+ .sample_times = (int []){ 40, 80, 160, 320 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp105] = {
@@ -272,7 +272,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 220,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp112] = {
@@ -282,7 +282,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 125,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 125, 250, 1000, 4000 },
+ .sample_times = (int []){ 125, 250, 1000, 4000 },
.alarm = true,
},
[tmp175] = {
@@ -291,7 +291,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 220,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp275] = {
@@ -300,7 +300,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 220,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp75] = {
@@ -309,14 +309,14 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 220,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
.resolutions = (u8 []) {9, 10, 11, 12 },
},
[tmp75b] = { /* not one-shot mode, Conversion rate 37Hz */
.clr_mask = 1 << 7 | 3 << 5,
.default_resolution = 12,
.default_sample_time = MSEC_PER_SEC / 37,
- .sample_times = (unsigned int []){ MSEC_PER_SEC / 37,
+ .sample_times = (int []){ MSEC_PER_SEC / 37,
MSEC_PER_SEC / 18,
MSEC_PER_SEC / 9, MSEC_PER_SEC / 4 },
.num_sample_times = 4,
@@ -331,7 +331,7 @@ static const struct lm75_params device_params[] = {
.default_resolution = 12,
.default_sample_time = 28,
.num_sample_times = 4,
- .sample_times = (unsigned int []){ 28, 55, 110, 220 },
+ .sample_times = (int []){ 28, 55, 110, 220 },
}
};
--
2.49.0
Powered by blists - more mailing lists