use SWIM pins as UART by cuvoodoo · Pull Request #498 · blackmagic-debug/blackmagic

cuvoodoo · 2019-08-03T19:36:49Z

some STM ST-LINK/V2 also provide SWIM functionality to flash STM8.
the SWIM pins are connected to UART (USART1 REMAP):

SWIM_RST is connected to PB6/USART1_TX
SWIM is connected to PB7/USART1_RX (pulled up by 680R)

cheap ST-LINK/V2 clones in form of USB dongle with aluminium case
provide SWD and SWIM signals.
since blackmagic does not support SWIM functionality, the pins can
be re-used as UART port for the USB to UART bridge.

STM ST-LINK/V2 can also offer UART, using USART2, but then the
SWIM signals are not present.
this changes verifies if the SWIM signals are present and remaps
them to USART1 if this is the case.

This commit modifies lpc_iap_call() to work with IAP commands that return additional data. If the "result" argument is non-null, 16 bytes of data (the maximum returned by any IAP command) are copied to the specified address.

Restore the RAM and registers which are clobbered by an LPC IAP call. This does not restore any additional RAM which might be clobbered by a *particular* IAP call. (For example, flash programming always clobbers the last page of RAM.)

some STM ST-LINK/V2 also provide SWIM functionality to flash STM8. the SWIM pins are connected to UART (USART1 REMAP): - SWIM_RST is connected to PB6/USART1_TX - SWIM is connected to PB7/USART1_RX (pulled up by 680R) cheap ST-LINK/V2 clones in form of USB dongle with aluminium case provide SWD and SWIM signals. since blackmagic does not support SWIM functionality, the pins can be re-used as UART port for the USB to UART bridge. STM ST-LINK/V2 can also offer UART, using USART2, but then the SWIM signals are not present. this changes verifies if the SWIM signals are present and remaps them to USART1 if this is the case.

This prepares to allow other architectures than pc that don't use libopencm3. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

This allows the platform to specify another optimization level than the default (like -Os if space is limited) Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

It's a global symbol and LTO will complain if the one in this file and the one in EFM32 target are inconsistent. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

snprintf is needed anyway, that's one less function to have :p And it's bad practice anyway. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

UweBonnes · 2019-08-13T17:51:50Z

Did you check that your patches do not collide with the SWO functionality?

Minor enhancements

Required fixes: * use usbd_request_return_codes -- commit 54b117c * drop deprecated timer_reset() -- commit 034dbf2 * drop 48 & 120 MHz configs -- commit a9dde28 * ld scripts: drop duplication of standard sections -- commit 9a05dcb

Target latest libopencm3 version

cuvoodoo · 2019-08-18T13:40:55Z

I did not test it.

For the stlink platform SWO is defined on UART1_RX/PA10
On the baite and mini/aluminium ST-Link/V2 clones with SWIM, PA10 is not connected.
The baite has JTDI, but it cannot be reused as SWO since it's only on the GPIO PA4.
Thus the clones do not support SWO.

The ST-Link/V2-1 supports SWO on UART1_RX/PA10, but since it is not detected as clone with SWIM support, the UART1 is not re-used on the SWIM pin.
Thus this functionality should not be affected.

Ideally for the clones with SWIM, PLATFORM_HAS_TRACESWO should be set to 0.
But since we are trying get use the same platform this should not be redefined with less support.
The clone user should just not use the SWO functionality.

FIXME: Writing CPU registers on M4 of STM32H745 seems not to work.

Fix as seen in openocd. SYSROM_PIDR2 has only 3 bits for the JEP106 identity code field, bits [6:4].

…umber.

Pc stlinkv2

After update of GNU make the tests for Windows OS are no longer needed. Note that GNU make 4.2.1 is used to build BMP under Windows OS. Make native working Again, the newer version of GCC make does not need the OS tests.

Noticed in blackmagic-debug#506.

swlink: Remove redundant conditional usbart_init().

Added braces for compatibility with python3, should still be valid python2 which interpretes these braces as a tuple with a single element and simplifies it to that element. Replaced whitespaces with tabs as required by python3, since indentation must be uniform (either whitespaces or tabs). Removed trailing whitespaces. Tested by flashing various STM32 boards with blackmagic firmware using python3.7.

