infineon: Psc3m5 counter

[billwatersiii]

Add psc3m5 counter driver:

• Add a new counter driver implementation based on the PDL for
  Infineon CAT1B devices. This enables support for hardware
  counters on the PSC3M5 platform.
• Add IFX_TCPWM_Counter_DeInit and IFX_TCPWM_Counter_Init
  macros to include/zephyr/drivers/timer/ifx_tcpwm.h
  and sort all of the macros in that file
• Enable the alarm sample for the kit_psc3m5_evk

[teburd]

This should really be added to the counter API test.

Would be great to see some test coverage on the neat tricks being used for counter_is_bit_mask, counter_ticks_add, counter_ticks_sub functions which look well done but took a moment to read and understand. Some tests would make it easier to ensure the edge scenarios are covered and stay covered over time.

I'm unsure the likely() to shortcut masking bits when the top value is a 2^n-1 is worth while, but you likely know better than me, and its a neat how its done.

I'm not super familiar with tcpwm IP but if it can do counter capture (capture the count on a gpio trigger) there's a PR from @XenuIsWatching in #89127 that might be worth watching and implementing.

addressed

[billwatersiii]

This should really be added to the counter API test.

Would be great to see some test coverage on the neat tricks being used for counter_is_bit_mask, counter_ticks_add, counter_ticks_sub functions which look well done but took a moment to read and understand. Some tests would make it easier to ensure the edge scenarios are covered and stay covered over time.

I'm unsure the likely() to shortcut masking bits when the top value is a 2^n-1 is worth while, but you likely know better than me, and its a neat how its done.

I'm not super familiar with tcpwm IP but if it can do counter capture (capture the count on a gpio trigger) there's a PR from @XenuIsWatching in #89127 that might be worth watching and implementing.

I added the counter API test. We will address you other suggestions with future pull requests.

[JarmouniA]

@billwatersiii Please leave comments resolving to reviewers.

[nashif]

platform related change, assign to platform maintainer.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud