Add two Nucleo boards, GDB debugging and SVD files for all boards #38

maxgerhardt · 2021-03-16T21:46:30Z

This superseedes my PR #37.

See explanations in PR#37 why certain things were done to support and flash the new boards.

To get debugging working, I fixed a few things, restructured a bit and added a bit of code:

OpenOCD information for target .cfg file is placed in the board manifest as openocd_target as with other platforms (e.g. stm32)
Building in built_type = debug mode was broken because SDCC was given the default debug_build_flags with -Og -g2 etc., which it does not understand. Thanks to research in Add debug support using STLink(/V2) via OpenOCD #31, the right flags were found, --debug --out-fmt-elf
However, this broke the .ihx build output file in debug mode because when converting it with SDCC to hex it also dumps the debug information in there, at address 0x0000, which crashes the uploader program and makes no sense at all. The ELF file is fine however, so I used stm8-objcopy to convert the ELF to IHX (while removing the wrong sections).
The SVD files come from @gicking at https://github.com/gicking/STM8_headers. The SVD files are flawless and pass all checks by the SVDConv tool in the CMSIS toolset. All peripheral info can thus be displayed
PR was tested against the STM8S-Discovery board, the Nucleo-8S208RB board and a STM8S103F3, with the ST-Link V2.1 and V2 debug probes respectively
A good bare-metal test project for debugging: https://github.com/maxgerhardt/stm8-headers-pio

This brings debugging from nothing for most boards to full debugging support for all boards with peripheral view thanks to SVD files!

Closes #33, #34 and #31.

Also corrects the logic that derives the value for -p switch in the stm8flash invocation (would do 'stm8s207?8' and not 'stm8s207k8')

Both the boards have the user-LED connected to PC5 per board manual. E.g. https://www.st.com/resource/en/user_manual/dm00489875-stm8-nucleo32-board-mb1442-stmicroelectronics.pdf

valeros · 2021-03-16T22:06:01Z

Many thanks, looks promising! Please see my comments in review.

boards/nucleo_8s207k8.json

builder/main.py

platform.py

maxgerhardt · 2021-03-17T00:37:19Z

I'll address the comments tomorrow and also fix a still wrongly built .ihx file, since I just got a report saying that in some cases it still includes bytes at address 0x0001 which don't belong there -- more infos pending hopefully :)

Switching from SDCC to stm8-objcopy to Intel HEX file generation has the advantage of just copying the relevant sections (and ignoring the debug sections that sdcc would otherwise include), but also care must be taken that all sections which do not end up in FLASH are removed. Two more sections are removed during copying, which are created when initialized and uninitialized global variables are used within a project.

maxgerhardt · 2021-03-21T20:26:19Z

I've addressed the review comments and also fixed a bug in the Intel Hex file generation logic which previously prevented an upload as soon as a global variable was being created.

platform.json

platform.py

valeros · 2021-03-22T11:09:55Z

Thanks for the fixes! BTW, that new hex file generation logic works fine with the Arduino framework, right?

maxgerhardt · 2021-03-22T13:24:04Z

that new hex file generation logic works fine with the Arduino framework, right?

I can confirm that it does in normal release build mode. I've forked an Arduino project that plays the Tetris theme on a buzzer at https://github.com/maxgerhardt/TetrisThemeArduino, and it works with the my new platform perfectly. I've tested it on my STM8S103F3 board (stm8sblue).

However, debugging the Arduino framework code is currently not possible because of one freaking file failing compilation in --debug mode: WInterrupts.c. I've filed a bug at https://sourceforge.net/p/sdcc/bugs/3199/. Would be nice to get that resolved before release.

This will be re-downloaded before a debugging session starts and we do not need to worry.

valeros · 2021-03-22T15:22:30Z

Many thanks for such a great contribution! I'm working on updates for SDCC and Sduino in a separate branch, so it will be a great release!

maxgerhardt · 2021-03-22T16:20:45Z

Awesome! Might also consider merging PR #28 for a STM8S003 chip and add the few extra items for debug support there? :)

valeros · 2021-03-22T16:29:02Z

I've just merged my branch into dev and also added minor changes to your previous code, would be great if you could take a look. Unfortunately, I don't have any ST8-based board at hand so it would be really great if you could test the latest revision (compiling, uploading, debugging).

maxgerhardt · 2021-03-22T16:41:25Z

I've built my stm8-headers-pio project from the PR decsription with the latest dev platform; it updated some packages, then compiled and uploaded just fine. Also works (blinks LED).

