--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
Board IDs
  Mainboard:    Ok(Some(6))
  PowerButton:  Err(Response(InvalidResponse))
  Touchpad:     Ok(Some(7))
  AudioBoard:   Ok(Some(7))
  dGPU0:        Err(Response(InvalidCommand))
  dGPU1:        Err(Response(InvalidCommand))
```

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

Author: JohnAZoidberg

EXAMPLES.md

@@ -380,6 +380,26 @@ Positions:
   Pos 4: GenericC
 ```
 
+### Checking board ID
+
+Most inputdeck checking is implemented by Board ID. To read those directly for
+debugging low level issues, use the `--boardid` command.
+This host command was added recently is not available on every system yet.
+
+```
+# Example on Framework Laptop 13
+# Touchpad Board ID is only present on Laptop 12
+# dGPU Board IDs are only present on Laptop 16
+> framework_tool --boardid
+Board IDs
+  Mainboard:    Ok(Some(6))
+  PowerButton:  Err(Response(InvalidResponse))
+  Touchpad:     Ok(Some(7))
+  AudioBoard:   Ok(Some(7))
+  dGPU0:        Err(Response(InvalidCommand))
+  dGPU1:        Err(Response(InvalidCommand))
+```
+
 ## Check temperatures and fan speed
 
 ```

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,34 @@ 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;