Skip to content
This repository has been archived by the owner on Jul 15, 2019. It is now read-only.

Serial port implementation #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

timonmentink
Copy link
Contributor

Added serial port implementation.
Added a lot of doxygen comments.
Currently Working:

  • Search cpu`s
  • Read registers
  • Write registers
  • Add debug channels
  • Read debug channels

Moved some code to Medium, so we dont get duplicate code in different connectors.
Added a lot of doxygen documentation.
Currently Working:
- Search cpu`s
- Read registers
- Write registers
- Add debug channels
- Read debug channels
@timonmentink timonmentink added the enhancement New feature or request label Jan 31, 2019
@timonmentink timonmentink requested a review from beralt January 31, 2019 12:49
@timonmentink
Copy link
Contributor Author

@beralt could you review this pull request?

Copy link

@beralt beralt left a comment

Choose a reason for hiding this comment

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

lgtm, but the shear amount of changes to implement the serial port transport alone is an issue. Please make your pull request smaller and split them into several small ones.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants