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.

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

WIP: Refactor: Update to esp-idf v.5.0.2 #40

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from

Conversation

alufers
Copy link

@alufers alufers commented May 28, 2023

Hi!

As you might know the 4.x branch of esp-idf has entered maintenance mode, so only high severity bugs are fixed. It will stop being maintained in 2024, so I've decided to update this project to esp idf v5.0.2.

image

Currently the project compiles under v5.0.2, connects to Wi-Fi but I'm having trouble getting it to actually debug and flash the flipper using my Wifi devboard. I will need to debug it a bit further, but any input from you guys would be appreciated.

Task list:

  • Add components to the REQUIRES argument of idf_component_register
  • Add missing includes
  • Change constants and breaking API changes
  • Use Espressif's tiny usb
  • Add esp-idf component-manager (needed for MDNS and tinyusb)
  • Replace usage of tcpip_adapter_ip_info_t with esp_netif_ip_info_t
  • Change CI
  • Update docs in README.md
  • Test and debug

alufers added 15 commits May 28, 2023 02:10
Needed for idf v5.0.2. `esp_attr.h` is needed for IRAM_ATTR
Espressif decided to move the mdns functionality to https://github.com/espressif/esp-protocols,
which is an managed components, so their package manager has to be used.

managed_components has been added to .giitignore, because it is downloaded by esp-idf.
Renamed the component to avoid conflicts with the esp-idf component. The tinyusb
submodule has been removed and replaced, with Espressif's tinyusb managed component.

Adjusted CMakeLists.txt not to build tinyusb, leave it up to the managed component.

Added missing include to esp_mac.h
This has changed in idf v5.0.2, need to rewrite
@DrZlo13
Copy link
Member

DrZlo13 commented May 29, 2023

Wow, this is amazing work.

We always wanted to switch to esp-idf v5, but we did not have enough time. We will follow the progress, thanks!

@gautam-dev-maker
Copy link

This PR seems to work fine against IDF VERSION v5.1.1.
I was able to load and flash through gdb over the wifi on my stm32 board.
I am using ESP32S3. I think a sdkconfig.defaults file should be maintained so that this repository will be supported for all the espressif devices.

@augustozanellato
Copy link

augustozanellato commented Feb 1, 2024

Any plans on getting this PR rebased and merged?
I'd like to develop new features for the devboard firmware, specifically I was thinking about implementing and RPC<->TCP proxy using the new Expansion Module protocol, and I'd like to know which branch would be better to base my work on.
Thanks in advance :)

@skotopes
Copy link
Member

skotopes commented Feb 1, 2024

@augustozanellato yes we are planning to accept this PR. As soon as all issues with it will be fixed.

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.

5 participants