Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add embot_hw_testpoint #383

Merged
merged 1 commit into from
May 26, 2023
Merged

Add embot_hw_testpoint #383

merged 1 commit into from
May 26, 2023

Conversation

sgiraz
Copy link
Contributor

@sgiraz sgiraz commented May 25, 2023

What's new:

  • A new embot::hw::testpoint is now available to be used for HW debugging on both CM7 and CM4 cores.

Usage:

// declare a TESTPOINT
embot::hw::TESTPOINT tp1 {embot::hw::TESTPOINT::one};

// init the testpoint
embot::hw::testpoint::init(tp1);

// trigger the testpoint using the available public methods (e.g. on, off, toggle)
embot::hw::testpoint::on(tp1);

// ... other code ...

// trigger the testpoint again
embot::hw::testpoint::off(tp1);

Note:

  • Tested on AMC board
  • With this tool, you can monitor the state of the pin connected to a test point on the oscilloscope while measuring the time/frequency at which the test point changes state.

cc @davidetome @Nicogene @mfussi66

@sgiraz sgiraz requested a review from marcoaccame May 25, 2023 10:38
@sgiraz sgiraz self-assigned this May 25, 2023
Copy link
Contributor

@marcoaccame marcoaccame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK @sigraz, the embot::hw device is implemented correctly:

  • one .h file w/ the API,
  • one _bsp.h file w/ the description of the data structure needed to adapt pinout and else to a given board,
  • one .cpp w/ the implemenation,
  • one ._bsp.cpp file specific to a given board.

@marcoaccame marcoaccame merged commit 8373f7a into robotology:devel May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants