diff --git a/tool/src/ec.rs b/tool/src/ec.rs
index 868971737..0bd793784 100644
--- a/tool/src/ec.rs
+++ b/tool/src/ec.rs
@@ -14,7 +14,7 @@ use crate::{
     SpiTarget,
 };
 
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, Eq, PartialEq)]
 #[repr(u8)]
 enum Cmd {
     // None = 0,
@@ -46,7 +46,7 @@ const CMD_SPI_FLAG_DISABLE: u8 = 1 << 1;
 const CMD_SPI_FLAG_SCRATCH: u8 = 1 << 2;
 const CMD_SPI_FLAG_BACKUP: u8 = 1 << 3;
 
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, Eq, PartialEq)]
 #[repr(u8)]
 pub enum SecurityState {
     // Default value, flashing is prevented, cannot be set with security_set
diff --git a/tool/src/spi.rs b/tool/src/spi.rs
index b3d8cf343..abb6fe858 100644
--- a/tool/src/spi.rs
+++ b/tool/src/spi.rs
@@ -21,7 +21,7 @@ pub trait Spi {
 }
 
 /// Target which will receive SPI commands
-#[derive(Clone, Copy)]
+#[derive(Clone, Copy, Debug, Eq, PartialEq)]
 pub enum SpiTarget {
     /// The ROM normally used by the EC
     Main,