Processing stm8sblue_stm8s103f3 (platform: https://github.com/platformio/platform-ststm8.git; board: stm8sblue)
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------Tool Manager: Installing platformio/tool-stm8gal @ ~1.5.0
Tool Manager: tool-stm8gal @ 1.5.0 has been installed!
Tool Manager: Installing platformio/tool-stm8flash @ ~1.1.0
Tool Manager: tool-stm8flash @ 1.1.20170616 has been installed!
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/ststm8/stm8sblue.html
PLATFORM: ST STM8 (1.0.3+sha.dbba972) > ST STM8S103F3 Breakout Board
HARDWARE: STM8S103F3P6 16MHz, 1KB RAM, 8KB Flash
DEBUG: Current (stlink) External (stlink)
PACKAGES:
 - tool-openocd 2.1000.200630 (10.0)
 - tool-stm8binutils 0.230.0 (2.30)
 - tool-stm8flash 1.1.20170616
 - tool-stm8gal 1.5.0
 - toolchain-sdcc 1.40100.12072 (4.1.0)
LDF: Library Dependency Finder -> http://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 1 compatible libraries
Scanning dependencies...
Dependency Graph
|-- <STM8_headers> 0.0.1
Building in release mode
Compiling .pio\build\stm8sblue_stm8s103f3\src\main.rel
Archiving .pio\build\stm8sblue_stm8s103f3\libc36\libSTM8_headers.lib
Indexing .pio\build\stm8sblue_stm8s103f3\libc36\libSTM8_headers.lib
src\main.c:73:4: warning: #warning "[INFO] USING STM8S103F3 PINMAP PORTB, PIN 5"
Linking .pio\build\stm8sblue_stm8s103f3\firmware.elf
Checking size .pio\build\stm8sblue_stm8s103f3\firmware.elf
Advanced Memory Usage is available via "PlatformIO Home > Project Inspect"
RAM:   [          ]   0.5% (used 5 bytes from 1024 bytes)
Flash: [          ]   1.4% (used 115 bytes from 8192 bytes)
Building .pio\build\stm8sblue_stm8s103f3\firmware.hex
Configuring upload protocol...
AVAILABLE: serial, stlinkv2
CURRENT: upload_protocol = stlinkv2
Uploading .pio\build\stm8sblue_stm8s103f3\firmware.hex
Determine FLASH area

libusb: error [init_device] device '\\.\USB#VID_062A&PID_5918&MI_01#9&1953ACC1&0&0001' is no longer connected!
Due to its file extension (or lack thereof), ".pio\build\stm8sblue_stm8s103f3\firmware.hex" is considered as INTEL HEX format!
115 bytes at 0x8000... OK
Bytes written: 115
=============== [SUCCESS] Took 4.84 seconds ===============

However, debugging broke. There doesn't seem to be any debug information in the ELF file anymore, also it doesn't halt at the initial breakpoint anymore (main) because it can't figure out the address of it.

~~Using Verbose Build I can see that the final sdcc invocation that creates the ELF file doesn't have the --debug flag anymore.~~ My platform also didn't do that, so the cause might be somewhere else. I'll compare flags.

sdcc -o .pio\build\stm8sblue_stm8s103f3\src\main.rel -c -mstm8 --debug -DF_CPU=16000000L -DPLATFORMIO=50101 -DSTM8S_BLUE -DSTM8S103 -D__PLATFORMIO_BUILD_DEBUG__ -Iinclude -Isrc -Ilib\STM8_headers -Ilib\STM8_headers\include src\main.c
sdar rcs .pio\build\stm8sblue_stm8s103f3\libc36\libSTM8_headers.lib
sdranlib .pio\build\stm8sblue_stm8s103f3\libc36\libSTM8_headers.lib
src\main.c:73:4: warning: #warning "[INFO] USING STM8S103F3 PINMAP PORTB, PIN 5"
sdcc -o .pio\build\stm8sblue_stm8s103f3\firmware.elf -mstm8 --nostdlib --code-size 8192 --iram-size 1024 --out-fmt-elf .pio\build\stm8sblue_stm8s103f3\src\main.rel -LC:\Users\Max\.platformio\packages\toolchain-sdcc\lib\stm8 .pio\build\stm8sblue_stm8s103f3\libc36\libSTM8_headers.lib -lstm8

maxgerhardt · 2021-03-22T16:47:17Z

Ah, I think I figured out the cause. The --out-fmt-elf flag is missing in this platform code. This must be there for DWARF formatted debug info per #31.

My platform:

sdcc -o .pio\build\stm8sblue_stm8s103f3\src\main.rel -c -mstm8 --debug --out-fmt-elf -DF_CPU=16000000L -DPLATFORMIO=50101 -DSTM8S_BLUE -DSTM8S103 -D__PLATFORMIO_BUILD_DEBUG__ -Iinclude -Isrc -Ilib\STM8_headers -Ilib\STM8_headers\include src\main.c

Yours:

sdcc -o .pio\build\stm8sblue_stm8s103f3\src\main.rel -c -mstm8 --debug -DF_CPU=16000000L -DPLATFORMIO=50101 -DSTM8S_BLUE -DSTM8S103 -D__PLATFORMIO_BUILD_DEBUG__ -Iinclude -Isrc -Ilib\STM8_headers -Ilib\STM8_headers\include src\main.cx

You removed it in e16be52

If I add build_flags = --out-fmt-elf again with your platform it works again :)

