MCXA HIL tests

[diondokter]

Let's collect the HIL tests and setup we will want.

Interesting peripherals:

• GPIO + interrupts (reuse ADC pins)
• I2C controller + target
• Uart
• ADC
• I3C
• CRC
• Watchdog (but how to test? reset reason?)
• Flash
• CTimer PWM
• CTimer capture (reuse ADC pins)
• SPI

We'll need to find the right wiring setup to be able to do this.

• I2C controller + target: LPI2C1 on arduino connector J4 to LPI2C2 on camera connector J9
  • SDA: P1_0 -> P1_8
  • SCL: P1_1 -> P1_9
• Uart: LPUART3 on mikroBUS connector J5 to LPUART4 on flexIO connector J8
  • P4_5 (TX) -> P4_4 (RX)
  • P4_2 (RX) -> P4_3 (TX)
• ADC + GPIO + CTimer capture + CTimer PWM: ADC0_A1 on arduino connector J2 to CT_INP8 + CT0_MAT2 on arduino connector J2
  • P2_4 -> P1_8 (overlaps with I2C SDA but on different connector)
• I3C: Only one peripheral, so no loopback tests. But it's connected to the P3T1755DP temperature sensor
• SPI: LPSPI0 on mikroBUS connector J6 to LPSPI1 on arduino connector J2
  • CS: P1_3 -> P3_11
  • SCK: P1_1 -> P3_10 (overlaps with I2C SCL but on different connector)
  • MISO/SDI: P1_2 -> P3_9
  • MOSI/SDO: P1_0 -> P3_8

That's 9 wire connections in total.

There could be more in the future, like DAC + OPAMP -> ADC or CAN0 -> CAN1. But I don't think they're in scope now.

Building the HIL tests should be done here: https://github.com/embassy-rs/embassy/tree/main/tests

[jamesmunns]

You haven't bumped into it yet, but just a reminder/note for later that J7's headers aren't populated, so we should avoid it for now (we can add it if we need to, it'll just require rework)

[diondokter]

@jamesmunns WIP branch here: https://github.com/diondokter/embassy/tree/mcxa-hil

[felipebalbi]

@diondokter, FYI the P3T1755DP is compatible with the TMP108

[felipebalbi]

@jamesmunns WIP branch here: https://github.com/diondokter/embassy/tree/mcxa-hil

Looks like there are issues:

     Running `teleprobe local run --chip MCXA276 --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\crc`
 WARN  probe_rs::config::registry > Ignoring 4 ambiguous matches for specified chip name MCXA276
Error: Found multiple chips matching 'MCXA276', unable to select a single chip. (MCXA276VLH, MCXA276VLL, MCXA276VLQ, MCXA276VPN)
error: process didn't exit successfully: `teleprobe local run --chip MCXA276 --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\crc` (exit code: 1)

When updating config.toml to use MCXA276VLQ, then teleprobe is unable to halt the core:

     Running `teleprobe local run --chip MCXA276VLQ --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\crc`
 INFO  teleprobe::run > run_from_ram: true
Error: An ARM specific error occurred.

Caused by:
    The core has to be halted for the operation, but was not.
error: process didn't exit successfully: `teleprobe local run --chip MCXA276VLQ --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\crc` (exit code: 1)

Are you using a special version of teleprobe?

[diondokter]

No I don't have a board yet, so haven't been able to run it

[felipebalbi]

Okay, I'll have a look, see if I can figure out which might be going on :)

[felipebalbi]

probe-rs needs to be updated on teleprobe. Here's a quick hack:

diff --git a/teleprobe/Cargo.toml b/teleprobe/Cargo.toml
index c4b65ad..6e3b919 100644
--- a/teleprobe/Cargo.toml
+++ b/teleprobe/Cargo.toml
@@ -12,8 +12,8 @@ defmt-decoder = { version = "1.0.0", features = ["unstable"] }
 jsonwebtoken = { version = "10.2.0", features = ["rust_crypto"] }
 log = "0.4.28"
 object = "0.37.3"
