From c58c130b48b801b70ea1ffecbf7176425412ee5d Mon Sep 17 00:00:00 2001 From: Ryan McClelland Date: Fri, 1 Nov 2024 15:35:35 -0700 Subject: [PATCH] drivers: sensor: default_rtio_sensor: fix limited range warning `chan_type` is defined as a `uint16_t`. This makes checking if it is < 0 always false. A warning is shown with -Wtype-limits. Remove the check as it is unnecessary. Signed-off-by: Ryan McClelland --- drivers/sensor/default_rtio_sensor.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/sensor/default_rtio_sensor.c b/drivers/sensor/default_rtio_sensor.c index 11b8fc0c140..123d6b1ff95 100644 --- a/drivers/sensor/default_rtio_sensor.c +++ b/drivers/sensor/default_rtio_sensor.c @@ -348,7 +348,7 @@ int sensor_natively_supported_channel_size_info(struct sensor_chan_spec channel, __ASSERT_NO_MSG(base_size != NULL); __ASSERT_NO_MSG(frame_size != NULL); - if (((int)channel.chan_type < 0) || channel.chan_type >= (SENSOR_CHAN_ALL)) { + if (channel.chan_type >= SENSOR_CHAN_ALL) { return -ENOTSUP; } @@ -474,7 +474,7 @@ static int decode(const uint8_t *buffer, struct sensor_chan_spec chan_spec, return -EINVAL; } - if (((int)chan_spec.chan_type < 0) || chan_spec.chan_type >= (SENSOR_CHAN_ALL)) { + if (chan_spec.chan_type >= SENSOR_CHAN_ALL) { return 0; }