michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: adc: use asserts in adc_gain_invert implementations

Browse source 
no need for runtime checks here, let's save a few bytes of code

Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
This commit is contained in:
Benjamin Cabé 2025-06-12 21:55:09 +02:00 • committed by Daniel DeGrasse
commit e7f58da491
1 changed files with 9 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

17
drivers/adc/adc_common.c
View file

@ -5,6 +5,7 @@
*/ */
#include <zephyr/drivers/adc.h> #include <zephyr/drivers/adc.h>
#include <zephyr/sys/util.h>
struct gain_desc { struct gain_desc {
uint8_t mul; uint8_t mul;
@ -41,10 +42,10 @@ int adc_gain_invert(enum adc_gain gain, int32_t *value)
if ((uint8_t)gain < ARRAY_SIZE(gains)) { if ((uint8_t)gain < ARRAY_SIZE(gains)) {
const struct gain_desc *gdp = &gains[gain]; const struct gain_desc *gdp = &gains[gain];
if ((gdp->mul != 0) && (gdp->div != 0)) { __ASSERT_NO_MSG(gdp->mul != 0);
*value = (gdp->mul * *value) / gdp->div; __ASSERT_NO_MSG(gdp->div != 0);
rv = 0; *value = (gdp->mul * *value) / gdp->div;
} rv = 0;
} }
return rv; return rv;
@ -57,10 +58,10 @@ int adc_gain_invert_64(enum adc_gain gain, int64_t *value)
if ((uint8_t)gain < ARRAY_SIZE(gains)) { if ((uint8_t)gain < ARRAY_SIZE(gains)) {
const struct gain_desc *gdp = &gains[gain]; const struct gain_desc *gdp = &gains[gain];
if ((gdp->mul != 0) && (gdp->div != 0)) { __ASSERT_NO_MSG(gdp->mul != 0);
*value = (gdp->mul * *value) / gdp->div; __ASSERT_NO_MSG(gdp->div != 0);
rv = 0; *value = (gdp->mul * *value) / gdp->div;
} rv = 0;
} }
return rv; return rv;