AP_CanManager: use milliseconds for timeout

[peterbarker]

The use of PRIu64 was breaking people's builds

[peterbarker]

@bugobliterator was there a reason this was using micros?

[bugobliterator]

The use of PRIu64 was breaking people's builds

This was added by @OXINARF . But yeah I don't see a reason for this.

[OXINARF]

I don't remember anymore why I used micros there. However, I think this isn't going to fix it since there are other PRI* macros in there. In my opinion, the fix should be to include the cinttypes header as my original code does.

[peterbarker]

I don't remember anymore why I used micros there. However, I think this isn't going to fix it since there are other PRI* macros in there. In my opinion, the fix should be to include the cinttypes header as my original code does.

Nuked the extra uses of PRI* - thanks.

I am tempted to just make the include as least-possible-fix, mind you.

This current PR will save 4 bytes per ESC....

Hmmm.

[OXINARF]

Yeah, this is included in the normal builds now (my original code was basically the entire firmware), so space should be a concern.

I still prefer the appropriate prints rather than the casts 😜

[tridge]

using PRIu64 is correct, but hard to read. Discussed on the call, decided to approve