rename const generics

This commit is contained in:
dimi 2022-11-24 22:12:19 +01:00
parent f0876952b7
commit ab74fc47fd
3 changed files with 29 additions and 29 deletions

View File

@ -241,7 +241,7 @@ pub unsafe trait PwmPeripheral {
/// Get a pointer to the peripheral RegisterBlock
fn block() -> *const crate::pac::pwm0::RegisterBlock;
/// Get operator GPIO mux output signal
fn output_signal<const O: u8, const IS_A: bool>() -> OutputSignal;
fn output_signal<const OP: u8, const IS_A: bool>() -> OutputSignal;
}
unsafe impl PwmPeripheral for crate::pac::PWM0 {
@ -253,8 +253,8 @@ unsafe impl PwmPeripheral for crate::pac::PWM0 {
Self::ptr()
}
fn output_signal<const O: u8, const IS_A: bool>() -> OutputSignal {
match (O, IS_A) {
fn output_signal<const OP: u8, const IS_A: bool>() -> OutputSignal {
match (OP, IS_A) {
(0, true) => OutputSignal::PWM0_0A,
(1, true) => OutputSignal::PWM0_1A,
(2, true) => OutputSignal::PWM0_1A,
@ -275,8 +275,8 @@ unsafe impl PwmPeripheral for crate::pac::PWM1 {
Self::ptr()
}
fn output_signal<const O: u8, const IS_A: bool>() -> OutputSignal {
match (O, IS_A) {
fn output_signal<const OP: u8, const IS_A: bool>() -> OutputSignal {
match (OP, IS_A) {
(0, true) => OutputSignal::PWM1_0A,
(1, true) => OutputSignal::PWM1_1A,
(2, true) => OutputSignal::PWM1_1A,

View File

@ -15,11 +15,11 @@ use crate::{
/// * Superimposes a carrier on the PWM signal, if configured to do so. (Not yet
/// implemented)
/// * Handles response under fault conditions. (Not yet implemented)
pub struct Operator<const O: u8, PWM> {
pub struct Operator<const OP: u8, PWM> {
phantom: PhantomData<PWM>,
}
impl<const O: u8, PWM: PwmPeripheral> Operator<O, PWM> {
impl<const OP: u8, PWM: PwmPeripheral> Operator<OP, PWM> {
pub(super) fn new() -> Self {
// TODO maybe set timersel to 3 to disable?
@ -32,13 +32,13 @@ impl<const O: u8, PWM: PwmPeripheral> Operator<O, PWM> {
///
/// ### Note:
/// By default TIMER0 is used
pub fn set_timer<const T: u8>(&mut self, timer: &Timer<T, PWM>) {
pub fn set_timer<const TIM: u8>(&mut self, timer: &Timer<TIM, PWM>) {
let _ = timer;
let block = unsafe { &*PWM::block() };
block.operator_timersel.modify(|_, w| match O {
0 => w.operator0_timersel().variant(T),
1 => w.operator1_timersel().variant(T),
2 => w.operator2_timersel().variant(T),
block.operator_timersel.modify(|_, w| match OP {
0 => w.operator0_timersel().variant(TIM),
1 => w.operator1_timersel().variant(TIM),
2 => w.operator2_timersel().variant(TIM),
_ => {
unreachable!()
}
@ -50,7 +50,7 @@ impl<const O: u8, PWM: PwmPeripheral> Operator<O, PWM> {
self,
pin: Pin,
config: PwmPinConfig<true>,
) -> PwmPin<Pin, PWM, O, true> {
) -> PwmPin<Pin, PWM, OP, true> {
PwmPin::new(pin, config)
}
@ -59,7 +59,7 @@ impl<const O: u8, PWM: PwmPeripheral> Operator<O, PWM> {
self,
pin: Pin,
config: PwmPinConfig<false>,
) -> PwmPin<Pin, PWM, O, false> {
) -> PwmPin<Pin, PWM, OP, false> {
PwmPin::new(pin, config)
}
@ -70,7 +70,7 @@ impl<const O: u8, PWM: PwmPeripheral> Operator<O, PWM> {
config_a: PwmPinConfig<true>,
pin_b: PinB,
config_b: PwmPinConfig<false>,
) -> (PwmPin<PinA, PWM, O, true>, PwmPin<PinB, PWM, O, false>) {
) -> (PwmPin<PinA, PWM, OP, true>, PwmPin<PinB, PWM, OP, false>) {
(PwmPin::new(pin_a, config_a), PwmPin::new(pin_b, config_b))
}
}
@ -104,14 +104,14 @@ impl<const IS_A: bool> PwmPinConfig<IS_A> {
}
/// A pin driven by an MCPWM operator
pub struct PwmPin<Pin, PWM, const O: u8, const IS_A: bool> {
pub struct PwmPin<Pin, PWM, const OP: u8, const IS_A: bool> {
_pin: Pin,
phantom: PhantomData<PWM>,
}
impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<Pin, PWM, O, IS_A> {
impl<Pin: OutputPin, PWM: PwmPeripheral, const OP: u8, const IS_A: bool> PwmPin<Pin, PWM, OP, IS_A> {
fn new(mut pin: Pin, config: PwmPinConfig<IS_A>) -> Self {
let output_signal = PWM::output_signal::<O, IS_A>();
let output_signal = PWM::output_signal::<OP, IS_A>();
pin.enable_output(true)
.connect_peripheral_to_output(output_signal);
let mut pin = PwmPin {
@ -134,7 +134,7 @@ impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<P
// SAFETY:
// `bits` is a valid bit pattern
unsafe {
match (O, IS_A) {
match (OP, IS_A) {
(0, true) => block.gen0_a.write(|w| w.bits(bits)),
(1, true) => block.gen1_a.write(|w| w.bits(bits)),
(2, true) => block.gen2_a.write(|w| w.bits(bits)),
@ -151,7 +151,7 @@ impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<P
pub fn set_update_method(&mut self, update_method: PwmUpdateMethod) {
let block = unsafe { &*PWM::block() };
let bits = update_method.0;
match (O, IS_A) {
match (OP, IS_A) {
(0, true) => block
.gen0_stmp_cfg
.modify(|_, w| w.gen0_a_upmethod().variant(bits)),
@ -181,7 +181,7 @@ impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<P
pub fn set_update_method(&mut self, update_method: PwmUpdateMethod) {
let block = unsafe { &*PWM::block() };
let bits = update_method.0;
match (O, IS_A) {
match (OP, IS_A) {
(0, true) => block
.cmpr0_cfg
.modify(|_, w| w.cmpr0_a_upmethod().variant(bits)),
@ -212,7 +212,7 @@ impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<P
#[cfg(esp32)]
pub fn set_timestamp(&mut self, value: u16) {
let block = unsafe { &*PWM::block() };
match (O, IS_A) {
match (OP, IS_A) {
(0, true) => block.gen0_tstmp_a.write(|w| w.gen0_a().variant(value)),
(1, true) => block.gen1_tstmp_a.write(|w| w.gen1_a().variant(value)),
(2, true) => block.gen2_tstmp_a.write(|w| w.gen2_a().variant(value)),
@ -231,7 +231,7 @@ impl<Pin: OutputPin, PWM: PwmPeripheral, const O: u8, const IS_A: bool> PwmPin<P
#[cfg(esp32s3)]
pub fn set_timestamp(&mut self, value: u16) {
let block = unsafe { &*PWM::block() };
match (O, IS_A) {
match (OP, IS_A) {
(0, true) => block.cmpr0_value0.write(|w| w.cmpr0_a().variant(value)),
(1, true) => block.cmpr1_value0.write(|w| w.cmpr1_a().variant(value)),
(2, true) => block.cmpr2_value0.write(|w| w.cmpr2_a().variant(value)),

View File

@ -12,11 +12,11 @@ use crate::{
/// Every timer of a particular [`MCPWM`](super::MCPWM) peripheral can be used
/// as a timing reference for every
/// [`Operator`](super::operator::Operator) of that peripheral
pub struct Timer<const T: u8, PWM> {
pub struct Timer<const TIM: u8, PWM> {
pub(super) phantom: PhantomData<PWM>,
}
impl<const T: u8, PWM: PwmPeripheral> Timer<T, PWM> {
impl<const TIM: u8, PWM: PwmPeripheral> Timer<TIM, PWM> {
pub(super) fn new() -> Self {
Timer {
phantom: PhantomData,
@ -63,7 +63,7 @@ impl<const T: u8, PWM: PwmPeripheral> Timer<T, PWM> {
pub fn set_counter(&mut self, phase: u16) {
// FIXME add phase_direction to SVD, then add CounterDirection as a parameter
let block = unsafe { &*PWM::block() };
match T {
match TIM {
0 => block.timer0_sync.modify(|r, w| {
w.timer0_phase()
.variant(phase as u32)
@ -92,7 +92,7 @@ impl<const T: u8, PWM: PwmPeripheral> Timer<T, PWM> {
pub fn status(&self) -> (u16, CounterDirection) {
let block = unsafe { &*PWM::block() };
match T {
match TIM {
0 => {
let reg = block.timer0_status.read();
(
@ -128,7 +128,7 @@ impl<const T: u8, PWM: PwmPeripheral> Timer<T, PWM> {
// SAFETY:
// The CFG0 registers are identical for all timers so we can pretend they're
// TIMER0_CFG0
match T {
match TIM {
0 => &block.timer0_cfg0,
1 => unsafe { &*(&block.timer1_cfg0 as *const _ as *const _) },
2 => unsafe { &*(&block.timer2_cfg0 as *const _ as *const _) },
@ -145,7 +145,7 @@ impl<const T: u8, PWM: PwmPeripheral> Timer<T, PWM> {
// SAFETY:
// The CFG1 registers are identical for all timers so we can pretend they're
// TIMER0_CFG1
match T {
match TIM {
0 => &block.timer0_cfg1,
1 => unsafe { &*(&block.timer1_cfg1 as *const _ as *const _) },
2 => unsafe { &*(&block.timer2_cfg1 as *const _ as *const _) },