[riot-notifications] [RIOT-OS/RIOT] black/white e-Paper/e-Ink display driver (#12509)

Alexandre Abadie notifications at github.com
Wed May 19 17:44:55 CEST 2021


@aabadie commented on this pull request.



> @@ -0,0 +1,10 @@
+BOARD ?= nucleo-f411re
+
+include ../Makefile.tests_common
+
+USEMODULE += xtimer
+USEMODULE += epd_bw_spi
+
+INCLUDES += -I$(APPDIR)

You could use CFLAGS from the command line. But I would avoid any custom setup specific configuration in a RIOT application, this is not generic and cannot be reused in another setup.
If you set the driver default params with your preferred values (let's say it's a privilege given to the initial contributor), then you won't have to change anything. But if someone wants to reuse this application for its needs with a different setup, he can add CFLAGS to the build command line without having to change any line of code. If it's a custom setup hardwired on a board (like the stm32l0 disco), then the defines will have to defined in board.h of this board (board.h must be included in `xx_params.h` header and the application can be built as is.

-- 
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/12509#discussion_r635367869
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210519/c64d0340/attachment.htm>


More information about the notifications mailing list