Since v2: - split again in 2... this part is cleanup/tracing - add more tracepoints - move some code reusable by sdbus in sdmmc-internal.h
Since v1: - rewrote mostly all patches to keep it simpler. $ git backport-diff 001/12:[0007] [FC] 'sdcard: reorder SDState struct members' 002/12:[0002] [FC] 'sdcard: replace DPRINTF() by trace events' 003/12:[down] 'sdcard: add a trace event for command responses' 004/12:[down] 'sdcard: replace fprintf() by qemu_hexdump()' 005/12:[0035] [FC] 'sdcard: add more trace events' 006/12:[down] 'sdcard: do not trace CMD55 when expecting ACMD' 007/12:[down] 'sdcard: define SDMMC_CMD_MAX instead of using the magic '64'' 008/12:[0083] [FC] 'sdcard: display command name when tracing CMD/ACMD' 009/12:[down] 'sdcard: display protocol used when tracing' 010/12:[----] [--] 'sdcard: rename sd_set_$REG() functions called once as sd_reset_$REG()' 011/12:[0002] [FC] 'sdcard: use G_BYTE from cutils' 012/12:[0008] [FC] 'sdcard: use the registerfields API to access the OCR register' Based-on: 20180123030630.26613-15-f4...@amsat.org Philippe Mathieu-Daudé (12): sdcard: reorder SDState struct members sdcard: replace DPRINTF() by trace events sdcard: add a trace event for command responses sdcard: replace fprintf() by qemu_hexdump() sdcard: add more trace events sdcard: do not trace CMD55 when expecting ACMD sdcard: define SDMMC_CMD_MAX instead of using the magic '64' sdcard: display command name when tracing CMD/ACMD sdcard: display protocol used when tracing sdcard: rename sd_set_$REG() functions called once as sd_reset_$REG() sdcard: use G_BYTE from cutils sdcard: use the registerfields API to access the OCR register include/hw/sd/sd.h | 1 - hw/sd/sdmmc-internal.h | 20 +++++ hw/sd/sd.c | 212 ++++++++++++++++++++++++++++++++++--------------- hw/sd/sdmmc-common.c | 72 +++++++++++++++++ hw/sd/Makefile.objs | 2 +- hw/sd/trace-events | 20 +++++ 6 files changed, 259 insertions(+), 68 deletions(-) create mode 100644 hw/sd/sdmmc-internal.h create mode 100644 hw/sd/sdmmc-common.c -- 2.15.1
