drivers: sensor: omron: add d6f driver #95781
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zephyrbot
requested review from
asemjonovs,
avisconti,
decsny,
MaureenHelm,
mbolivar,
rruuaanng,
teburd,
tristan-google,
ubieda and
yperess
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
b761418 to
15dabc1
Compare
|
I forgot to add an offset property. Changing to draft. |
jeppenodgaard
force-pushed
the
sensors/flow
branch
5 times, most recently
from
0a50f26 to
714876c
Compare
MaureenHelm
previously approved these changes
Sep 15, 2025
decsny
reviewed
Sep 18, 2025
dts/bindings/vendor-prefixes.txt
Outdated
| oki Oki Electric Industry Co., Ltd. | ||
| olimex OLIMEX Ltd. | ||
| olpc One Laptop Per Child | ||
| omron Omron Corporation |
There was a problem hiding this comment.
@nashif please help review this prefix
There was a problem hiding this comment.
Changed to OMRON due to this comment
ubieda
reviewed
Sep 24, 2025
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
714876c to
bfa4932
Compare
zephyrbot
added
area: Tests
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
bfa4932 to
da052ab
Compare
ubieda
previously approved these changes
Sep 29, 2025
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
da052ab to
c7b6265
Compare
Add flow rate to `enum sensor_channel` in litres per minute. The SI unit for flow rate is cubic metres per second. Due to a sensor value resolution of 1/100000 this unit is not granular enough for low flow rate sensors. Signed-off-by: Jeppe Odgaard <jeppe.odgaard@prevas.dk>
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
c7b6265 to
9dff114
Compare
added 3 commits
September 30, 2025 21:02
Add OMRON vendor prefix. Signed-off-by: Jeppe Odgaard <jeppe.odgaard@prevas.dk>
Add support for Omron D6F mass flow rate sensor series. The sensor series outputs an analogue voltage which is read using an ADC. Signed-off-by: Jeppe Odgaard <jeppe.odgaard@prevas.dk>
Add Omron D6F to sensor build_all test. Signed-off-by: Jeppe Odgaard <jeppe.odgaard@prevas.dk>
jeppenodgaard
force-pushed
the
sensors/flow
branch
from
9dff114 to
a588938
Compare
|
ubieda
approved these changes
Oct 1, 2025
MaureenHelm
approved these changes
Oct 1, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add support for Omron D6F flow rate sensor series.
The sensor series outputs an analogue voltage which is read using an ADC.
A new sensor channel has been added for flow rate. The unit is litres per minute.
The SI unit for flow rate is cubic metres per second.
Due to a sensor value resolution of 1/100000 this unit is not granular enough for low flow rate sensors.