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.

Sign up for GitHub

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

Jump to bottom

transport: fix zephyr uart transport. See #409 #410

Merged
merged 1 commit into from
Mar 4, 2024
Merged

transport: fix zephyr uart transport. See #409 #410

merged 1 commit into from
Mar 4, 2024

Conversation

gabbla
Copy link
Contributor

@gabbla gabbla commented Feb 22, 2024

Pull request

Choose Correct

  • bug
  • feature

Describe the pull request

This pull request address the issue described in #409.

Steps you didn't forgot to do

  • I checked if other PR isn't solving this issue.
  • I read Contribution details and did appropriate actions.
  • PR code is tested.
  • PR code is formatted.
  • Allow edits from maintainers pull request option is set (recommended).

Hadatko
Hadatko requested changes Feb 22, 2024
View reviewed changes
Copy link
Member

@Hadatko Hadatko left a comment

Choose a reason for hiding this comment

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

We need new implementation likely instead of this change which is removing buffer size control.

@JanKomarekNXP JanKomarekNXP merged commit 2890823 into EmbeddedRPC:develop Mar 4, 2024
5 of 8 checks passed
@gabbla gabbla deleted the fix_zephyr_uart_transport branch April 11, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hadatko Hadatko Hadatko requested changes

@JanKomarekNXP JanKomarekNXP JanKomarekNXP approved these changes

Assignees

@JanKomarekNXP JanKomarekNXP

Labels
bug
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gabbla @Hadatko @JanKomarekNXP @MichalPrincNXP