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

Device ID reported by DFU is capitalized #1432

Closed
monkbroc opened this issue Nov 15, 2017 · 1 comment · Fixed by #1437

Comments

@monkbroc
Copy link
Member

commented Nov 15, 2017

Bug Report

Listing devices in DFU mode list the device ID as capitalized.

Observed Behavior

$ dfu-util -l
dfu-util 0.9

Copyright 2005-2009 Weston Schmidt, Harald Welte and OpenMoko Inc.
Copyright 2010-2016 Tormod Volden and Stefan Schmidt
This program is Free Software and has ABSOLUTELY NO WARRANTY
Please report bugs to http://sourceforge.net/p/dfu-util/tickets/

Found DFU: [2b04:d006] ver=0250, devnum=94, cfg=1, intf=0, path="1-2.3.1", alt=1, name="@DCT Flash   /0x00000000/01*016Kg", serial="1A002A000347353100000000"
Found DFU: [2b04:d006] ver=0250, devnum=94, cfg=1, intf=0, path="1-2.3.1", alt=0, name="@Internal Flash   /0x08000000/03*016Ka,01*016Kg,01*064Kg,07*128Kg", serial="1A002A000347353100000000"

Expected Behavior

Found DFU: [2b04:d006] ver=0250, devnum=94, cfg=1, intf=0, path="1-2.3.1", alt=1, name="@DCT Flash   /0x00000000/01*016Kg", serial="1a002a000347353100000000"

References

Here's the code that converts a byte buffer to a capitalized hex string.
https://github.com/spark/firmware/blob/60f703bd4165a292f89124a7e3755e4eb37b1a03/services/inc/bytes2hexbuf.h#L24

@m-mcgowan m-mcgowan added this to the 0.7.0 milestone Nov 16, 2017

@m-mcgowan

This comment has been minimized.

Copy link
Contributor

commented Nov 16, 2017

Since we are making bootloader changes for the next pre-release, let's get this one in too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.