> > + if (rate < 32000 || rate > BIT(22)) {
>
> Perhaps
>
> #define RTCA0SCMP_MIN 32000
> #define RTCA0SCMP_MASK GEN_MASK(21. 0)
>
> and
>
> if (rate < RTCA0SCMP_MIN || rate > FIELD_MAX(RTCA0SCMP_MASK) + 1)
>
> ?
You really think this is more readable than the original code? I am
really tired of bike-shedding so I don't care much, but I do wonder...
Attachment:
signature.asc
Description: PGP signature
