[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 00/13] SDHCI: housekeeping
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v6 00/13] SDHCI: housekeeping |
Date: |
Thu, 11 Jan 2018 13:11:02 -0800 |
On Thu, Jan 11, 2018 at 11:30 AM, Philippe Mathieu-Daudé
<address@hidden> wrote:
> All patches reviewed :)
Do you think this can go through you Peter?
Alistair
>
> 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
>
>
- [Qemu-devel] [PATCH v6 06/13] sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize(), (continued)
- [Qemu-devel] [PATCH v6 06/13] sdhci: refactor common sysbus/pci unrealize() into sdhci_common_unrealize(), Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 07/13] sdhci: use qemu_log_mask(UNIMP) instead of fprintf(), Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 08/13] sdhci: convert the DPRINT() calls into trace events, Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 09/13] sdhci: move MASK_TRNMOD with other SDHC_TRN* defines in "sd-internal.h", Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 10/13] sdhci: rename the SDHC_CAPAB register, Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 11/13] sdhci: fix CAPAB/MAXCURR registers, both are 64bit and read-only, Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 12/13] sdhci: Implement write method of ACMD12ERRSTS register, Philippe Mathieu-Daudé, 2018/01/11
- [Qemu-devel] [PATCH v6 13/13] sdhci: add a "dma" property, Philippe Mathieu-Daudé, 2018/01/11
- Re: [Qemu-devel] [PATCH v6 00/13] SDHCI: housekeeping,
Alistair Francis <=