8000 Use F_CPU if (?) CPU frequency switch is compile-time only by dok-net · Pull Request #6833 · esp8266/Arduino · GitHub
[go: up one dir, main page]
Skip to content

Use F_CPU if (?) CPU frequency switch is compile-time only #6833

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

Merged
merged 10 commits into from
Apr 15, 2020
Merged

Use F_CPU if (?) CPU frequency switch is compile-time only #6833

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
4062650
At least the F_CPU define in host mock.h is needed by host Arduino.h …
dok-net Nov 26, 2019
eba0042
Geting the include order right
dok-net Nov 27, 2019
26f6ac8
According to leading core developer, runtime CPU frequency switch is …
dok-net Nov 24, 2019
689187f
Prepare for runtime CPU clock rate selection
dok-net Nov 25, 2019
426884e
Fix compile for not defined F_CPU
dok-net Nov 26, 2019
9968a62
Use defines for register CPU2X instead of hex value
dok-net Nov 26, 2019
08927ba
Fix build for host - getCpuFreqMHz there was also in conflict with ge…
dok-net Nov 26, 2019
7210ad3
Asymmetrical includes and defines on host
dok-net Nov 26, 2019
f719bb2
Support restart switch from 160MHz to 80MHz, e.g for OTA. Fixes #579
dok-net Apr 11, 2020
39d5ef9
Merge branch 'master' into fastgetcpufreq
devyte Apr 15, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use defines for register CPU2X instead of hex value 
Prepare PolledTimeout for dynamic CPU frequency - would fail to compile if F_CPU is not defined, helps as marker that something needs to be done.
  • Loading branch information
@dok-net
dok-net committed Apr 11, 2020
commit 9968a629f88b1beef0d980c73d26b3eee6e16156
2 changes: 1 addition & 1 deletion cores/esp8266/PolledTimeout.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ struct TimeSourceCycles

using timeType = decltype(ESP.getCycleCount());
static timeType time() {return ESP.getCycleCount();}
static constexpr timeType ticksPerSecond = F_CPU; // 80'000'000 or 160'000'000 Hz
static constexpr timeType ticksPerSecond = ESP.getCpuFreqMHz() * 1000000UL; // 80'000'000 or 160'000'000 Hz
static constexpr timeType ticksPerSecondMax = 160000000; // 160MHz
};

Expand Down
6 changes: 4 additions & 2 deletions cores/esp8266/core_esp8266_main.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,11 @@ void initVariant() {
extern "C" void __preloop_update_frequency() {
#if defined(F_CPU) && (F_CPU == 160000000L)
ets_update_cpu_frequency(160);
REG_SET_BIT(0x3ff00014, BIT(0));
CPU2X |= 1;
#elif !defined(F_CPU)
if (system_get_cpu_freq() == 160) REG_SET_BIT(0x3ff00014, BIT(0));
if (system_get_cpu_freq() == 160) {
CPU2X |= 1;
}
#endif
}

Expand Down
0