Can't set program_usb_boot_mode pi-config option

Bug #1689302 reported by Sergey Borovkov
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
snapd
Confirmed
Medium
Unassigned

Bug Description

It allows to boot from usb. Core allows it but currently there is no way to set it. Oliver suggested this on this PR https://github.com/snapcore/core/pull/38#issuecomment-299859765

>given that this option permanently changes the ROM code i'm not sure we should expose it in a generic config option you can run on any Pi.
i'd recommend to rather ship it in the gadget config.txt for this specific device and make the config-option carry over that setting when re-generating config.txt if it exists.

Tags: ce
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

What is the status of this bug? It was filed by a client project developer and I'd like to know if it is fixed, invalid, etc. Cheers.

Simon Fels (morphis)
tags: added: ce
Revision history for this message
Oliver Grawert (ogra) wrote :

this should simply be set in their gadget snap in the original config.txt. the bootloader will apply it before the first boot (where potentially the config.txt gets re-written by the core config hook)

Revision history for this message
Simon Fels (morphis) wrote :

@Sergey, Kyle: Would Oliver's suggestion help to solve your issue?

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

Sergey asked whether this would require two gadget snaps: one for device that boot from usb, one from devices that work from sd.

Only one gadget snap would seem to be needed IF there is a boot fallback: try usb, else try sdd.

Does this solution use such a fallback?

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I just checked that program_usb_mode is indeed not surfaced through snappy configuration. Having said that, it feels like a weird configuration, especially since it is one-way-fuse that is being programmed.

I'm marking it as confirmed / medium but I think it warrants more discussion on how it should be exposed.

affects: snappy → snapd
Changed in snapd:
status: New → Confirmed
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.