Skip to content

Conversation

jeromecoutant
Copy link
Contributor

Description

With the latest ST Link FW version available in https://www.st.com/en/development-tools/stsw-link007.html
ST-Link FW version is now displayed in the mbedls console print:

$ mbedls
| platform_name       | platform_name_unique   | mount_point | serial_port | target_id                | daplink_version |
|---------------------|------------------------|-------------|-------------|--------------------------|-----------------|
| DISCO_H747I         | DISCO_H747I[0]         | D:          | COM13       | 081402210D03E72132477E08 | V3J7M2          |
| DISCO_L475VG_IOT01A | DISCO_L475VG_IOT01A[0] | E:          | COM9        | 07640221683B630A577FF553 | V2J37M26        |

@MarceloSalazar proposed to replace "daplink_version" by "interface_version"

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@MarceloSalazar
Copy link

I'm afraid isn't that simple. The fw version for DAPLink version is shown as unknown

| platform_name | platform_name_unique | mount_point      | serial_port            | target_id  | interface_version |
|---------------|----------------------|------------------|------------------------|------------|-------------------|
| K64F          | K64F[0]              | /Volumes/DAPLINK | /dev/tty.usbmodem14102 | 0240...... | unknown           |

@jeromecoutant
Copy link
Contributor Author

Mmm, it's my fault, I didn't test :-(

@jeromecoutant
Copy link
Contributor Author

Looks OK now:

$ mbedls
| platform_name       | platform_name_unique   | mount_point | serial_port | target_id                | interface_version |
|---------------------|------------------------|-------------|-------------|--------------------------|-------------------|
| DISCO_H747I         | DISCO_H747I[0]         | D:          | COM13       | 081402210D03E72132477E08 | V3J7M2            |
| DISCO_L475VG_IOT01A | DISCO_L475VG_IOT01A[0] | E:          | COM9        | 07640221683B630A577FF553 | V2J37M26          |

@MarceloSalazar MarceloSalazar self-requested a review May 28, 2020 15:42
Copy link

@MarceloSalazar MarceloSalazar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - I confirm it works ok on DAPLink based boards.

@MarceloSalazar
Copy link

FYI @Patater

@jeromecoutant
Copy link
Contributor Author

ping

@rwalton-arm
Copy link
Contributor

Sorry for the delay. I see we've confirmed this works on DAPLink boards.

@rwalton-arm rwalton-arm merged commit c960314 into ARMmbed:master Jun 8, 2020
@jeromecoutant jeromecoutant deleted the PR_INTERFACE branch June 8, 2020 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants