SDK 17.1.0: is the "Need to close the ANT channel to make it safe to write to flash" warning still valid?

RE: SDK 17.1.0: is the "Need to close the ANT channel to make it safe to write to flash" warning still valid?

robca — Mon, 15 Nov 2021 18:27:03 GMT

Great, thanks. At the moment I haven't noticed any negative side effect. If you leave this ticket open for a couple of days, I will get back with confirmation after I can run more tests

RE: SDK 17.1.0: is the "Need to close the ANT channel to make it safe to write to flash" warning still valid?

Kenneth — Mon, 15 Nov 2021 17:21:10 GMT

[quote user="robca"]I would simply remove the sd_ant_channel_close() [/quote]

Please do that and let me know if there are any issues, if there are no issues I will suggest internally we can remove this for future as you suggest yes.

Kenneth

RE: SDK 17.1.0: is the "Need to close the ANT channel to make it safe to write to flash" warning still valid?

robca — Mon, 15 Nov 2021 17:08:21 GMT

Thanks for the super-fast reply. Just one clarification, please.

If there's no need anymore to stop the ANT channel on BLE disconnect, then I would simply remove the sd_ant_channel_close() call and ant_evt_handler(). In my application, it's actually required that the ANT channel be always active. Say, for example, that the user is monitoring the cycling power via a ANT+ cyclocomputer while changing settings using BLE (which is supported in our case). At he end of the BLE connection, there should be no interruption to the ANT+ packets.

Granted, a sd_ant_channel_close(0 followed immediately by ant_hrm_disp_open() (my equivalent in this case) will at worst result in one lost packet, hardly noticeable. But it's better to avoid unnecessary ANT channel disturbances

I would also suggest to remove that comment/code from future SDK examples. I would simplify the logic and make the example easier to understand

RE: SDK 17.1.0: is the "Need to close the ANT channel to make it safe to write to flash" warning still valid?

Kenneth — Mon, 15 Nov 2021 16:44:51 GMT

[quote user=""]First of all, is that warning still valid for SDK17.1.0? I found conflicting information searching here.[/quote]

I indeed believe so, I can find this comment all the way back to before we did support executing flash operations while in a connection. So please disregard this comment. This also make the other code you refer to make sense I think; it should advertise on disconnected.

Kenneth