Add LPC11xx command to read out unique ID from target. Restore Ram and registers after call.

Rework MSC layout check

This is a bit in the Lock Bit (LB) flash page, so it can only be cleared by this routine

Supported functionality through this AP: * Issuing a DEVICEERASE command

Rebase EFM patch blackmagic-debug#438, compiling fine in all intermediate steps

…boarder.

Needed when reading some memory region of unknown size.

- Program, read and verify flash - Test mode: Probe and attach to device and exit

Command line interaction for pc-hosted platforms.

Issue reset only when flashing or when reset was requested.

… flash. Allow to erase from command line.

Both V2 and V21 switch to (ST) bootloader with powered reset.

some STM ST-LINK/V2 also provide SWIM functionality to flash STM8. the SWIM pins are connected to UART (USART1 REMAP): - SWIM_RST is connected to PB6/USART1_TX - SWIM is connected to PB7/USART1_RX (pulled up by 680R) cheap ST-LINK/V2 clones in form of USB dongle with aluminium case provide SWD and SWIM signals. since blackmagic does not support SWIM functionality, the pins can be re-used as UART port for the USB to UART bridge. STM ST-LINK/V2 can also offer UART, using USART2, but then the SWIM signals are not present. this changes verifies if the SWIM signals are present and remaps them to USART1 if this is the case.

…uart

dragonmux · 2022-06-28T11:48:02Z

Thank you for the contribution, however unfortunately it is a duplicate of #396 and doesn't really fit the direction the project is being taken. Closing in favour of #396.

dlaw and others added 9 commits May 30, 2019 14:01

Add LPC command to read out unique ID from target.

d3979a5

This commit modifies lpc_iap_call() to work with IAP commands that return additional data. If the "result" argument is non-null, 16 bytes of data (the maximum returned by any IAP command) are copied to the specified address.

document SWIM to UART

1725094

build: Use a separate NO_LIBOPENCM3 flag to disable libopencm3

ef372d0

This prepares to allow other architectures than pc that don't use libopencm3. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

build: Apply OPT_FLAGS after platform Makefile.inc include

804ec24

This allows the platform to specify another optimization level than the default (like -Os if space is limited) Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

build: Allow support for .S assembly file in SRC

544bcd9

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

target: Make sure variant_string is consistent in size

1d4152a

It's a global symbol and LTO will complain if the one in this file and the one in EFM32 target are inconsistent. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

src: Replace sprintf with snprintf

4289788

snprintf is needed anyway, that's one less function to have :p And it's bad practice anyway. Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

UweBonnes and others added 3 commits August 13, 2019 19:58

Merge pull request blackmagic-debug#499 from smunaut/minor_enhancements

c9c8b08

Minor enhancements

Target latest libopencm3

443ced6

Required fixes: * use usbd_request_return_codes -- commit 54b117c * drop deprecated timer_reset() -- commit 034dbf2 * drop 48 & 120 MHz configs -- commit a9dde28 * ld scripts: drop duplication of standard sections -- commit 9a05dcb

Merge pull request blackmagic-debug#500 from btashton/update_libopencm3

e52e2f5

Target latest libopencm3 version

esden and others added 16 commits August 26, 2019 17:22

Changed the Gitter chat link to Discord.

ebf3e9d

pc-stlinkv2: CPU register read and write must be done with the AP set.

6bf4fd3

FIXME: Writing CPU registers on M4 of STM32H745 seems not to work.

Remove debug_bmp command on PC hosted platforms.

fa27649

PC-STLINKV2/Libftdi: Better version string.

5fbb3a4

Stlink: Add a README.md, mostly deprecating reflashing Stlinkv2/1.

82be49f

pc-stlinkv2: READMEM_8BIT needs to read 2 bytes for 1 byte requested.

44f471d

Fix as seen in openocd. SYSROM_PIDR2 has only 3 bits for the JEP106 identity code field, bits [6:4].

pc-stlinkv2: Try first to reset device if we read a strange version n…

5d59338

…umber.

Merge pull request blackmagic-debug#505 from UweBonnes/pc-stlinkv2

e8bd066

Pc stlinkv2

