[riot-notifications] [RIOT-OS/RIOT] drivers/si70xx: Add support for Si705x sensors (#15684)

iosabi notifications at github.com
Mon Jan 25 02:35:48 CET 2021


@iosabi commented on this pull request.



> @@ -1,6 +1,8 @@
 include ../Makefile.tests_common
 
-# This test should also work with Si7006, Si7013 and Si7020 variants.
-USEMODULE += si7021
+# This test should also work with si7006, si7013, si7020 and si705x variants.

I was going with all lowercase when referring to the RIOT module name, and upper case when referring to the chip itself. Here since this is the string you need to pass to the Makefile I'd prefer to leave it lowercase.

-- 
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/15684#discussion_r563420773
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210124/bdbd91b2/attachment.htm>


More information about the notifications mailing list