Skip to content

Split NTP and clock functions #1729

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
Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

Split NTP and clock functions #1729

wants to merge 46 commits into from

Conversation

MaxineMuster
Copy link
Contributor

To allow other sources than NTP for setting the clock, the general approach needs to be changing.

Here's my actual proposal.

MaxineMuster and others added 30 commits July 10, 2025 21:34
@MaxineMuster
Try NTP split update to actual source
1f646d2
@MaxineMuster
fix windows makefiles and timed_events
7e12b02
@MaxineMuster
fix typo
5f4dfff
@MaxineMuster
Fix sprintf format
0b6c917
@MaxineMuster
sprintf format - next try to fix for ESP and ESP8266
ccdcd2a
@MaxineMuster
sprintf - special case for ESP8266
021155e
@MaxineMuster
Adding GUI button to set clock
c488e60
@MaxineMuster
fix
dec2b85
@MaxineMuster
Add debug output
207d748
@MaxineMuster
fix missing HTTP output
c83f337 
use original obk_config.h - only renaming functions - to see filesizes
@MaxineMuster
limit functions to NTP ode CLOCK present
c6bdab2
@MaxineMuster
try fix selftests
3ad7ddc
@MaxineMuster
Merge branch 'openshwprojects:main' into Split_NTP_new
36cb4a1
@MaxineMuster
Enable simple clock for all platforms, even if NTP is not enabled
03fb760
@MaxineMuster
Merge branch 'openshwprojects:main' into Split_NTP_new
9aed9f6
@MaxineMuster
test fixing merge-conflict
8e20944
@MaxineMuster
Merge branch 'Split_NTP_new' of github.com:MaxineMuster/OpenBK7231T_A…
17c4b66 
…pp into Split_NTP_new
@MaxineMuster
try fixing merge conflicts #1
dc11774
@MaxineMuster
Merge branch 'main' into Split_NTP_new
49348f4
@MaxineMuster
try fix merge conflict
f1480da
@MaxineMuster
Resolve merge conflicts
c95ffdd
@MaxineMuster
Fix typos
4cade04 
consolidate time display on main page
fix double comments preventing several documentation details ($mday, $month ...) on docs/constants.md
enabling more time stuff for W800 for local testing.
@MaxineMuster
Fix setDST argument passing
fee56dc
@MaxineMuster
only test DST if clock is running
4e155ea
@MaxineMuster
Merge branch 'openshwprojects:main' into Split_NTP_new
46ebc00
@MaxineMuster
Fix setting epoch time - use atol(<string arg>) instead of Tokenizer_…
c5d50a9 
…GetArgInteger(0) for epoch

Change user_main.c to use xticks as uptime for g_secondsElapsed
@MaxineMuster
Merge branch 'Split_NTP_new' of github.com:MaxineMuster/OpenBK7231T_A…
a59092d 
…pp into Split_NTP_new
@MaxineMuster
Fix g_secondsElapesd for Windows - no xTicks there so stick to old wa…
f614f81 
…y with g_secondsElapesd++ here ...
@MaxineMuster
Fix simulator build
9e06c81 
remove additional defines for W800 used for testing
@MaxineMuster
Use esp_timer_get_time() instead of xTicks for ESP.
f4dbf2f 
Seems much better, 30 seconds difference in one day
(used to be over 10 minutes off in 12 hours)
MaxineMuster and others added 13 commits August 23, 2025 16:44
@MaxineMuster MaxineMuster mentioned this pull request Oct 3, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaxineMuster