-
Notifications
You must be signed in to change notification settings - Fork 41
50-snagboot.rules: add support for "Toradex USB download gadget" #61
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
base: main
Are you sure you want to change the base?
50-snagboot.rules: add support for "Toradex USB download gadget" #61
Conversation
Tested on Toradex Verdin iMX8MM Mini V1.1D in Dahlia V1.1D carrier board Signed-off-by: Michael Opdenacker <[email protected]>
For your information, the "snagrecover" command executes correctly (which was not the case before applying the proposed changes), but there are still errors during execution: snagrecover -s imx8mm -f imx8-verdin-mx8m-mini.yaml There could be issues with my flashing image too (work in progress). |
Hello Michael, thanks for the PR. Could you please elaborate on what kind of issues you've encountered before applying this udev rule? Snagrecover only needs a udev rule for the ROM code stage of the recovery process. There is already a rule in place for i.MX8MM ROM code gadgets, so I'm a bit confused as to why you'd need an additional rule. Some logs of your previous errors would help. On the other hand, if this USB gadget is only used in the flashing stage, then it's out of scope for snagrecover, and shouldn't be included in these udev rules. We voluntarily avoid writing udev rules for flashing gadgets because their VID:PID pairs can be customized at will by vendors, so we can't reasonably write rules for all of them. |
Hi Romain snagrecover -s imx8mm -f imx8-verdin-mx8m-mini.yaml Happy to run as many tests as needed and to contribute to Snagboot if I can. |
Tested on Toradex Verdin iMX8MM Mini V1.1D in Dahlia V1.1D carrier board