> > - rwdt_write(priv, 0, RWTCSRB); > > - rwdt_write(priv, priv->cks, RWTCSRA); > > Isn't this done implicitly with the above already ? > After all, priv->cks won't have RWTCSRA_TME set. Yes. The request came from a group doing some (safety?) audit who didn't like the implicit handling which might change if the code in probe() might change somewhen. And the datasheet explicitly says to disable the timer first before doing anything with the clock dividers. Given that, I can agree to this change, too.
Attachment:
signature.asc
Description: PGP signature