-probe-rs = { git = "https://github.com/akiles-dev/probe-rs", branch = "akiles-v0.30.x" }
-probe-rs-debug = { git = "https://github.com/akiles-dev/probe-rs", branch = "akiles-v0.30.x" }
+probe-rs = { git = "https://github.com/probe-rs/probe-rs.git" }
+probe-rs-debug = { git = "https://github.com/probe-rs/probe-rs.git" }
 reqwest = { version = "0.12.24", default-features = false, features = [
     "json",
     "native-tls",
diff --git a/teleprobe/src/probe/mod.rs b/teleprobe/src/probe/mod.rs
index 26e7059..2b10469 100644
--- a/teleprobe/src/probe/mod.rs
+++ b/teleprobe/src/probe/mod.rs
@@ -1,6 +1,6 @@
 use std::time::Instant;

-use anyhow::{anyhow, bail, Result};
+use anyhow::{Result, bail};
 use clap::Parser;
 use probe_rs::config::Registry;
 use probe_rs::probe::list::Lister;
@@ -160,7 +160,7 @@ pub fn connect(opts: &Opts) -> Result<Session> {
 fn open_probe(opts: &Opts) -> Result<Probe> {
     let lister = Lister::new();

-    match &opts.probe {
+    match opts.probe.clone() {
         None => {
             let probes = lister.list_all();
             if probes.is_empty() {
@@ -177,7 +177,7 @@ fn open_probe(opts: &Opts) -> Result<Probe> {
 }

 #[cfg(not(target_os = "linux"))]
-fn power_reset(probe_serial: &str, cycle_delay_seconds: f64) -> Result<()> {
+fn power_reset(_probe_serial: &str, _cycle_delay_seconds: f64) -> Result<()> {
     anyhow::bail!("USB power reset is only supported on linux")
 }

diff --git a/teleprobe/src/run.rs b/teleprobe/src/run.rs
index f9e7693..d79ce20 100644
--- a/teleprobe/src/run.rs
+++ b/teleprobe/src/run.rs
@@ -7,8 +7,8 @@ use std::time::{Duration, Instant};
 use anyhow::{anyhow, bail};
 use defmt_decoder::{DecodeError, Location, StreamDecoder, Table};
 use log::{info, warn};
-use object::read::{File as ElfFile, Object as _, ObjectSection as _};
 use object::ObjectSymbol;
+use object::read::{File as ElfFile, Object as _, ObjectSection as _};
 use probe_rs::config::MemoryRegion;
 use probe_rs::flashing::{DownloadOptions, ElfOptions, Format};
 use probe_rs::rtt::{Rtt, ScanRegion};
@@ -421,7 +421,7 @@ impl Runner {
         let initial_registers = DebugRegisters::from_core(core);
         let exception_handler = probe_rs_debug::exception_handler_for_core(core.core_type());
         let instruction_set = core.instruction_set().ok();
-        let stack_frames = di.unwind(core, initial_registers, exception_handler.as_ref(), instruction_set)?;
+        let stack_frames = di.unwind(core, initial_registers, exception_handler.as_ref(), instruction_set, 16)?;

         for (i, frame) in stack_frames.iter().enumerate() {
             let mut s = String::new();

Then tests pass (no need to change --chip argument on config.toml.

     Running `teleprobe local run --chip MCXA276 --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\crc`
 INFO  teleprobe::run > run_from_ram: true
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  teleprobe::run                        > flashing program...
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  teleprobe::run                        > flashing done!
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  device                                > 0.000035 CRC example
 INFO  device                                > 0.002463 Test OK

Now, the thing is that it seems Dario needs something from his version of probe-rs for Akiles, that needs to be sorted out with him, I guess :)

[felipebalbi]

ADC also works:

     Running `teleprobe local run --chip MCXA276 --protocol swd --elf target\thumbv8m.main-none-eabihf\debug\adc`
 INFO  teleprobe::run > run_from_ram: true
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  teleprobe::run                        > flashing program...
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  teleprobe::run                        > flashing done!
 WARN  probe_rs::vendor::nxp::sequences::mcx > unknown variant, using default watchpoint configuration
 INFO  device                                > 0.027104 Test OK