[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v7 00/14] SDHCI: housekeeping (part 1)
From: |
Philippe Mathieu-Daudé |
Subject: |
[Qemu-devel] [PATCH v7 00/14] SDHCI: housekeeping (part 1) |
Date: |
Sat, 13 Jan 2018 02:07:03 -0300 |
Since v6:
- addressed Peter reviews
- do not use an unique Property[] for both sysbus/pci
Peter didn't recommend me to use the qdev_property_add_static() API since
it is only used by the ARM cpus and may be due for removal, however I found
it cleaner.
At the end of those series, the only differences is the sysbus having 2
more properties: "pending-insert-quirk" and "dma", all other are generic
to SDHCI and shared.
- use the PCI AS for DMA.
Should we check for the PCI_COMMAND_MASTER bit?
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/14:[----] [--] 'sdhci: clean up includes'
002/14:[----] [--] 'sdhci: remove dead code'
003/14:[0037] [FC] 'sdhci: refactor same sysbus/pci properties into a common
one'
004/14:[0003] [FC] 'sdhci: refactor common sysbus/pci class_init() into
sdhci_common_class_init()'
005/14:[----] [-C] 'sdhci: refactor common sysbus/pci realize() into
sdhci_common_realize()'
006/14:[0003] [FC] 'sdhci: refactor common sysbus/pci unrealize() into
sdhci_common_unrealize()'
007/14:[----] [--] 'sdhci: use qemu_log_mask(UNIMP) instead of fprintf()'
008/14:[----] [--] 'sdhci: convert the DPRINT() calls into trace events'
009/14:[----] [--] 'sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in
"sd-internal.h"'
010/14:[----] [--] 'sdhci: rename the SDHC_CAPAB register'
011/14:[0008] [FC] 'sdhci: fix CAPAB/MAXCURR registers, both are 64bit and
read-only'
012/14:[----] [--] 'sdhci: Implement write method of ACMD12ERRSTS register'
013/14:[down] 'sdhci: fix the PCI device, using the PCI address space for DMA'
014/14:[0046] [FC] 'sdhci: add a 'dma' property to the sysbus devices'
Andrey Smirnov (1):
sdhci: Implement write method of ACMD12ERRSTS register
Philippe Mathieu-Daudé (13):
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: fix the PCI device, using the PCI address space for DMA
sdhci: add a 'dma' property to the sysbus devices
include/hw/sd/sdhci.h | 19 +++-
hw/sd/sdhci-internal.h | 7 +-
hw/sd/sdhci.c | 291 +++++++++++++++++++++++++++++--------------------
hw/sd/trace-events | 14 +++
4 files changed, 204 insertions(+), 127 deletions(-)
--
2.15.1
- [Qemu-devel] [PATCH v7 00/14] SDHCI: housekeeping (part 1),
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v7 01/14] sdhci: clean up includes, Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 02/14] sdhci: remove dead code, Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 04/14] sdhci: refactor common sysbus/pci class_init() into sdhci_common_class_init(), Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 03/14] sdhci: refactor same sysbus/pci properties into a common one, Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 05/14] sdhci: refactor common sysbus/pci realize() into sdhci_common_realize(), Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 07/14] sdhci: use qemu_log_mask(UNIMP) instead of fprintf(), Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 06/14] sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize(), Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 08/14] sdhci: convert the DPRINT() calls into trace events, Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 09/14] sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in "sd-internal.h", Philippe Mathieu-Daudé, 2018/01/13
- [Qemu-devel] [PATCH v7 10/14] sdhci: rename the SDHC_CAPAB register, Philippe Mathieu-Daudé, 2018/01/13