[riot-notifications] [RIOT-OS/RIOT] stm32 eth peripheral driver (#10633)
notifications at github.com
Thu Jun 27 14:11:07 CEST 2019
kaspar030 requested changes on this pull request.
Looks good codewise!
There are two stm32_specific ```"eth_phy_*()```-functions, feel free to rename to ```stm32_eth_phy_*()``` to spare the next guy the name clashes.
At this point I think it is safe to drop the test applications, or make them generic, or guard them with ```FEATURES_REQUIRED += stm32_eth```. As you like. ;)
> @@ -0,0 +1,13 @@
+APPLICATION = stm32_eth_gnrc
isn't this application basically part of examples/default?
> @@ -0,0 +1,21 @@
+Tests for stm32 Ethernet periph driver
same here, can we reuse another lwip test application, or make this a generic "lwip_udp_echo" application that pulls in stm32_eth using ```FEATURES_OPTIONAL += stm32_eth```?
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications