search

devanlai / dapboot

DFU Bootloader for STM32 chips
Other
339 stars 111 forks source link

Using WINUSB_MS_VENDOR_CODE in extra string #7

Closed karelbilek closed 5 years ago

karelbilek commented 7 years ago

This might make some future debugging easier.

I did not test this, but it should work in theory :)

karelbilek commented 7 years ago

You are probably right. I did not test it on our device yet (WinUSB still doesn't load correctly; I think I will have to add the extended properties descriptor).

karelbilek commented 6 years ago

It's static const now.

devanlai commented 6 years ago

Thanks, when I get a chance to dig out my windows test machine, I'll verify that it still works.

karelbilek commented 6 years ago

Seems to be working in our FW

https://github.com/trezor/trezor-mcu/pull/260

devanlai commented 5 years ago

@karel-3d Sorry this took so long to merge - I just finally got around to digging out my Windows 10 test machine. Thanks for the patch!

karelbilek commented 5 years ago

Ha I don’t work at the job I needed this for anymore. Thanks for replying though :) and patching

.... now I remember all the USB debugging pain again. >:(

On Mon, 19 Aug 2019 at 23:53, Devan Lai notifications@github.com wrote:

@karel-3d https://github.com/karel-3d Sorry this took so long to merge

  • I just finally got around to digging out my Windows 10 test machine. Thanks for the patch!

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/devanlai/dapboot/pull/7?email_source=notifications&email_token=AAAZT4PVMYVEWP2TESSTZDDQFLFZ7A5CNFSM4EGOIALKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4TTE3I#issuecomment-522662509, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAZT4L4KQZMQNKCE24EGXDQFLFZ7ANCNFSM4EGOIALA .