[riot-notifications] [RIOT-OS/RIOT] boards/common/stm32l0: use dual bank with openocd if available (#11174)

danpetry notifications at github.com
Thu Mar 14 12:34:11 CET 2019


danpetry commented on this pull request.



> @@ -1,3 +1,8 @@
-source [find target/stm32l0.cfg]
+# Try to use dual bank if available and supported
+try {
+    source [find target/stm32l0_dual_bank.cfg]
+} on error {} {
+    source [find target/stm32l0.cfg]

+1 for not explicitly removing the single slot support. We're effectively preventing users from flashing smaller binaries for this board, if they have the openocd stable version. I would point out that this will also lead to questions about why it's suddenly stopped working, for certain users.

Suggest keeping `source [find target/stm32l0.cfg]` and giving them an info message, which also declares that support for this toolchain for this board is deprecated (i.e. we don't do support activities) and that they should upgrade if they want to use the dual bank functionality.

-- 
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/11174#discussion_r265526810
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190314/1aa74378/attachment.html>


More information about the notifications mailing list