[Qemu-devel] [PATCH v6 00/13] SDHCI: housekeeping

["Philippe Mathieu-Daudé" <f4bug@amsat.org>]

All patches reviewed :)

Since v5:
- addressed Alistair reviews
- added Alistair R-b
- renamed the dma property "dma-memory" -> "dma"

Since v4:
- fixed incorrect use of &local_err in sdhci_sysbus/pci_realize()

Since v3:
- since the series was getting too big and first part reviewed, split in 2. 
- addressed Fam's review from "refactor the common sysbus/pci qdev"
  - improved commit descriptions
  - restored useful s->fifo_buffer = NULL
- added Alistair R-b

Since v2:
- more detailed 'capabilities', all boards converted to use these properties
- since all qtests pass, removed the previous 'capareg' property
- added Stefan/Alistair R-b
- corrected 'access' LED behavior (Alistair's review)
- more uses of the registerfields API
- remove some dead code
- cosmetix:
  - added more comments
  - renamed a pair of registers
  - reordered few struct members

Since v1:
- addressed Alistair Francis review comments, added some R-b
- only move register defines to "sd-internal.h"
- fixed deposit64() arguments
- dropped unuseful s->fifo_buffer = NULL
- use a qemu_irq for the LED, restrict the logging to ON/OFF
- fixed a trace format string error
- included Andrey Smirnov ACMD12ERRSTS write patch
- dropped few unuseful patches, and separate the Python polemical ones for later

>From the "SDHCI housekeeping" series:
- 1: we restrict part of "sd/sd.h" into local "sd-internal.h",
- 2,3: we somehow beautiful the code, no logical changes,
- 4-7: we refactor the common sysbus/pci qdev code,
- 8-10: we add plenty of trace events which will result useful later,
- 11: we finally expose a "dma-memory" property.
>From the "SDHCI: add a qtest and fix few issues" series:
- 12,13: fix registers
- 14,15: boards can specify which SDHCI Spec to use (v2 and v3 so far)
- 15-20: HCI qtest

Regards,

Phil.

$ git backport-diff
001/13:[----] [--] 'sdhci: clean up includes'
002/13:[----] [--] 'sdhci: remove dead code'
003/13:[0004] [FC] 'sdhci: refactor same sysbus/pci properties into a common one'
004/13:[----] [--] 'sdhci: refactor common sysbus/pci class_init() into sdhci_common_class_init()'
005/13:[----] [--] 'sdhci: refactor common sysbus/pci realize() into sdhci_common_realize()'
006/13:[----] [--] 'sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize()'
007/13:[----] [--] 'sdhci: use qemu_log_mask(UNIMP) instead of fprintf()'
008/13:[0020] [FC] 'sdhci: convert the DPRINT() calls into trace events'
009/13:[----] [--] 'sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in "sd-internal.h"'
010/13:[----] [-C] 'sdhci: rename the SDHC_CAPAB register'
011/13:[0002] [FC] 'sdhci: fix CAPAB/MAXCURR registers, both are 64bit and read-only'
012/13:[----] [--] 'sdhci: Implement write method of ACMD12ERRSTS register'
013/13:[0004] [FC] 'sdhci: add a "dma" property'

Andrey Smirnov (1):
  sdhci: Implement write method of ACMD12ERRSTS register

Philippe Mathieu-Daudé (12):
  sdhci: clean up includes
  sdhci: remove dead code
  sdhci: refactor same sysbus/pci properties into a common one
  sdhci: refactor common sysbus/pci class_init() into sdhci_common_class_init()
  sdhci: refactor common sysbus/pci realize() into sdhci_common_realize()
  sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize()
  sdhci: use qemu_log_mask(UNIMP) instead of fprintf()
  sdhci: convert the DPRINT() calls into trace events
  sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in "sd-internal.h"
  sdhci: rename the SDHC_CAPAB register
  sdhci: fix CAPAB/MAXCURR registers, both are 64bit and read-only
  sdhci: add a "dma" property

 include/hw/sd/sdhci.h  |  19 ++--
 hw/sd/sdhci-internal.h |   7 +-
 hw/sd/sdhci.c          | 260 ++++++++++++++++++++++++++++---------------------
 hw/sd/trace-events     |  14 +++
 4 files changed, 176 insertions(+), 124 deletions(-)

-- 
2.15.1