Skip to content

Conversation

@jessebraham
Copy link
Member

Another small PR, these traits were never implemented for this peripheral (and they're essentially the same as the previous traits, so easy enough to add).

Also fixed some warnings I accidentally introduced in a previous PR.

@jessebraham jessebraham changed the title Implement embedded-hal-nb::serial::* traits for UsbSerialJtag Implement embedded_hal_nb::serial::* traits for UsbSerialJtag May 1, 2023
Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

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

LGTM

@MabezDev MabezDev merged commit 6fdc1ea into esp-rs:main May 2, 2023
@jessebraham jessebraham deleted the fixes/usb-serial-jtag branch May 2, 2023 12:41
i404788 pushed a commit to i404788/esp-hal that referenced this pull request Jul 22, 2023
…p-rs#498)

* Implement `embedded-hal-nb` serial traits for `UsbSerialJtag`

* Fix warnings in `usb_serial_jtag` examples
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