Separate clockConfigRead and clockConfigWrite, Add GPIO_TCLKA_TCLKC_SEL

[bahadirdonmez]

No description provided.

[mastadle]

Can't you just print outputMapValue as %016X ?

[bahadirdonmez]

Yeah this was a bit annoying, I tried the followings but none of them worked...

printf("Current clock output map: 0x%016X\r\n", outputMapValue);
printf("Current clock output map: 0x%016lX\r\n", outputMapValue);
printf("Current clock output map: 0x%016llX\r\n", outputMapValue);
printf("Current clock output map: 0x%016" PRIx64 "\r\n", outputMapValue);

Could it be that the micro-controller cannot print long long unsigned integer (64-bit)?

[mastadle]

You would have to add printlli, similar to https://github.com/tiqi-group/openMMC/blob/feature/afck/modules/printf-stdarg.c#L127 (and adapt print accordingly)

But there is no real reason to use a custom printf implementation.
https://github.com/eyalroz/printf/ would be a better alternative.

At least write a comment why you did this.

[bahadirdonmez]

Alright, I now added some comments about this and added a TODO for using https://github.com/eyalroz/printf/

[mastadle]

Don't you want to include GPIO_TCLKA_TCLKC_SEL in the gpio set/read command in the cli commands?

[bahadirdonmez]

Don't you want to include GPIO_TCLKA_TCLKC_SEL in the gpio set/read command in the cli commands?

Good point, I will add it quickly

[bahadirdonmez]

Don't you want to include GPIO_TCLKA_TCLKC_SEL in the gpio set/read command in the cli commands?

I added and tested it quickly. It works.

[bahadirdonmez]

I now added gpio_direction_read and gpio_direction_write CLI functions. They are necessary to dynamically set the GPIO_TCLKA_TCLKC_SEL pin as input in order to disable the low-jitter clock bypass.