Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flash.c: stlink_open_usb is actually able to open v1 and v2 (and v3) #922

Merged

Conversation

grevaillot
Copy link
Collaborator

flash.c: stlink_open_usb is actually able to open v1 and v2 (and v3 .…….)** devices.

no need to check that not used o.devname ..
no need to specify devname in commandline

get rid of the devname cmdline stuff, and update comments

should probably help with #378 ..

Copy link
Member

@Nightwalker-87 Nightwalker-87 left a comment

Choose a reason for hiding this comment

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

Yeah, very good - just get rid of that old junk... 👍

@Nightwalker-87
Copy link
Member

This is actually a result of #918, right?

@grevaillot
Copy link
Collaborator Author

i think it was half-working before my previous PR - i'm pretty sure i managed to use a v1 without specifying any device path, at least this could work in specific conditions.

@Nightwalker-87
Copy link
Member

@grevaillot: Can you update this PR to resolve the conflict in flash.c? Is there any specific ticket to be linked to this PR as seen from your side?

@grevaillot
Copy link
Collaborator Author

pr conflict fixed - not much time to scan old tickest - This can probably close initial issue of #378 - v1/v2 selection issue. Still, switching to something "working" to parse cmdline opt would be nice, but we now avoid that bug.

src/tools/flash_opts.c Outdated Show resolved Hide resolved
….) devices.

no need to check that _not used_ o.devname ..
no need to specify devname in commandline

get rid of the devname cmdline stuff, and update comments
@Nightwalker-87
Copy link
Member

not much time to scan old tickets - This can probably close initial issue of #378 - v1/v2 selection issue. Still, switching to something "working" to parse cmdline opt would be nice, but we now avoid that bug.

Thx, not necessary for you to do so, I think we're fine to link #378 here. Any further more general improvements for opt parsing should be addressed by a new ticket, should such be desired.

@Nightwalker-87 Nightwalker-87 linked an issue Apr 14, 2020 that may be closed by this pull request
@Nightwalker-87 Nightwalker-87 merged commit b5be03a into stlink-org:develop Apr 14, 2020
@Nightwalker-87
Copy link
Member

Merci @grevaillot!

@grevaillot grevaillot deleted the to_merge/st-flash_v1v2 branch April 21, 2020 15:03
@stlink-org stlink-org locked as resolved and limited conversation to collaborators May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

CLI tools need better argument parsing
3 participants