Skip to content
Commit 59ca3a2f authored by Simon Glass's avatar Simon Glass Committed by Tom Rini
Browse files

usb: bootm: Drop old USB-device-removal code



USB is stopped using driver model now, in dm_remove_devices_flags() in
announce_and_cleanup() at the top of this file.

The usb_stop() call actually unbinds devices.

When a USB device is unbound, it causes any bootflows attached to it to
be removed, via a call to bootdev_clear_bootflows() from
bootdev_pre_unbind(). This obviously makes it impossible to boot the
bootflow.

However, when booting a bootflow that relies on USB, usb_stop() is
called, which unbinds the device. At that point any information
attached to the bootflow is dropped.

This is quite risky since the contents of freed memory are not
guaranteed to remain unchanged. Depending on what other options are
done before boot, a hard-to-find bug may crop up.

Drop the call to this old function.

Leave the netconsole call there, since this needs conversion to
driver model.

Signed-off-by: default avatarSimon Glass <sjg@chromium.org>
Suggested-by: default avatarShantur Rathore <i@shantur.com>
Reviewed-by: default avatarMattijs Korpershoek <mkorpershoek@baylibre.com>
parent c502d7ad
Loading
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment