[riot-notifications] [RIOT-OS/RIOT] Cifra: initial import of package (#12172)

benpicco notifications at github.com
Thu Sep 5 15:00:03 CEST 2019


benpicco commented on this pull request.

Tested on esp32, test code looks good.

I think we should make use of the optimized ARM assembly implementations in [`src/arm/unacl`](https://github.com/ctz/cifra/tree/master/src/arm/unacl) that are used by curve25519.

Might just need a conditional include of the assembly files and a

```C
#if defined(CPU_ARCH_CORTEX_M0) || defined(CPU_ARCH_CORTEX_M0PLUS)
#define CORTEX_M0
#endif 
#if defined(CPU_ARCH_CORTEX_M3)
#define CORTEX_M3
#endif
#if defined(CPU_ARCH_CORTEX_M4)
#define CORTEX_M4
#endif
```

If those are not set, it falls back to tweetnacl.



-- 
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/12172#pullrequestreview-284235515
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190905/4000133d/attachment.htm>


More information about the notifications mailing list