ccgx: Add reset_device function

framwork_tool --pd-reset 0
framwork_tool --pd-reset 1

Signed-off-by: Daniel Schaefer <dhs@frame.work>

Author: JohnAZoidberg

framework_lib/src/ccgx/device.rs

@@ -358,4 +358,12 @@ impl PdController {
             base_ver, app_ver
         );
     }
+
+    pub fn reset_device(&self) -> EcResult<()> {
+        self.ccgx_write(
+            ControlRegisters::ResetRequest,
+            &[HPI_RESET_SIGNATURE as u8, HPI_RESET_DEV_CMD],
+        )?;
+        Ok(())
+    }
 }

framework_lib/src/commandline/clap_std.rs

@@ -83,6 +83,10 @@ struct ClapCli {
     #[arg(long)]
     pd_info: bool,
 
+    /// Reset a specific PD controller (for debugging only)
+    #[arg(long)]
+    pd_reset: Option<u8>,
+
     /// Show details about connected DP or HDMI Expansion Cards
     #[arg(long)]
     dp_hdmi_info: bool,
@@ -376,6 +380,7 @@ pub fn parse(args: &[String]) -> Cli {
         autofanctrl: args.autofanctrl,
         pdports: args.pdports,
         pd_info: args.pd_info,
+        pd_reset: args.pd_reset,
         dp_hdmi_info: args.dp_hdmi_info,
         dp_hdmi_update: args
             .dp_hdmi_update

framework_lib/src/commandline/mod.rs

@@ -156,6 +156,7 @@ pub struct Cli {
     pub pdports: bool,
     pub privacy: bool,
     pub pd_info: bool,
+    pub pd_reset: Option<u8>,
     pub dp_hdmi_info: bool,
     pub dp_hdmi_update: Option<String>,
     pub audio_card_info: bool,
@@ -999,6 +1000,17 @@ pub fn run_with_args(args: &Cli, _allupdate: bool) -> i32 {
         smbios_info();
     } else if args.pd_info {
         print_pd_details(&ec);
+    } else if let Some(pd) = args.pd_reset {
+        println!("Resetting PD {}...", pd);
+        print_err(match pd {
+            0 => PdController::new(PdPort::Left01, ec.clone()).reset_device(),
+            1 => PdController::new(PdPort::Right23, ec.clone()).reset_device(),
+            2 => PdController::new(PdPort::Back, ec.clone()).reset_device(),
+            _ => {
+                error!("PD {} does not exist", pd);
+                Ok(())
+            }
+        });
     } else if args.dp_hdmi_info {
         #[cfg(feature = "hidapi")]
         print_dp_hdmi_details(true);

framework_lib/src/commandline/uefi.rs

@@ -70,6 +70,7 @@ pub fn parse(args: &[String]) -> Cli {
         autofanctrl: false,
         pdports: false,
         pd_info: false,
+        pd_reset: None,
         dp_hdmi_info: false,
         dp_hdmi_update: None,
         audio_card_info: false,
@@ -518,6 +519,22 @@ pub fn parse(args: &[String]) -> Cli {
         } else if arg == "--pd-info" {
             cli.pd_info = true;
             found_an_option = true;
+        } else if arg == "--pd-reset" {
+            cli.pd_reset = if args.len() > i + 1 {
+                if let Ok(pd) = args[i + 1].parse::<u8>() {
+                    Some(pd)
+                } else {
+                    println!(
+                        "Invalid value for --pd-reset: '{}'. Must be 0 or 1.",
+                        args[i + 1],
+                    );
+                    None
+                }
+            } else {
+                println!("--pd-reset requires specifying the PD controller");
+                None
+            };
+            found_an_option = true;
         } else if arg == "--privacy" {
             cli.privacy = true;
             found_an_option = true;