Windows Build Fixes GNU make 4.2.1 (blackmagic-debug#502)

35cb4e5

After update of GNU make the tests for Windows OS are no longer needed. Note that GNU make 4.2.1 is used to build BMP under Windows OS. Make native working Again, the newer version of GCC make does not need the OS tests.

swlink: Remove redundant conditional usbart_init().

2b7d4c5

Noticed in blackmagic-debug#506.

Merge pull request blackmagic-debug#508 from UweBonnes/swlink

5704c2f

swlink: Remove redundant conditional usbart_init().

pc-stlinkv2: Fix crash with serial given not matching connected devices.

d994565

pc-stlinkv2: Return failure if STLINK_DEBUG_APIV2_INIT_AP fails.

a011095

adiv5: If setup AP0 fails, fail immediate.

aef055b

pc-stlinkv2: Remove redundant read of CoreID.

0d61106

ruabmbua and others added 22 commits December 8, 2019 14:34

Support arbitrary bit counts jtagtap_tdi_tdo_seq pc-hosted

5ec12e4

Merge pull request blackmagic-debug#203 from dlaw/master

f89542c

Add LPC11xx command to read out unique ID from target. Restore Ram and registers after call.

[efm32] add new devices PG12B, JG12B, GG11B, TG11B, GG12B

e85df76

Rework MSC layout check

[efm32] Print MSC Interrupt Flags to DEBUG after each flash write

7f0d5fe

[efm32] Add command to set and print bootloader enable status

260fc88

This is a bit in the Lock Bit (LB) flash page, so it can only be cleared by this routine

[efm32] Probe for the EFM32 Authentication Access Port (AAP)

5943552

Supported functionality through this AP: * Issuing a DEVICEERASE command

Merge pull request blackmagic-debug#567 from UweBonnes/438

ea779d1

Rebase EFM patch blackmagic-debug#438, compiling fine in all intermediate steps

efm32: Allow to compile with -Og.

25d24e5

Erase: Fix endless erase when erase-area did not end on (page|block) …

bdd76de

…boarder.

stlinkv2: Propagate AP errors

6d01c3c

Needed when reading some memory region of unknown size.

adiv5: Abort scanning APs after 8 void APs.

1bef51e

Allow %z specifier in windows builds. Supercedes blackmagic-debug#562.

ab396f9

Infrastructure for command line interface to target

da5a0cd

- Program, read and verify flash - Test mode: Probe and attach to device and exit

Merge pull request blackmagic-debug#561 from UweBonnes/pc_prog

0b6f393

Command line interaction for pc-hosted platforms.

cl_utils: Allow to issue reset.

269d38c

Issue reset only when flashing or when reset was requested.

Command line: Restrict verify to length of binary file.

b9eb467

target_flash_erase: Do not crash when requesting erase of unavailable…

470c8e8

… flash. Allow to erase from command line.

stlink for st-bootloader: Reset with platform_request_boot()

ee31643

Both V2 and V21 switch to (ST) bootloader with powered reset.

Hosted: Allow command line interaction.

c3a3f77

document SWIM to UART

462a37a

Merge branch 'swim_uart' of github.com:cuvoodoo/blackmagic into swim_…

ddb94c7

…uart

dragonmux mentioned this pull request Jun 28, 2022

Add option to turn SWIM/RST into UART RX/TX #396

Closed

dragonmux added this to the v1.9 release milestone Jun 28, 2022

dragonmux added Enhancement General project improvement HwIssue Mitigation Solving or mitigating a Hardware issue in Software BMP Firmware Black Magic Probe Firmware (not PC hosted software) and removed New Host Board New hardware to run Black Magic firmware on labels Jun 28, 2022

dragonmux closed this Jun 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Comments

use SWIM pins as UART#498

use SWIM pins as UART#498
cuvoodoo wants to merge 98 commits intoblackmagic-debug:mainfrom
cuvoodoo:swim_uart

cuvoodoo commented Aug 3, 2019

Uh oh!

UweBonnes commented Aug 13, 2019

Uh oh!

cuvoodoo commented Aug 18, 2019

Uh oh!

dragonmux commented Jun 28, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Uh oh!

Comments

Conversation

cuvoodoo commented Aug 3, 2019

Uh oh!

UweBonnes commented Aug 13, 2019

Uh oh!

cuvoodoo commented Aug 18, 2019

Uh oh!

dragonmux commented Jun 28, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants