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

Blog - Working on ESP devkits using WSL2 #149

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Blog - Working on ESP devkits using WSL2 #149

merged 1 commit into from
Nov 5, 2024

Conversation

jakub-kocka
Copy link
Contributor

@jakub-kocka jakub-kocka commented Oct 15, 2024 •
edited
Loading

Description

  • Introduces the topic of working with Espressif's devkits using the WSL
  • Simple guide on how to use the WSL with serial port forwarding

Related

  • Internal tracker IDF-4361

Testing

2024-10-15 15 42 36 localhost 8327fde99f8a

@jakub-kocka jakub-kocka marked this pull request as draft October 15, 2024 07:36
@jakub-kocka jakub-kocka force-pushed the blog/esp-devkits-and-wsl2 branch 2 times, most recently from abc1903 to 47d818e Compare October 15, 2024 07:46
@jakub-kocka jakub-kocka marked this pull request as ready for review October 15, 2024 07:56
@jakub-kocka jakub-kocka force-pushed the blog/esp-devkits-and-wsl2 branch 3 times, most recently from d045bba to a3348a2 Compare October 15, 2024 08:08
@mfialaf
Copy link
Contributor

mfialaf commented Oct 15, 2024

Things that may be worth mentioning:

  • In step 2, the usbipd can be seen only after opening new terminal
  • Step usbipd bind --busid 1-1 requires Administrator privileges

I've successfully managed to complete the tutorial. I can see, the new device using the susb command in the WSL terminal.
But when listing the ports ls /dev/tty*, there is no change. I assumed that I can use this article to communicate with the chip from wsl, but that require the port specification.

Am I missing something? Is this the point where I need to use the esp_rfc2217_server?

@jakub-kocka
Copy link
Contributor Author

Thank you, @mfialaf for the review.

I have improved the description of the steps you have pointed out. Also updated the guide with the mentioned issue as we have solved this in person. Thank you for the help!

@pedrominatel
Copy link
Member

Hi @jakub-kocka, thank you for the article.

Can you please rebase? This will fix some issues and the CI will pass. Also, update the article date to a recent one.

@jakub-kocka jakub-kocka force-pushed the blog/esp-devkits-and-wsl2 branch 4 times, most recently from dd60ff5 to c4aa3b2 Compare October 31, 2024 08:18
@jakub-kocka
Copy link
Contributor Author

Hi @pedrominatel, thanks for the review. However, rebase doesn't seem to help.

@pedrominatel
Copy link
Member

Don't worry, @jakub-kocka . I'll take a look.

@pedrominatel pedrominatel self-requested a review October 31, 2024 16:40
jakub-kocka
jakub-kocka commented Nov 1, 2024
View reviewed changes
Copy link
Contributor Author

@jakub-kocka jakub-kocka left a comment

Choose a reason for hiding this comment

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

Thank you @pedrominatel for the review and changes, it looks good to me.
I have left just a note about one sentence.

content/blog/espressif-devkits-with-wsl2/index.md Outdated Show resolved Hide resolved
@pedrominatel pedrominatel force-pushed the blog/esp-devkits-and-wsl2 branch 2 times, most recently from 1b042b6 to b97ad57 Compare November 4, 2024 15:05
@jakub-kocka @pedrominatel
Blog: Added article ESP devices and WSL2
ab03315 
Editorial changes and added the author json file

Changes on the introduction

Publication date updated
@f-hollow
Copy link
Collaborator

f-hollow commented Nov 5, 2024

Not sure what is wrong with the product selector link. It keeps appearing as invalid in this PR. However, as soon as I push these changes to a test PR, the checks always pass successfully and the status of checks here immediately switches to all passed.

This PR should fix such issues, hopefully, once and for all!

@pedrominatel pedrominatel merged commit 3510d38 into espressif:main Nov 5, 2024
5 of 6 checks passed
@dobairoland dobairoland mentioned this pull request Nov 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrominatel pedrominatel Awaiting requested review from pedrominatel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jakub-kocka @mfialaf @pedrominatel @f-hollow