[riot-notifications] [RIOT-OS/RIOT] nrf52: Implement EasyDMA-based SPI peripheral implemenation (#14057)

benpicco notifications at github.com
Tue May 12 09:51:45 CEST 2020


@benpicco commented on this pull request.



> @@ -180,6 +186,34 @@ typedef struct {
 } uart_conf_t;
 #endif
 
+/**
+ * @brief  SPI configuration values
+ */
+typedef struct {
+    NRF_SPIM_Type *dev; /**< SPI device used */
+    gpio_t sclk;        /**< CLK pin */
+    gpio_t mosi;        /**< MOSI pin */
+    gpio_t miso;        /**< MISO pin */
+    uint8_t ppi;        /**< PPI channel */

I'd say so - but I'd suggest to ifdef for the errata, not the CPU family so when adding new members of the family there only has to be a changed in one place.

-- 
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/14057#discussion_r423532132
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200512/f54f28f0/attachment-0001.htm>


More information about the notifications mailing list