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

Remove esp8266 support #31

Merged
merged 2 commits into from
Feb 6, 2024
Merged

Remove esp8266 support #31

merged 2 commits into from
Feb 6, 2024

Conversation

MabezDev
Copy link
Member

@MabezDev MabezDev commented Feb 6, 2024

Draft until I verify everything works on real hardware.

Closes esp-rs/xtensa-lx-rt#62

@MabezDev
Copy link
Member Author

MabezDev commented Feb 6, 2024

Tested on hardware, esp-hal examples with interrupts work, and esp-wifi examples still work.

@MabezDev MabezDev marked this pull request as ready for review February 6, 2024 12:20
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 - what a relief, finally being able to throw that one remaining 8266 into the bin 😄
I think there is a file exception-esp8266.x left which can be removed and we shouldn't forget to archive the original xtensa-lx-rt repo. Thanks for doing this!

@bjoernQ
Copy link
Contributor

bjoernQ commented Feb 6, 2024

Also tested with esp-wifi ... I guess if that works it's all fine

@MabezDev MabezDev merged commit fd882de into master Feb 6, 2024
@MabezDev MabezDev deleted the remove-esp8266 branch February 6, 2024 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

merge repo into xtensa-lx repo
2 participants