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.

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

Jump to bottom

Current amplifier sensor driver #60415

Closed
nixward opened this issue Jul 16, 2023 · 1 comment
Closed

Current amplifier sensor driver #60415

nixward opened this issue Jul 16, 2023 · 1 comment
Assignees
@nixward
Labels
Feature Request A request for a new feature

Comments

@nixward
Copy link
Member

nixward commented Jul 16, 2023

Is your feature request related to a problem? Please describe.
Similar to other ADC based sensors, like the mcp970x temperature sensor, it would be great to have a current amplifier sensor.

Describe the solution you'd like
A current sensor based on the current-sense-amplifier binding.

Describe alternatives you've considered
Using the current-sense-amplifier binding binding on it's own.

Additional context
Having access to one or more current amplifier sensors from the Zephyr shell with only a little bit of Devicetree configuration is a convenient feature to have.
@nixward nixward added the Feature Request A request for a new feature label Jul 16, 2023
@nixward nixward self-assigned this Jul 16, 2023
@nixward nixward changed the title Current amplifier driver Current amplifier sensor driver Jul 16, 2023
nixward added a commit to nixward/zephyr that referenced this issue Jul 16, 2023
@nixward
drivers: sensor: current_amp: add driver
62bf9e4 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
@nixward nixward mentioned this issue Jul 16, 2023
Merged
nixward added a commit to nixward/zephyr that referenced this issue Jul 16, 2023
@nixward
drivers: sensor: current_amp: add driver
05aaead 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
nixward added a commit to nixward/zephyr that referenced this issue Jul 17, 2023
@nixward
drivers: sensor: current_amp: add driver
1fd7c4b 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
nixward added a commit to nixward/zephyr that referenced this issue Aug 10, 2023
@nixward
drivers: sensor: current_amp: add driver
283f6a3 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
nixward added a commit to nixward/zephyr that referenced this issue Aug 28, 2023
@nixward
drivers: sensor: current_amp: add driver
45091e7 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
carlescufi pushed a commit that referenced this issue Sep 13, 2023
@nixward @carlescufi
drivers: sensor: current_amp: add driver
17f2046 
Add current sense amplifier driver.

Implements #60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
coreboot-org-bot pushed a commit to coreboot/zephyr-cros that referenced this issue Sep 14, 2023
@nixward
drivers: sensor: current_amp: add driver
c897434 
Add current sense amplifier driver.

Implements zephyrproject-rtos/zephyr#60415

(cherry picked from commit 17f2046)

Original-Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>
Original-Signed-off-by: Nick Ward <nix.ward@gmail.com>
GitOrigin-RevId: 17f2046
Change-Id: I951d43d93671c2c5ce988bfe646de4e5e5129bc5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/4862407
Reviewed-by: Tristan Honscheid <honscheid@google.com>
Tested-by: Tristan Honscheid <honscheid@google.com>
Tested-by: ChromeOS Prod (Robot) <chromeos-ci-prod@chromeos-bot.iam.gserviceaccount.com>
Commit-Queue: Tristan Honscheid <honscheid@google.com>
Dat-NguyenDuy pushed a commit to nxp-zephyr/zephyr that referenced this issue Sep 20, 2023
@nixward @Dat-NguyenDuy
drivers: sensor: current_amp: add driver
ac1bb6c 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
coran21 pushed a commit to coran21/zephyr that referenced this issue Sep 21, 2023
@nixward @coran21
drivers: sensor: current_amp: add driver
3de8e87 
Add current sense amplifier driver.

Implements zephyrproject-rtos#60415

Co-authored-by: Marco Argiolas <marco.argiolas@ftpsolutions.com.au>

Signed-off-by: Nick Ward <nix.ward@gmail.com>
@nixward
Copy link
Member Author

nixward commented Apr 5, 2024

Merged: #59139

@nixward nixward closed this as completed Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nixward nixward

Labels
Feature Request A request for a new feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nixward