Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix verbose of check_clocks for 32 bit systems #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

koalo
Copy link
Contributor

@koalo koalo commented Feb 24, 2023

On 32 bit systems uint64_t is long long unsigned int and thus requires %llu in printf to prevent garbled output. Use PRIu64/PRId64 to handle this correctly for both 32 bit and 64 bit systems.

Furthermore, removed err that was never used.

@vcgomes
Copy link
Collaborator

vcgomes commented Feb 28, 2023

Sorry for the delay. Really minor optional suggestion, would you mind putting the "remove err" bit on a separate commit?

@koalo
Copy link
Contributor Author

koalo commented Mar 1, 2023

No Problem, I will do that shortly

On 32 bit systems uint64_t is long long unsigned int and thus
requires %llu in printf to prevent garbled output. Use
PRIu64/PRId64 to handle this correctly for both 32 bit and
64 bit systems.

Signed-off-by: Florian Kauer <florian.kauer@linutronix.de>
Signed-off-by: Florian Kauer <florian.kauer@linutronix.de>
@koalo koalo force-pushed the develop/fix-check_clocks-for-32-bit branch from 5f82120 to 945aa49 Compare March 9, 2023 08:18
@koalo
Copy link
Contributor Author

koalo commented Mar 9, 2023

I've splitted the commit now as requested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants