[riot-notifications] [RIOT-OS/RIOT] boards/common/nrf5x: add configurable RTT_FREQUENCY (#14025)

Alexandre Abadie notifications at github.com
Tue May 5 16:51:57 CEST 2020


@aabadie commented on this pull request.



> @@ -29,11 +29,21 @@ extern "C" {
  * @name    Real time counter configuration
  * @{
  */
-#define RTT_DEV             (1)             /* NRF_RTC1 */
-#define RTT_MAX_VALUE       (0x00ffffff)
-#define RTT_FREQUENCY       (1024)
+#ifndef RTT_DEV
+#define RTT_DEV             (1)                 /* NRF_RTC1 */
+#endif
+
+#define RTT_MAX_VALUE       (0x00ffffff)         /* 24bit */
+#define RTT_MAX_FREQUENCY   (32768U)             /* in Hz */

RTT_MAX_FREQUENCY and RTT_MIN_FREQUENCY constants are not used by the nrf driver. So I either remove them or add some pre-processor logic that checks RTT_FREQUENCY is within bounds.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/14025#pullrequestreview-405867328
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200505/b6e57796/attachment.htm>


More information about the notifications mailing list