Skip to content

Add BWP and performance mode API to BMI270#800

Merged
PetrESP merged 1 commit into
masterfrom
petr/feat/bmi270_bwp
Jun 24, 2026
Merged

Add BWP and performance mode API to BMI270#800
PetrESP merged 1 commit into
masterfrom
petr/feat/bmi270_bwp

Conversation

@PetrESP

@PetrESP PetrESP commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

ESP-BSP Pull Request checklist

  • Version of modified component bumped
  • CI passing

Change description

Added bandwidth and filter performance API
Closes #792


Note

Low Risk
Additive, opt-in sensor driver API with no changes to default init or data paths; risk is mainly misconfigured register values affecting IMU quality or power.

Overview
Bumps the bmi270 component to 1.1.0 and exposes optional tuning for accelerometer and gyroscope beyond existing ODR/range setters.

New public enums cover accelerometer bandwidth (bmi270_acce_bwp_e), gyro bandwidth (bmi270_gyro_bwp_e), and power vs performance (bmi270_perf_e). Five setters read-modify-write BMI270_ACC_CONF_REG / BMI270_GYR_CONF_REG with the same validation as other config helpers: bmi270_set_acce_bwp, bmi270_set_acce_filter_perf, bmi270_set_gyro_bwp, bmi270_set_gyro_noise_perf, and bmi270_set_gyro_filter_perf. Private bit masks and field positions were added to support those writes.

bmi270_start and bmi270_config_t are unchanged, so apps must call the new APIs explicitly when they want different filtering or performance.

Reviewed by Cursor Bugbot for commit a42bc0f. Bugbot is set up for automated code reviews on this repo. Configure here.

@PetrESP PetrESP self-assigned this Jun 23, 2026
@PetrESP PetrESP requested a review from espzav June 23, 2026 14:58
Comment thread components/sensors/bmi270/idf_component.yml Outdated

@espzav espzav left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PetrESP Thank you! LGTM

@PetrESP PetrESP force-pushed the petr/feat/bmi270_bwp branch from 9f18098 to a42bc0f Compare June 24, 2026 11:49
@github-actions

Copy link
Copy Markdown

⚠️ Build failure was ignored (continue-on-error enabled).

See details: View build logs

@PetrESP PetrESP merged commit 0c926f9 into master Jun 24, 2026
99 checks passed
@PetrESP PetrESP deleted the petr/feat/bmi270_bwp branch June 24, 2026 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add config to espressif/bmi270 1.0.1~1

2 participants