[riot-notifications] [RIOT-OS/RIOT] riscv_common: Refactor common fe310 code to riscv_common (#15718)

Koen Zandberg notifications at github.com
Thu Jan 7 17:13:26 CET 2021


@bergzand commented on this pull request.



> +
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/**
+ * @name    Power management configuration
+ * @{
+ */
+#define PROVIDES_PM_SET_LOWEST
+/** @} */
+
+/**
+ * @brief   Length of the CPU_ID in octets
+ */
+#define CPUID_LEN           (12U)

Should probably be moved to the fe310, there is no common CPU_ID register on the RISC-V architecture. For the fe310, the vendor ID, the architecture ID and the implementation ID are concatenated. Not perfect and probably identical between different boards, but I haven't found a better ID in the mean time.

I'll move this to the fe310 header

-- 
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/15718#discussion_r553427983
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210107/96ab3f90/attachment.htm>


More information about the notifications mailing list