--boardid: Print all available board IDs

Using a native host command to get the ID from the EC
instead of having the sotware measure ADC and evaluate the host command.

This is better because the ADC values represent different board IDs on
different platforms (Microchip vs Nuvoton based ECs)

> sudo framework_tool --boardid
Mainboard   Board ID: Ok(Some(4))
PowerButton Board ID: Err(Response(InvalidResponse))
Touchpad    Board ID: Err(Response(InvalidResponse))
AudioBoard  Board ID: Err(Response(InvalidResponse))
dGPU0       Board ID: Err(Response(InvalidResponse))
dGPU1       Board ID: Err(Response(InvalidResponse))
```

Signed-off-by: Daniel Schaefer <[email protected]>

Author: JohnAZoidberg

framework_lib/src/commandline/clap_std.rs

@@ -273,6 +273,10 @@ struct ClapCli {
     #[arg(long)]
     test_retimer: bool,
 
+    /// Print all board IDs
+    #[arg(long)]
+    boardid: bool,
+
     /// Force execution of an unsafe command - may render your hardware unbootable!
     #[arg(long, short)]
     force: bool,
@@ -472,6 +476,7 @@ pub fn parse(args: &[String]) -> Cli {
         pd_ports,
         test: args.test,
         test_retimer: args.test_retimer,
+        boardid: args.boardid,
         dry_run: args.dry_run,
         force: args.force,
         // TODO: Set help. Not very important because Clap handles this by itself

framework_lib/src/commandline/mod.rs

@@ -35,6 +35,7 @@ use crate::ccgx::device::{FwMode, PdController, PdPort};
 use crate::ccgx::hid::{check_ccg_fw_version, find_devices, DP_CARD_PID, HDMI_CARD_PID};
 use crate::ccgx::{self, MainPdVersions, PdVersions, SiliconId::*};
 use crate::chromium_ec;
+use crate::chromium_ec::commands::BoardIdType;
 use crate::chromium_ec::commands::DeckStateMode;
 use crate::chromium_ec::commands::FpLedBrightnessLevel;
 use crate::chromium_ec::commands::RebootEcCmd;
@@ -192,6 +193,7 @@ pub struct Cli {
     pub driver: Option<CrosEcDriverType>,
     pub test: bool,
     pub test_retimer: bool,
+    pub boardid: bool,
     pub dry_run: bool,
     pub force: bool,
     pub intrusion: bool,
@@ -279,6 +281,7 @@ pub fn parse(args: &[String]) -> Cli {
             driver: cli.driver,
             test: cli.test,
             test_retimer: cli.test_retimer,
+            boardid: cli.boardid,
             dry_run: cli.dry_run,
             // force
             intrusion: cli.intrusion,
@@ -1487,6 +1490,8 @@ pub fn run_with_args(args: &Cli, _allupdate: bool) -> i32 {
         if let Err(err) = selftest_retimer(&ec) {
             println!("  Failed: {:?}", err);
         }
+    } else if args.boardid {
+        print_board_ids(&ec);
     } else if args.power {
         return power::get_and_print_power_info(&ec);
     } else if args.thermal {
@@ -1850,6 +1855,16 @@ fn hash(data: &[u8]) {
     util::print_buffer(sha512);
 }
 
+fn print_board_ids(ec: &CrosEc) {
+    println!("Board IDs");
+    println!("  Mainboard:    {:?}", ec.read_board_id_hc(BoardIdType::Mainboard));
+    println!("  PowerButton:  {:?}", ec.read_board_id_hc(BoardIdType::PowerButtonBoard));
+    println!("  Touchpad:     {:?}", ec.read_board_id_hc(BoardIdType::Touchpad));
+    println!("  AudioBoard:   {:?}", ec.read_board_id_hc(BoardIdType::AudioBoard));
+    println!("  dGPU0:        {:?}", ec.read_board_id_hc(BoardIdType::DGpu0));
+    println!("  dGPU1:        {:?}", ec.read_board_id_hc(BoardIdType::DGpu1));
+}
+
 fn selftest(ec: &CrosEc) -> Option<()> {
     if let Some(platform) = smbios::get_platform() {
         println!("  SMBIOS Platform:     {:?}", platform);

framework_lib/src/commandline/uefi.rs

@@ -87,6 +87,7 @@ pub fn parse(args: &[String]) -> Cli {
         pd_ports: None,
         test: false,
         test_retimer: false,
+        boardid: false,
         dry_run: false,
         force: false,
         help: false,
@@ -511,6 +512,9 @@ pub fn parse(args: &[String]) -> Cli {
         } else if arg == "-t" || arg == "--test-retimer" {
             cli.test_retimer = true;
             found_an_option = true;
+        } else if arg == "--boardid" {
+            cli.boardid = true;
+            found_an_option = true;
         } else if arg == "-f" || arg == "--force" {
             cli.force = true;
             found_an_option = true;