Skip to content

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

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
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
@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