On Thu, 1 Dec 2022 at 15:42, ~axelheider <axelheider@git.sr.ht> wrote:
>
> This patch set improves the i.MX EPIT emulation:
> - fix #1263 for writes to CR
> - ensure SR_OCIF is set properly even if CR_OCIEN is off
> - hardware reset initialized CR to 0
> - interrupt state update on CR writes (e.g. software reset)
> - remove explicit fields cnt and freq (they are redundant)
> - general code and documentation improvements
>
> v3 addresses the comments from the previous iterations, but still keeps
> the scope of this patchset limited to addressing the obvious bugs in the
> behavior. It does not try to improve the timer accuracy, thus the
> following remarks remain to be fixed in a future patch after this one is
> merged:
> - don't use PTIMER_POLICY_LEGACY. Fine tuning this requires more
> time and currently this is not a major concern, because the timer is
> working reasonably well.
> - replace the modestly harmful sequence
> counter = ptimer_get_count(s->timer_reload);
> ...
> ptimer_set_count(s->timer_cmp, counter);
> by something better that does not lose or gain time. The current
> patchset does not introduce this sequence, it has been there
> before already. Again, the current lack of accuracy here is not a
> major concern because the timer is working reasonably well.
Applied to target-arm.next, thanks. Sorry it took me so long to
get to this.
-- PMM