mbox series

[v3,00/24] Fix various bugs

Message ID 20240815202535.767869-1-sjg@chromium.org
Headers show
Series Fix various bugs | expand

Message

Simon Glass Aug. 15, 2024, 8:25 p.m. UTC 
This series includes the patches needed to make make the EFI 'boot' test
work. That test has now been split off into a separate series along with
the EFI patches.

This series fixes these problems:
- sandbox memory-mapping conflict with PCI
- the fix for that causes the mbr test to crash as it sets up pointers
  instead of addresses for its 'mmc' commands
- the mmc and read commands which cast addresses to pointers
- a tricky bug to do with USB keyboard and stdio
- a few other minor things

Changes in v3:
- Include the usb.h header file in all cases
- Correct the commit subject and message
- Add a Fixes tag
- use SZ_512 instead of 0x200

Changes in v2:
- Add many new patches to resolve all the outstanding test issues

Simon Glass (24):
  nvmxip: Drop the message on probe
  nvmxip: Avoid probing on boot
  bootstd: Add UT_TESTF_CONSOLE_REC to bootflow tests
  test/py: Fix some pylint warnings in test_ut.py
  scripts: Update pylint.base
  bootstd: Create a function to reset USB
  usb: Drop old non-DM code
  log: Add a new log category for the console
  usb: Add DEV_FLAGS_DM to stdio for USB keyboard
  dm: usb: Deal with USB keyboard persisting across tests
  test: mbr: Adjust test to use lower-case hex
  test: mbr: Adjust test to drop 0x
  sandbox: Change the range used for memory-mapping tags
  sandbox: Update cpu to use logging
  sandbox: Unmap old tags
  sandbox: Add some debugging to pci_io
  sandbox: Implement reference counting for address mapping
  mmc: Use map_sysmem() with buffers in the mmc command
  read: Tidy up use of map_sysmem() in the read command
  cmd: Fix memory-mapping in cmp command
  test: mbr: Unmap the buffers after use
  test: mbr: Use a constant for the block size
  test: mbr: Use RAM for the buffers
  test: mbr: Drop a duplicate test

 arch/sandbox/cpu/cpu.c             |  38 ++-
 arch/sandbox/cpu/state.c           |   9 +-
 arch/sandbox/include/asm/state.h   |   3 +
 arch/sandbox/lib/pci_io.c          |   9 +-
 cmd/mem.c                          |  26 +-
 cmd/mmc.c                          |  15 +-
 cmd/read.c                         |  10 +-
 cmd/usb.c                          |  20 --
 common/console.c                   |  36 +++
 common/log.c                       |   1 +
 common/usb_kbd.c                   |  74 +----
 doc/arch/sandbox/sandbox.rst       |  21 +-
 drivers/mtd/nvmxip/nvmxip-uclass.c |  10 +-
 drivers/usb/Kconfig                |   3 +-
 include/console.h                  |   8 +
 include/log.h                      |   2 +
 include/usb.h                      |  20 +-
 scripts/pylint.base                | 462 +++++++++++++++++------------
 test/boot/bootdev.c                |  19 +-
 test/boot/bootflow.c               |  59 ++--
 test/boot/bootstd_common.c         |   6 +
 test/boot/bootstd_common.h         |   8 +
 test/cmd/mbr.c                     | 173 +++++------
 test/py/tests/test_ut.py           |  94 +++---
 test/test-main.c                   |  38 +++
 25 files changed, 651 insertions(+), 513 deletions(-)