Skip to content
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.

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

Add DT bindings for sensor channels #71198

Closed

Conversation

yperess
Copy link
Collaborator

@yperess yperess commented Apr 8, 2024

This is a first in 3 PRs that convert some of the current Sensor enum values (channel, attribute, and triggers) into DT compatible #defines. This will allow statically allocating the right amount of storage both inside and outside the drivers.

TODO: Add unit tests.

@yperess yperess requested a review from teburd April 8, 2024 05:01
@yperess yperess added the area: Sensors Sensors label Apr 8, 2024
@teburd teburd mentioned this pull request Apr 8, 2024
Move the sensor channel enum to a dts includable headers. This means
that the enum had to change to #define calls. In order to keep things
backwards compatible, There's a shim header that undefines them and
allows the enum to exist for now while we work on deprecating it.

Signed-off-by: Yuval Peress <[email protected]>
Once added, these should be accessible via the devicetree macros.

Signed-off-by: Yuval Peress <[email protected]>
@yperess yperess force-pushed the peress/dts-sensor-bindings branch from b1102e8 to 3cef8e9 Compare May 20, 2024 18:55
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Jul 20, 2024
@github-actions github-actions bot closed this Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant