On 04.08.2025 16:33, Philippe Mathieu-Daudé wrote:
Since v1:
- new patch factoring sd_response_size() out (Richard)
This series fix a pair of issues with SD cards used wired
via a SPI link / controller.
Such mode implementation was minimal. I was testing it with
the ARM Gumstix machines, but we remove them in the v9.2.0
release (commit a2ccff4d2bc ), so they bit-rotted.
Although the series looks big, I shrinked it a lot to have
the minimum amount of meaningful changes. Other changes
added during debugging will be shared later, as I believe
they will still be useful to debug other future issues.
The last patch add testing coverage, to avoid further bitrot.
Regards,
Phil.
Philippe Mathieu-Daudé (11):
hw/sd/sdcard: Do not ignore errors in sd_cmd_to_sendingdata()
hw/sd/sdcard: Factor sd_response_size() out
hw/sd/sdbus: Provide buffer size to sdbus_do_command()
hw/sd/sdcard: Fill SPI response bits in card code
hw/sd/sdcard: Implement SPI R2 return value
hw/sd/sdcard: Use complete SEND_OP_COND implementation in SPI mode
hw/sd/sdcard: Allow using SWITCH_FUNCTION in more SPI states
hw/sd/sdcard: Factor spi_cmd_SEND_CxD() out
hw/sd/sdcard: Disable checking STBY mode in SPI SEND_CSD/CID
hw/sd/sdcard: Remove SDState::mode field
tests/functional: Test SD cards in SPI mode (using sifive_u machine)
Hi!
Philippe, do you think this series is something which should
go to stable-10.0 (LTS) branch? I'm not sure for the impact if
it is not applied, though, - what do we miss in this case?
Thanks,
/mjt