valeros · 2021-03-22T17:52:08Z

That's interesting. IIRC, the --out-fmt-elf flag only makes sense for the linker:

--out-fmt-ihx - The linker output (final object code) is in Intel Hex format. This is the default option. The format
itself is documented in the documentation of srecord.

But if it's helps to produce a proper binary for debugging, let's then add it to CFLAGS as well.
Please retest with the latest revision.

maxgerhardt · 2021-03-22T17:53:35Z

Seems to be not well documented, but per https://sourceforge.net/p/sdcc/discussion/1864/thread/aa45ea7ea2/#fad2/429a

gdb expects debug info in DWARF format. AFAIK, SDCC generates CDB on --debug without --out-fmt-elf, DWARF on --debug --out-fmt-elf, so you'll need to add --out-fmt-elf to the compilation command line.

The comment below says

Since the SDCC 4.1.0 release is very close, it is too late for that. But I've just documented it in the manual in the sdcc-next branch, which will be merged after the release.

maxgerhardt · 2021-03-22T18:17:00Z

With the latest commit, debugging functionality is restored again with no extra flags needed.

But, the updated Arduino core does not run anymore on my STM8S103F3 board. It simply outputs nothing (refer Tetris example above), like it doesn't even reach setup() or loop().

If I roll back just the Arduino core version but still use the latest platform version, it works again

platform_packages =
        framework-arduinoststm8@~0.40.0

Compile output from 0.40.0 Arduino core:

RAM:   [=         ]   7.2% (used 74 bytes from 1024 bytes)
Flash: [======    ]  62.2% (used 5099 bytes from 8192 bytes)

section       size    addr

DATA            52       1
INITIALIZED     22      53
HOME           107   32768
GSINIT          26   32875
GSFINAL          3   32901
CONST         1588   32904
INITIALIZER     22   34492
CODE          4941   34514
SSEG             1   39455
Total         6762

from new 0.50.0 core:

RAM:   [          ]   1.3% (used 13 bytes from 1024 bytes)
Flash: [====      ]  42.1% (used 3446 bytes from 8192 bytes)

section       size    addr
DATA             9       1
INITIALIZED      4      10
CONST         1606   32768
INITIALIZER      4   34374
CODE          3442   34378
Total         5065

The size decreased quite a bit, but maybe something important was removed? I'll see what I can find out that makes this non-working..

maxgerhardt · 2021-03-22T18:23:14Z

