michael/zephyr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

input: use single evaluation clamp

Browse Source 
Use single evaluation clamp for few input drivers. No reason not to,
make some sense for the input_kbd_matrix call since one of the arguments
is a function.

Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
This commit is contained in:
Fabio Baltieri
2025-10-21 21:28:45 +01:00
committed by Johan Hedberg
parent 2f91d64997
commit b0b5c083fb
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/input/input_analog_axis.c
View File

@@ -204,7 +204,7 @@ static void analog_axis_loop(const struct device *dev)
out = analog_axis_out_linear(dev, i, raw_val);
}
out = CLAMP(out, axis_cfg->out_min, axis_cfg->out_max);
out = clamp(out, axis_cfg->out_min, axis_cfg->out_max);
if (axis_cfg->invert_output) {
out = axis_cfg->out_max - out;

2
drivers/input/input_kbd_matrix.c
View File

@@ -309,7 +309,7 @@ static void input_kbd_matrix_poll(const struct device *dev)
} else {
poll_period_us = cfg->stable_poll_period_us;
}
wait_period_us = CLAMP(poll_period_us - k_cyc_to_us_floor32(cycles_diff),
wait_period_us = clamp(poll_period_us - k_cyc_to_us_floor32(cycles_diff),
USEC_PER_MSEC, poll_period_us);
LOG_DBG("wait_period_us: %d", wait_period_us);

4
drivers/input/input_stmpe811.c
View File

@@ -344,8 +344,8 @@ static void stmpe811_report_touch(const struct device *dev)
y = (((int)data->touch_y - config->raw_y_min) * common->screen_height) /
(config->raw_y_max - config->raw_y_min);
x = CLAMP(x, 0, common->screen_width);
y = CLAMP(y, 0, common->screen_height);
x = clamp(x, 0, common->screen_width);
y = clamp(y, 0, common->screen_height);
}
input_touchscreen_report_pos(dev, x, y, K_FOREVER);