How to use OTA without HTTP server or Web Application

Updated https://github.com/openshwprojects/OpenBK7231T_App/pull/1471 with this idea:
Moved defines for file extension to obk_config.h as #define OBK_OTA_EXTENSION "<file extension>" for all platfoms with OTA
Replaced "#if NO_PLATFORM_OTA" with "#ifndef OBK_OTA_EXTENSION"

The changes seem acceptable. W800 OTA is a good addition. Is it ready to merge?

p.kaczmarek2 wrote:

Is it ready to merge?

It is, but while reading it again, I might squeeze some more bytes, let me check again if it's possible. Thanks

Added after 9 [hours] 33 [minutes]:

Did some small changes, from my side it should be ready to merge now.

Good job, I'm very happy with your recent contributions! Merged.

I'll be testing some W800/W600 soon, so this ota may come handy.

great job @max4elektroda


lmk if you want to see others in action

Good to hear it's kind of useful
maybe @divadiow I can ask for your support again to do some testing on my new try to separate NTP and clock as a requirement to use other clock sources.
Its https://github.com/openshwprojects/OpenBK7231T_App/pull/1483

Since it involves changes to many files, it's getting merge conflicts very fast meaning I have to do the PR again ans again...

This PR should only relocate and rename functions and only do one simple change to the system: The "clock" is present even if it's not synced or ntp driver not started.
This allows to drop the special handling, e.g. in json_interface.c implementing a "fake" NTP returning 0 in case NTP is not present.


This implementation (e.g. "Clock_GetCurrentTime()") will always report time 0 as time in this case, so we can drop this code.

I also fixed code which I believe was in wrong "#ifdef"s in cmd_if.c: there was a mix up of bl0937 and ntp code, no problem if both is defined but not working, if only ntp is active.

Prominent example: $yesterday and $today - they sound like ntp functions and where in NTP section, but they will return the energy consumption for yesterday/today, so they belong to the BL0397 section...

So if you could do some tests if all time related functions still work as expected?
Especially the advanced features like sunrise/sunset and events?
Thanks to @p.kaczmarek2 we have some basic tests already done be selftests, so it's not totally wrong

ooh ok. Sure. I'll see what I can do, it may not be for a couple of days now though, as I fear tomorrow will be a write-off after this eve

No problem, first things first!
Happy new year!