> Please keep init_full(); init() would be a version without some of the > arguments (e.g. the TimerListGroup, or the attributes).
Done. ... > > > > + scale: u32, > > While at it, can you add constants for the scale, i.e. > > pub const NS: u32 = bindings::SCALE_NS; > pub const US: u32 = bindings::SCALE_US; > pub const MS: u32 = bindings::SCALE_MS; > > ? Using Timer::NS is clear enough and removes the need to import from > "bindings". At least in theory, bindings should not even have to be "pub" > (or at least that's where the code should move towards). Great! I realized this case, too. Thanks, Zhao
