event_periodic_callback: add event_periodic_callback_create()

[benpicco]

Contribution description

Creating a periodic event should be no more difficult than creating a thread, so add a event_periodic_callback_create() convenience functions that reduces creating and starting a periodic event to a single function.

Testing procedure

Read the API to see if this is really a worthwhile improvement.

Issues/PRs references

After seeing many threads being created for simple periodic measurements, maybe this could be a remedy.

[Teufelchen1]

After seeing many threads...

Do you have an example? Can you refactor some code?
Can you also fix the typo in line 57 "..to to..."?

Reads fine, lgtm.

[benpicco]

Do you have an example? Can you refactor some code?

I unfortunately can't since that's code by a customer, but I also want to spare your eyes.
But they basically do multiple instances of

static void *_adc_thread(void *ctx)
{
    (void)ctx;

    while (1) {
        do_measure();
        send_data();
        ztimer_sleep():
    }
    return NULL;
}

If I can get them to instead do

static void _adc_event(void *ctx)
{
    do_measure();
    send_data();
}

that would be a win for everyone.

[Teufelchen1]

Ah, forgot to ask for squashing 😅

[benpicco]

eh but those are two distinct commits

[Teufelchen1]

well....if you insist 😀

[riot-ci]

Murdock results

✔️ PASSED

2bef44b event_periodic_callback: fix typo

Success	Failures	Total	Runtime
10008	0	10009	07m:51s

Artifacts

[kfessel]

very convenient