Hi Mat,

 >
> Works for me. Do we have a consensus around removing the nanosecond
> versions and adding:
>
> struct l_timeout *l_timeout_create_ms(unsigned long milliseconds,
>                      l_timeout_notify_cb_t callback,
>                                  void *user_data,
>                      l_timeout_destroy_cb_t destroy);
> void l_timeout_modify_ms(struct l_timeout *timeout,
>                                  unsigned long milliseconds);
>

This looks fine to me.

Regards,
-Denis