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

cpu/esp32: fixes for gcc 8.4.0 #18104

Merged
merged 2 commits into from
May 13, 2022

Conversation

gschorcht
Copy link
Contributor

Contribution description

This PR provides two fixes that are required to make the ESP32 code compilable with gcc 5.2.0 and gcc 8.2.0.

Testing procedure

Green CI

Issues/PRs references

Fixes the compilation problems occured in CI with RIOT-OS/riotdocker#190.

@github-actions github-actions bot added Area: cpu Area: CPU/MCU ports Platform: ESP Platform: This PR/issue effects ESP-based platforms labels May 13, 2022
@gschorcht gschorcht added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: full build disable CI build filter and removed Area: cpu Area: CPU/MCU ports labels May 13, 2022
@benpicco benpicco added Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels May 13, 2022
@maribu maribu merged commit ae0bd73 into RIOT-OS:master May 13, 2022
@chrysn chrysn added this to the Release 2022.07 milestone Aug 25, 2022
@gschorcht gschorcht deleted the cpu/esp32/fix_for_gcc_8.4.0 branch April 12, 2023 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: full build disable CI build filter CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ESP Platform: This PR/issue effects ESP-based platforms Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants