drivers: sensor: ina3221: fix double-promotion

XenuIsWatching · MaureenHelm · commit b92e749c3e5f · 2023-08-23T07:51:53.000-05:00
channel_get was doing using floating point constants with it's calculation.
The result is changed to be a float as this was generating a double
promotion warning.

Signed-off-by: Ryan McClelland &lt;ryanmcclelland@meta.com&gt;
diff --git a/drivers/sensor/ina3221/ina3221.c b/drivers/sensor/ina3221/ina3221.c
@@ -225,7 +225,7 @@ static int ina3221_channel_get(const struct device *dev, enum sensor_channel cha
 {
 	const struct ina3221_config *cfg = dev->config;
 	struct ina3221_data *data = dev->data;
-	double result;
+	float result;
 
 	switch (chan) {
 	case SENSOR_CHAN_VOLTAGE:
@@ -245,7 +245,7 @@ static int ina3221_channel_get(const struct device *dev, enum sensor_channel cha
 		return -ENOTSUP;
 	}
 
-	return sensor_value_from_double(val, result);
+	return sensor_value_from_float(val, result);
 }
 
 static int ina3221_attr_set(const struct device *dev, enum sensor_channel chan,

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ static int ina3221_channel_get(const struct device *dev, enum sensor_channel cha`
`225`	`225`	`{`
`226`	`226`	`const struct ina3221_config *cfg = dev->config;`
`227`	`227`	`struct ina3221_data *data = dev->data;`
`228`		`- double result;`
	`228`	`+ float result;`
`229`	`229`
`230`	`230`	`switch (chan) {`
`231`	`231`	`case SENSOR_CHAN_VOLTAGE:`
`@@ -245,7 +245,7 @@ static int ina3221_channel_get(const struct device *dev, enum sensor_channel cha`
`245`	`245`	`return -ENOTSUP;`
`246`	`246`	`}`
`247`	`247`
`248`		`- return sensor_value_from_double(val, result);`
	`248`	`+ return sensor_value_from_float(val, result);`
`249`	`249`	`}`
`250`	`250`
`251`	`251`	`static int ina3221_attr_set(const struct device *dev, enum sensor_channel chan,`