Hm, it works out of the box when installing the 0.5.0 core in the Arduino IDE (https://github.com/tenbaht/sduino#installation) and using the same Tetris code. I'll see what the differences during build are.

In the Arduino IDE, the build size is

Sketch uses 4721 Bytes (57%) of program memory. Maximum is 8192 Bytes.
Global variables use 76 Bytes (7%) of the dynamic memory. 948 Bytess for local variablen remain. Maximum is 1024 Bytes.

But the build there also uses SDCC 3.9.1 #11242 (MINGW32).

maxgerhardt · 2021-03-22T18:38:54Z

This is pretty absurd. Adding a Serial_begin(115200); in the setup() makes the firmware work again with PlatformIO and all latest version (0.5.0 Arduino core, SDCC 4.1.0, latest dev platform). The firmware.map file shows some differences in linked functions, maybe something crucial got optimized away.

valeros · 2021-03-22T18:41:25Z

Any better if you downgrade the toolchain?

maxgerhardt · 2021-03-22T18:49:40Z

Downgrading to 3.8.4 (1.30804.10766) did not work.

But if I use the exact toolchain that SDCC does, 3.9.1 (which isn't even listed in https://sourceforge.net/projects/sdcc/files/sdcc-win64/ so I had to pull it from C:\Users\Max\AppData\Local\Arduino15\packages\sduino\tools\sdcc\build.11242), I first get a linker error

ASlink-Warning-No definition of area SSEG
*** [.pio\build\stm8sblue\firmware.elf] Error 1

which is resolved by the two lines of code found in tenbaht/sduino#76 (comment).

Then it compiles, uploads and runs correctly.

So for some magical reason, that exact SDCC version has to be used and that dummy variable trick. If I read tenbaht/sduino@c7ebb07 correctly that's also what they do with

echo "void main(void); void (*dummy_variable) () = main;"; \

I'll retry with SDCC 4.0 though.

maxgerhardt · 2021-03-22T18:54:09Z

A SDCC 4.0 downloaded straight from https://sourceforge.net/projects/sdcc/files/sdcc-win64/4.0.0/ works too but also needs the dummy_variable workaround. Have not gotten it to work with SDCC 4.1 (with or without dummy_variable).

So I guess it's best to use either SDCC 4.0 or their exact SDCC 3.9.1 with that dummy variable inclusion, and open an issue in SDCC on why SDCC 4.1 won't make it work correctly?

I've looked in to the .map files and optimization in SDCC 4.1 is really aggressive, with most of the unused interrupts and functions removed. Still, SDCC 4.1 does retain the timer 4 overflow interrupt and millis() and everything, so it should work, if that's the correct timer..

If only I could debugging with the Arduino framework to work, then I could look where the chip is stuck..

valeros · 2021-03-22T19:02:28Z

Thanks for the help, let's stick to that SDCC 3.9.1 at least for Sduino.

valeros · 2021-03-23T12:12:34Z

Arduino is now switched to SDCC v3.9.1. I also added a hacky workaround to avoid the ASlink-Warning-No definition of area SSEG error. Would be great if you could retest. Thanks!

maxgerhardt · 2021-03-23T12:19:25Z

Just tested with standard platformio.ini+ dev platform + removed my own dummy_variable definition. Compiles, uploads and works, the Tetris tune plays again.

>pio run
Processing stm8sblue (platform: https://github.com/platformio/platform-ststm8.git; board: stm8sblue; framework: arduino)
------------------------------------------------------------------------------------------------------------------------
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/ststm8/stm8sblue.html
PLATFORM: ST STM8 (1.0.3+sha.acb8a45) > ST STM8S103F3 Breakout Board
HARDWARE: STM8S103F3P6 16MHz, 1KB RAM, 8KB Flash
PACKAGES:
 - framework-arduinoststm8 0.50.210317
 - tool-stm8binutils 0.230.0 (2.30)
 - toolchain-sdcc 1.30901.11242 (3.9.1)
LDF: Library Dependency Finder -> http://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 12 compatible libraries
Scanning dependencies...
No dependencies
Building in release mode
Compiling .pio\build\stm8sblue\src\TetrisThemeArduino.c.rel
Linking .pio\build\stm8sblue\firmware.elf
Checking size .pio\build\stm8sblue\firmware.elf
Advanced Memory Usage is available via "PlatformIO Home > Project Inspect"
RAM:   [=         ]   7.4% (used 76 bytes from 1024 bytes)
Flash: [======    ]  57.6% (used 4721 bytes from 8192 bytes)
Building .pio\build\stm8sblue\firmware.hex
================ [SUCCESS] Took 2.04 seconds ================

The firmware size now also is 100% equal to what the Arduino IDE outputs per #38 (comment).

So, all works normally in release mode :)

The debugging issue with WInterrupts.c is also present in SDCC 3.9.1, but as above, I've filed a bug for that already.

Still sad that the 0.50 core can't be compiled to a working firmware with SDCC 4.1 but 0.40 could. I'll open a separate issue in sduino for that (or SDCC if that's the culprit).

maxgerhardt · 2021-03-23T12:25:09Z

Slightly off-topic, but with platformio/platformio-core#3872 resolved this would make an even better release / less frustration and weirdness for more users 😄

maxgerhardt · 2021-03-23T12:31:29Z

Oh, but debugging has broken again. Pressing the debug button VSCode results in a short loading time, then the debug buttons disappear. I'll check if that change did that.

maxgerhardt · 2021-03-23T12:34:16Z

Yes 100%. Commenting out

platform-ststm8/platform.py

Lines 46 to 47 in acb8a45

    
           if link not in upload_protocols or link in debug["tools"]: 
        
               continue

makes debugging working again with my pio-stm8-headers example from the PR description.

The platformio.ini of that is

[env:stm8sblue_stm8s103f3]
platform = https://github.com/platformio/platform-ststm8.git
upload_protocol = stlinkv2
board = stm8sblue

Maybe has something to do with that upload_protocol can be

-c programmer Specify programmer used (stlink, stlinkv2, stlinkv21 or stlinkv3)

(for stm8flash, and PIO forwards it) and that isn't handled correctly with that continue logic?

valeros · 2021-03-23T12:42:01Z

Thanks, I see now. Removed that lines.

maxgerhardt · 2021-03-23T12:45:54Z

Retested with latest commit and now debugging is working again, Arduino sketch works as before.

maxgerhardt added 10 commits March 10, 2021 19:41

Add Nucleo-8S207K8 and Nucleo-8S208RB

43c63a7

Also corrects the logic that derives the value for -p switch in the stm8flash invocation (would do 'stm8s207?8' and not 'stm8s207k8')

Use correct LED for Nucleo boards in spl-blink example

28339c3

Both the boards have the user-LED connected to PC5 per board manual. E.g. https://www.st.com/resource/en/user_manual/dm00489875-stm8-nucleo32-board-mb1442-stmicroelectronics.pdf

Add SVD files

1e7adca

Restructure board files for debugging

7bcc53d

First version platform.py

d3c94cf

Work around Intel HEX generation bug

4b6556c

Better way to remove debug info

e4fab07

Correct OpenOCD adapter and typos

9227c13

Add right build flags debugging

8168e24

Make OpenOCD non-optional

e9373b1

maxgerhardt mentioned this pull request Mar 16, 2021

Add Nucleo-8S207K8 and Nucleo-8S208RB support #37

Closed

valeros requested changes Mar 16, 2021

View reviewed changes

boards/nucleo_8s207k8.json Outdated Show resolved Hide resolved

builder/main.py Outdated Show resolved Hide resolved

platform.py Outdated Show resolved Hide resolved

maxgerhardt added 4 commits March 21, 2021 20:58

Add newline to board manifest files

f629e7c

Remove unnecessary string format

ed212d0

Unify usage of double quotes

e13bc79

valeros reviewed Mar 22, 2021

View reviewed changes

platform.json Outdated Show resolved Hide resolved

valeros reviewed Mar 22, 2021

View reviewed changes

platform.py Outdated Show resolved Hide resolved

maxgerhardt added 2 commits March 22, 2021 14:38

Removed empty overrided function

9254750

Make OpenOCD optional again

2d2fec8

This will be re-downloaded before a debugging session starts and we do not need to worry.

valeros merged commit 3dd7514 into platformio:develop Mar 22, 2021

This was referenced Mar 23, 2021

Add debug support using STLink(/V2) via OpenOCD #31

Closed

Please add support for Nucleo-8S207K8 board #34

Closed

This was referenced Mar 23, 2021

Debugging not working with SPL examples #44

Open

Debugging not working with Arduino examples #48

Open

maxgerhardt mentioned this pull request Jun 10, 2021

Peripherial view does not show 8bit addressBlocks platformio/platformio-vscode-ide#2586

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add two Nucleo boards, GDB debugging and SVD files for all boards #38

Add two Nucleo boards, GDB debugging and SVD files for all boards #38

maxgerhardt commented Mar 16, 2021 •

edited

valeros commented Mar 16, 2021

maxgerhardt commented Mar 17, 2021

maxgerhardt commented Mar 21, 2021

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 •

edited

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

valeros commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 •

edited

valeros commented Mar 22, 2021

valeros commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021 •

edited

maxgerhardt commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021 •

edited

maxgerhardt commented Mar 23, 2021 •

edited

valeros commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021

Add two Nucleo boards, GDB debugging and SVD files for all boards #38

Add two Nucleo boards, GDB debugging and SVD files for all boards #38

Conversation

maxgerhardt commented Mar 16, 2021 • edited

valeros commented Mar 16, 2021

maxgerhardt commented Mar 17, 2021

maxgerhardt commented Mar 21, 2021

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 • edited

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 • edited

maxgerhardt commented Mar 22, 2021 • edited

valeros commented Mar 22, 2021 • edited

maxgerhardt commented Mar 22, 2021 • edited

maxgerhardt commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 • edited

maxgerhardt commented Mar 22, 2021 • edited

valeros commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021

maxgerhardt commented Mar 22, 2021 • edited

valeros commented Mar 22, 2021

valeros commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021 • edited

maxgerhardt commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021 • edited

maxgerhardt commented Mar 23, 2021 • edited

valeros commented Mar 23, 2021

maxgerhardt commented Mar 23, 2021

maxgerhardt commented Mar 16, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

valeros commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 22, 2021 •

edited

maxgerhardt commented Mar 23, 2021 •

edited

maxgerhardt commented Mar 23, 2021 •

edited

maxgerhardt commented Mar 23, 2021 •

edited