Skip to content

Commit

Permalink
unbreak grapefruit again
Browse files Browse the repository at this point in the history
  • Loading branch information
hawkw committed Dec 18, 2024
1 parent 007470b commit 4e838fb
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions drv/grapefruit-seq-server/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -270,14 +270,17 @@ impl<S: SpiServer + Clone> ServerImpl<S> {
self.jefe.set_state(state as u32);
}

fn validate_state_change(&self, state: PowerState) -> Result<(), SeqError> {
fn validate_state_change(
&self,
state: PowerState,
) -> Result<(), drv_cpu_seq_api::SeqError> {
match (self.get_state_impl(), state) {
(PowerState::A2, PowerState::A0)
| (PowerState::A0, PowerState::A2)
| (PowerState::A0PlusHP, PowerState::A2)
| (PowerState::A0Thermtrip, PowerState::A2) => Ok(()),

_ => Err(SeqError::IllegalTransition),
_ => Err(drv_cpu_seq_api::SeqError::IllegalTransition),
}
}
}
Expand All @@ -295,9 +298,9 @@ impl<S: SpiServer + Clone> idl::InOrderSequencerImpl for ServerImpl<S> {

fn set_state(
&mut self,
msg: &RecvMessage,
_: &RecvMessage,
state: PowerState,
) -> Result<(), RequestError<SeqError>> {
) -> Result<(), RequestError<drv_cpu_seq_api::SeqError>> {
self.validate_state_change(state)?;
self.set_state_impl(state);
Ok(())
Expand All @@ -308,7 +311,7 @@ impl<S: SpiServer + Clone> idl::InOrderSequencerImpl for ServerImpl<S> {
_: &RecvMessage,
state: PowerState,
_: StateChangeReason,
) -> Result<(), RequestError<SeqError>> {
) -> Result<(), RequestError<drv_cpu_seq_api::SeqError>> {
self.validate_state_change(state)?;
self.set_state_impl(state);
Ok(())
Expand Down

0 comments on commit 4e838fb

Please sign in to comment.