alex/rp-hal-boards
1
0
Fork 0
mirror of https://github.com/italicsjenga/rp-hal-boards.git synced 2025-01-11 04:51:31 +11:00
Code Issues Packages Projects Releases Wiki Activity

Rename set_instruction to exec_instruction

Browse source
This commit is contained in:
Jan Niehusmann 2021-11-01 20:01:20 +00:00
parent 3977bdd8c5
commit 3756cd2fb5
1 changed files with 16 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
rp2040-hal/src/pio.rs
View file

@ -473,11 +473,11 @@ impl<SM: ValidStateMachine> UninitStateMachine<SM> {
});
}
/// Set the current instruction.
/// Execute the instruction immediately.
// Safety: The Send trait assumes this is the only write to sm_instr while uninitialized. The
// initialized `StateMachine` may also use this register. The `UnintStateMachine` is consumed
// by `PIOBuilder.build` to create `StateMachine`
fn set_instruction(&mut self, instruction: u16) {
fn exec_instruction(&mut self, instruction: u16) {
self.sm()
.sm_instr
.write(|w| unsafe { w.sm0_instr().bits(instruction) })
@ -519,10 +519,14 @@ impl<SM: ValidStateMachine, State> StateMachine<SM, State> {
self.sm.sm().sm_addr.read().bits()
}
/// Set the current instruction.
pub fn set_instruction(&mut self, instruction: u16) {
// TODO: Check if this function is safe to call while the state machine is running.
self.sm.set_instruction(instruction);
///Execute the instruction immediately.
pub fn exec_instruction(&mut self, instruction: u16) {
// This is allowed even if the state machine is running.
//
// However, the datasheet says:
// "If EXEC instructions are used, instructions written to INSTR must not stall"
// It's unclear what happens if this is violated.
self.sm.exec_instruction(instruction);
}
/// Check if the current instruction is stalled.
@ -564,7 +568,7 @@ impl<SM: ValidStateMachine> StateMachine<SM, Stopped> {
.sm()
.sm_pinctrl
.write(|w| unsafe { w.set_base().bits(pin_num).set_count().bits(1) });
self.set_instruction(
self.exec_instruction(
pio::InstructionOperands::SET {
destination: pio::SetDestination::PINS,
data: if PinState::High == pin_state { 1 } else { 0 },
@ -593,7 +597,7 @@ impl<SM: ValidStateMachine> StateMachine<SM, Stopped> {
.sm()
.sm_pinctrl
.write(|w| unsafe { w.set_base().bits(pinnum).set_count().bits(1) });
self.set_instruction(
self.exec_instruction(
pio::InstructionOperands::SET {
destination: pio::SetDestination::PINDIRS,
data: if PinDir::Output == pin_dir { 1 } else { 0 },
@ -680,7 +684,7 @@ impl<SM: ValidStateMachine> StateMachine<SM, Running> {
// pause the state machine
self.sm.set_enabled(false);
// revert it to its wrap target
self.sm.set_instruction(
self.sm.exec_instruction(
pio::InstructionOperands::JMP {
condition: pio::JmpCondition::Always,
address: self.program.wrap_target(),
@ -829,7 +833,7 @@ impl<SM: ValidStateMachine> Tx<SM> {
}
.encode();
// Safety: The only other place this register is written is
// `UninitStatemachine.set_instruction`, `Tx` is only created after init.
// `UninitStatemachine.exec_instruction`, `Tx` is only created after init.
let mask = 1 << SM::id();
while self.register_block().fstat.read().txempty().bits() & mask != mask {
self.register_block().sm[SM::id()]
@ -1429,9 +1433,9 @@ impl<P: PIOExt> PIOBuilder<P> {
sm.restart();
sm.reset_clock();
// Set starting location by setting the state machine to execute a jmp
// Set starting location by forcing the state machine to execute a jmp
// to the beginning of the program we loaded in.
sm.set_instruction(
sm.exec_instruction(
pio::InstructionOperands::JMP {
condition: pio::JmpCondition::Always,
address: offset as u8,