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

SNS - Levitation Current Sensor #71

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
1 change: 1 addition & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 10 additions & 0 deletions boards/stm32l432kc/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

28 changes: 28 additions & 0 deletions boards/stm32l432kc/src/tasks/current_levitation.rs
Aux1r marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
use embassy_stm32::adc::Adc;
use embassy_time::Delay;
use hyped_sensors::{current_levitation::CurrentLevitation, SensorValueRange::*};
use defmt_rtt as _;


/// Test task that reads the current and prints it to console
#[embassy_executor::task]
pub async fn read_current_levitation() -> ! {
let p = embassy_stm32::init(Default::default());
let adc = Adc::new(p.ADC1, Delay);

let mut current_levitation_sensor = CurrentLevitation::new(&mut adc);

loop {
match current_levitation_sensor.read() {
Safe(value) => {
defmt::info!("Current: {} A (safe)", value)
}
Warning(value) => {
defmt::warn!("Current: {} A (warning)", value)
}
Critical(value) => {
defmt::error!("Current: {} A (critical)", value)
}
}
}
}
1 change: 1 addition & 0 deletions lib/sensors/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ embassy-sync = { version = "0.6.0", features = ["defmt"], git = "https://github.
hyped_core = { path = "../core" }
hyped_i2c = { path = "../io/hyped_i2c" }
hyped_gpio = { path = "../io/hyped_gpio" }
hyped_adc = { path = "../io/hyped_adc" }
defmt = "0.3"

[dev-dependencies]
Expand Down
52 changes: 52 additions & 0 deletions lib/sensors/src/current_levitation.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
use crate::SensorValueRange;
use hyped_adc::HypedAdc;

/// current_levitation implements the logic to read current from the ACHS-7121 current sensor using the
/// Hyped ADC trait.
///
/// The code constantly reads the channel for a current reading. It then processes this reading into the right format
/// (as seen on the data sheet) and returns this reading for use.
///
Aux1r marked this conversation as resolved.
Show resolved Hide resolved
/// The ACHS-7121 has a current range of +- 10 A, sensitivity of 185 mV/A.
/// Assuming we're supplying 5V to the sensor, our off-set is 2.5V in the output reading - note that this offset is given
/// by the supply voltage divided by 2, so if you change the supply voltage, you'll have to change the offset that we subtract
/// in the processed_current accordingly.
Aux1r marked this conversation as resolved.
Show resolved Hide resolved
///
/// Data sheet: https://docs.broadcom.com/doc/ACHS-712x-DS

Aux1r marked this conversation as resolved.
Show resolved Hide resolved
pub struct CurrentLevitation<T: HypedAdc> {
adc: T,
calculate_bounds: fn(f32) -> SensorValueRange<f32>,
}

impl<T: HypedAdc> CurrentLevitation<T> {
/// Create a new instance of the Current Levitation sensor
pub fn new(adc: T) -> CurrentLevitation<T> {
CurrentLevitation {
adc,
calculate_bounds: default_calculate_bounds,
Aux1r marked this conversation as resolved.
Show resolved Hide resolved
}
}
/// Read the voltage from the Vout Pin and process it before returning it - you have to subtract the voltage offset and divide by the sensitivity
pub fn read(&mut self) -> SensorValueRange<f32> {
let current = self.adc.read_value() as f32;
Copy link
Collaborator

Choose a reason for hiding this comment

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

you need to convert the value read on ADC into volts first (currently will be 0-4096)

Choose a reason for hiding this comment

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

is that something that can be done as part of the adc implementation?

Copy link
Collaborator

Choose a reason for hiding this comment

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

getting the resolution of the ADC is being added in #82 if that's what you mean?

I guess we could also add functionality to map it to a given range (e.g. 0-3.3) too

(self.calculate_bounds)((current - OFFSET) / SENSITIVITY)
}
}

pub fn default_calculate_bounds(value: f32) -> SensorValueRange<f32> {
if value <= MIN_AMPS || value >= MAX_AMPS {
SensorValueRange::Critical(value)
} else if value <= WARN_AMPS_LOW || value >= WARN_AMPS_HIGH {
SensorValueRange::Warning(value)
} else {
SensorValueRange::Safe(value)
}
}

const OFFSET: f32 = 2.5;
const SENSITIVITY: f32 = 0.185;
const MIN_AMPS: f32 = 10.0;
const MAX_AMPS: f32 = -10.0;
const WARN_AMPS_LOW: f32 = -8.0;
const WARN_AMPS_HIGH: f32 = 8.0;
1 change: 1 addition & 0 deletions lib/sensors/src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
#![no_std]

pub mod current_levitation;
pub mod keyence;
pub mod temperature;
pub mod time_of_flight;
Expand Down
Loading