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.

Sign up for GitHub

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

Jump to bottom

A bunch of flashing doc updates, including how to recover a device that isn't getting back to toboot after first flash, fixing udev, and flashing from chrome browser #105

Merged
merged 5 commits into from
Jan 15, 2021
Merged

A bunch of flashing doc updates, including how to recover a device that isn't getting back to toboot after first flash, fixing udev, and flashing from chrome browser #105

merged 5 commits into from
Jan 15, 2021

Conversation

marcmerlin
Copy link
Contributor

No description provided.

@mithro
Copy link
Member

mithro commented Dec 30, 2020

@marcmerlin - BTW It should work without root if you have the udev rules installed.

@marcmerlin
Copy link
Contributor Author

I'm sending a new/better version

@marcmerlin
Copy link
Contributor Author

Just FYI, I got ATTRS{idVendor}=="0483", ATTRS{idProduct}=="cdab" from https://tomu.im/tomu.html which @mithro recommended, but the USB numbers don't match on my tomus. I left it as the first line in case it's needed for other tomus.

@marcmerlin marcmerlin mentioned this pull request Dec 30, 2020
Open
@marcmerlin marcmerlin changed the title Make sure to run dfu-util with root, and show what happens without A bunch of flashing doc updates, including how to recover a device that isn't getting back to toboot after first flash, fixing udev, and flashing from chrome browser Dec 30, 2020
@xobs
Copy link
Member

xobs commented Dec 31, 2020

The vendor ID 0x0483 appears to be from ST: https://www.the-sz.com/products/usbid/index.php?v=0x0483

Likely someone had an ST product present in their system when they looked at the output. ST has their own, incompatible implementation of DFU. The page you linked to is incorrect -- I'll update it now, thanks for pointing that out!

@marcmerlin
Copy link
Contributor Author

Thanks @xobs . I cleaned this up a bit. Is it good for merge now?

@marcmerlin marcmerlin mentioned this pull request Jan 3, 2021
Open
@marcmerlin
Copy link
Contributor Author

I know that tomu is not the new hotness, but @mithro are you ok merging those doc updates and fixes?

mithro
mithro approved these changes Jan 14, 2021
View reviewed changes
Copy link
Member

@mithro mithro left a comment

Choose a reason for hiding this comment

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

These updates LGTM.

@xobs could you merge?

@xobs xobs merged commit 3607701 into im-tomu:master Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mithro mithro mithro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@marcmerlin @mithro @xobs