Skip to content

drivers: sensor: adi: adxl345: take selected g-range into account when converting samples #100746

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aescolar merged 1 commit into from
Dec 11, 2025
+25 −10
Merged

drivers: sensor: adi: adxl345: take selected g-range into account when converting samples #100746

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 25 additions & 9 deletions drivers/sensor/adi/adxl345/adxl345.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -312,14 +312,22 @@ int adxl345_read_sample(const struct device *dev,
return 0;
}

void adxl345_accel_convert(struct sensor_value *val, int16_t sample)
static void adxl345_accel_convert(struct sensor_value *val, int16_t sample,
uint8_t selected_range)
{
const int32_t sensitivity[] = {
[ADXL345_RANGE_2G] = INT32_C(SENSOR_G / 256),
[ADXL345_RANGE_4G] = INT32_C(SENSOR_G / 128),
[ADXL345_RANGE_8G] = INT32_C(SENSOR_G / 64),
[ADXL345_RANGE_16G] = INT32_C(SENSOR_G / 32),
};

if (sample & BIT(9)) {
sample |= ADXL345_COMPLEMENT;
}

val->val1 = ((sample * SENSOR_G) / 32) / 1000000;
val->val2 = ((sample * SENSOR_G) / 32) % 1000000;
val->val1 = (sample * sensitivity[selected_range]) / 1000000;
val->val2 = (sample * sensitivity[selected_range]) % 1000000;
}

static int adxl345_sample_fetch(const struct device *dev,
Expand Down Expand Up @@ -349,18 +357,26 @@ static int adxl345_channel_get(const struct device *dev,

switch (chan) {
case SENSOR_CHAN_ACCEL_X:
adxl345_accel_convert(val, data->samples.x);
adxl345_accel_convert(val, data->samples.x,
data->selected_range);
break;
case SENSOR_CHAN_ACCEL_Y:
adxl345_accel_convert(val, data->samples.y);
adxl345_accel_convert(val, data->samples.y,
data->selected_range);
break;
case SENSOR_CHAN_ACCEL_Z:
adxl345_accel_convert(val, data->samples.z);
adxl345_accel_convert(val, data->samples.z,
data->selected_range);
break;
case SENSOR_CHAN_ACCEL_XYZ:
adxl345_accel_convert(val++, data->samples.x);
adxl345_accel_convert(val++, data->samples.y);
adxl345_accel_convert(val, data->samples.z);
adxl345_accel_convert(val, data->samples.x,
data->selected_range);
val++;
adxl345_accel_convert(val, data->samples.y,
data->selected_range);
val++;
adxl345_accel_convert(val, data->samples.z,
data->selected_range);
break;
default:
return -ENOTSUP;
Expand Down
1 change: 0 additions & 1 deletion drivers/sensor/adi/adxl345/adxl345.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,6 @@ int adxl345_read_sample(const struct device *dev, struct adxl345_sample *sample)
#ifdef CONFIG_SENSOR_ASYNC_API
void adxl345_submit(const struct device *dev, struct rtio_iodev_sqe *iodev_sqe);
int adxl345_get_decoder(const struct device *dev, const struct sensor_decoder_api **decoder);
void adxl345_accel_convert(struct sensor_value *val, int16_t sample);
#endif /* CONFIG_SENSOR_ASYNC_API */

#ifdef CONFIG_ADXL345_STREAM
Expand Down