Dear Tom,
I am still trying to reproduce the usb_stop() issue on the sandbox, to understand what is wrong in DM. So your patch will go to next weeks pull request. But generally you are right that usb_stop() must be called before deleting the devices if at all.
The following changes since commit 454a97485a1854d4ac4f3b7d408caf3b355beca0:
Merge tag 'u-boot-amlogic-20211119' of https://source.denx.de/u-boot/custodians/u-boot-amlogic (2021-11-19 16:33:33 -0500)
are available in the Git repository at:
https://source.denx.de/u-boot/custodians/u-boot-efi.git efi-2022-01
for you to fetch changes up to 8bb8cde4f56ec45ac7b48fefc5bb0430b29e2878:
efi_loader: startup the tpm device when installing the protocol (2021-11-20 10:27:56 +0100)
Gitlab CI reported no issues: https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/9925
---------------------------------------------------------------- Pull request for efi-2022-01-rc3
Scripts: * Update spelling.txt
LMB: * remove extern keyword in lmb.h * drop unused lmb_size_bytes()
Test: * test truncation in snprintf()
Documentation: * add include/lmb.h to HTML documentation
UEFI: * reduce non-debug logging output for measured boot * fix use after free in measured boot * startup the tpm device when installing the protocol * implement EFI_EVENT_GROUP_BEFORE_EXIT_BOOT_SERVICES * record capsule result only if capsule is read
---------------------------------------------------------------- Heinrich Schuchardt (13): lmb: remove lmb_is_nomap() from include doc: add include/lmb.h to the HTML documentation lmb: remove extern keyword in lmb.h lmb: drop unused lmb_size_bytes() lmb: fix typo 'commun' test: test truncation in snprintf() configs: enable CMD_TPM on QEMU ARM scripts: update spelling.txt from upstream Linux doc: fix typos in trace.rst efi_loader: Sphinx comments in efi_api.h efi_loader: EFI_EVENT_GROUP_BEFORE_EXIT_BOOT_SERVICES efi_selftest: unit test for EFI_GROUP_BEFORE_EXIT_BOOT_SERVICE efi_loader: bump EFI_SPECIFICATION_VERSION to 2.9
Ilias Apalodimas (2): efi_loader: fix FinalEvents table if an EFI app invoked GetEventLog efi_loader: startup the tpm device when installing the protocol
Masahisa Kojima (2): efi: add comment for efi_system_table and efi_configuration_table efi_loader: Reduce efi_tcg2 logging statement
Masami Hiramatsu (1): efi_loader: capsule: Record capsule result only if capsule is read
configs/qemu_arm64_defconfig | 1 + configs/qemu_arm_defconfig | 1 + doc/api/index.rst | 1 + doc/api/lmb.rst | 7 + doc/develop/trace.rst | 16 +- include/efi_api.h | 59 +++- include/lmb.h | 72 ++--- lib/Kconfig | 2 +- lib/efi_loader/efi_boottime.c | 13 + lib/efi_loader/efi_capsule.c | 6 +- lib/efi_loader/efi_tcg2.c | 110 +++++-- lib/efi_selftest/efi_selftest_exitbootservices.c | 67 ++++- scripts/spelling.txt | 365 ++++++++++++++++++++++- test/lib/lmb.c | 5 + test/print_ut.c | 18 ++ test/py/tests/test_tpm2.py | 1 + 16 files changed, 634 insertions(+), 110 deletions(-) create mode 100644 doc/api/lmb.rst