Add a delay after rtc_set_datetime

[peterharperuk]

If you call rtc_get_datetime immediately after rtc_set_datetime you get junk back. According to the datasheet "Writing to the RTC will take 2 clk_rtc clock periods to arrive". So add a delay after calling rtc_set_datetime in aon_timer_set_time_calendar.

Fixes #2148

[lurch]

ping @armandomontanez - any idea why the CMake build is passing but the Bazel build is failing?

[armandomontanez]

Looks like you need to add //src/rp2_common/pico_aon_timer to the deps of the test. You also might need to add //src/common/pico_time to the deps of pico_aon_timer since it's pulling in a new header. If you hit another snag let me know and I'll cook up a diff to get it working.

[kilograham]

@peterharperuk can you fix merge conflicts and i'll re-review

[peterharperuk]

Apologies. Merge conflicts fixed. And I squashed everything together.

[kilograham]

Ah, not sure if this is fixed in different issue but you have added #include "pico/aon_timer.h" to pico_time_test.c which will break host build; it (and test which use it) should be guarded by #if LIB_PICO_AON_TIMER probably

pico_aon_timer now depends on hardware_clocks AND pico_time.. probably should be explicit about dependencies (note just pico_time_headers is likely fine. Note in the CMakeLists.txt I'd also make the dependency (and the includes) RP2040 only

[peterharperuk]

I've tried to make the changes you suggested. You'll have to tell me how to test aon timer in a host build.

[kilograham]

cmake -DPICO_PLATFORM=host .. then build everything