[PATCH v2 0/6] spi: spi-uclass: Fix spi_claim_bus() speed/mode setup logic

Ovidiu Panait ovidiu.panait at windriver.com
Mon Dec 14 18:06:45 CET 2020


This patchset fixes spi_claim_bus() handling of the speed/mode settings
when multiple spi slaves claim the bus consecutively, as reported here:
https://lists.denx.de/pipermail/u-boot/2019-December/393854.html
https://lists.denx.de/pipermail/u-boot/2020-November/431554.html

It also does some minor cleanups and adds a sandbox testcase for
spi_claim_bus().

Changes in v2:
- add reviewed-by tag
- add reviewed-by tag
- none
- add reviewed-by tag
- add reviewed-by tag
- add reviewed-by tag
- properly restore slave_a->mode from platdata:
  <snip>
  -       slave_a->mode = slave_plat->max_hz;
  +       slave_a->mode = slave_plat->mode;

Ovidiu Panait (6):
  sandbox: spi: Drop unused sandbox_spi_parse_spec function
  sandbox: test: Add a second SPI slave on sandbox_spi bus
  spi: sandbox_spi: Implement speed/mode setup
  test: spi: Add sandbox_spi_get_{speed, mode} interface
  spi: spi-uclass: Fix spi_claim_bus() speed/mode setup logic
  test: dm: spi: Add testcase for spi_claim_bus()

 arch/sandbox/dts/test.dts       | 10 +++-
 arch/sandbox/include/asm/spi.h  | 10 ----
 arch/sandbox/include/asm/test.h | 16 +++++++
 drivers/mmc/mmc_spi.c           |  1 -
 drivers/spi/sandbox_spi.c       | 58 ++++++++++++++++-------
 drivers/spi/spi-uclass.c        | 17 +++++--
 include/spi.h                   | 18 +++++--
 test/dm/spi.c                   | 84 ++++++++++++++++++++++++++++++++-
 8 files changed, 175 insertions(+), 39 deletions(-)

-- 
2.17.1



More information about the U-Boot mailing list