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

Added GSC functionality #56

Merged
merged 2 commits into from
Jul 6, 2024
Merged

Added GSC functionality #56

merged 2 commits into from
Jul 6, 2024

Conversation

Penguronik
Copy link
Contributor

Description

What was completed, changed, or updated?

Implemented GSC transmit and removed the ISR as the driver already handles that.. Also renamed lastFullMessageEndIndex() to bytesUntilLastMessageEnd().


Why was this done (if applicable)?


Testing

What manual tests were used to validate the code?


What unit tests were used to validate the code?


Documentation

Milestone number and name: Milestone 2

Link to Asana task: https://app.asana.com/0/1204699115748030/1206596856283249/f

Link to Confluence documentation:


Reminders

  • Add reviewers to the PR

  • Mention the PR in the appropriate discord channel

Implemented GSC transmit and removed the ISR as the driver already handles that
Also renamed lastFullMessageEndIndex() to bytesUntilLastMessageEnd()
Copy link
Contributor

@Yaremadzulynsky Yaremadzulynsky left a comment

Choose a reason for hiding this comment

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

Everything looks rly good. Great job! Just had a question about the internal buffer. Also, are you able to remove redundant/commented out code to make it as clean as possible before we merge into main?

Copy link
Contributor

@HardyYu HardyYu left a comment

Choose a reason for hiding this comment

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

Good stuff! I am mostly good with this pr and only left one comment there.

Drivers/common/drivers_config/inc/drivers_config.hpp Outdated Show resolved Hide resolved
@Yaremadzulynsky Yaremadzulynsky merged commit cfcbd14 into main Jul 6, 2024
2 checks passed
@Yaremadzulynsky Yaremadzulynsky deleted the feature/tm/GSC branch July 6, 2024 14:07
Yaremadzulynsky added a commit that referenced this pull request Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants