Hi,
Any more reviews...?

Dear Stefan
If you don't mind, Could you give it "reviewed-by"?
And is there anything else I should do...?

Thanks,
Jeuk

On 2023-07-04 오후 5:33, Jeuk Kim wrote:
From: Jeuk Kim <jeuk20....@samsung.com>

Since v3:
- Replace softmmu_ss -> system_ss in meson

Since v2:
Addressed review comment from Stefan Hajnoczi. The main fixes are as
follows.
- Use of SPDX licence identifiers
- fixed endianness error
- removed memory leak
- fixed DMA error handling logic

Since v1:
- use macros of "hw/registerfields.h" (Addressed Philippe's review
   comments)

This patch series adds support for a new PCI-based UFS device.

The UFS pci device id (PCI_DEVICE_ID_REDHAT_UFS) is not registered
in the Linux kernel yet, so it does not work right away, but I confirmed
that it works with Linux when the UFS pci device id is registered.

I have also verified that it works with Windows 10.

Jeuk Kim (3):
   hw/ufs: Initial commit for emulated Universal-Flash-Storage
   hw/ufs: Support for Query Transfer Requests
   hw/ufs: Support for UFS logical unit

  MAINTAINERS              |    6 +
  docs/specs/pci-ids.rst   |    2 +
  hw/Kconfig               |    1 +
  hw/meson.build           |    1 +
  hw/ufs/Kconfig           |    4 +
  hw/ufs/lu.c              | 1441 +++++++++++++++++++++++++++++++++++
  hw/ufs/meson.build       |    1 +
  hw/ufs/trace-events      |   59 ++
  hw/ufs/trace.h           |    1 +
  hw/ufs/ufs.c             | 1545 ++++++++++++++++++++++++++++++++++++++
  hw/ufs/ufs.h             |  131 ++++
  include/block/ufs.h      | 1048 ++++++++++++++++++++++++++
  include/hw/pci/pci.h     |    1 +
  include/hw/pci/pci_ids.h |    1 +
  include/scsi/constants.h |    1 +
  meson.build              |    1 +
  16 files changed, 4244 insertions(+)
  create mode 100644 hw/ufs/Kconfig
  create mode 100644 hw/ufs/lu.c
  create mode 100644 hw/ufs/meson.build
  create mode 100644 hw/ufs/trace-events
  create mode 100644 hw/ufs/trace.h
  create mode 100644 hw/ufs/ufs.c
  create mode 100644 hw/ufs/ufs.h
  create mode 100644 include/block/ufs.h



Reply via email to