qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 2/3] hw/ssi: Add Nuvoton PSPI Module


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v2 2/3] hw/ssi: Add Nuvoton PSPI Module
Date: Wed, 8 Feb 2023 08:48:14 +0100
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.6.1

On 7/2/23 20:45, Hao Wu wrote:
Nuvoton's PSPI is a general purpose SPI module which enables
connections to SPI-based peripheral devices.

Signed-off-by: Hao Wu <wuhaotsh@google.com>
Reviewed-by: Chris Rauer <crauer@google.com>
Reviewed-by: Philippe Mathieu-Daude <philmd@linaro.org>
---
  MAINTAINERS                |   6 +-
  hw/ssi/meson.build         |   2 +-
  hw/ssi/npcm_pspi.c         | 221 +++++++++++++++++++++++++++++++++++++
  hw/ssi/trace-events        |   5 +
  include/hw/ssi/npcm_pspi.h |  53 +++++++++
  5 files changed, 283 insertions(+), 4 deletions(-)
  create mode 100644 hw/ssi/npcm_pspi.c
  create mode 100644 include/hw/ssi/npcm_pspi.h


+static const MemoryRegionOps npcm_pspi_ctrl_ops = {
+    .read = npcm_pspi_ctrl_read,
+    .write = npcm_pspi_ctrl_write,
+    .endianness = DEVICE_LITTLE_ENDIAN,
+    .valid = {
+        .min_access_size = 1,
+        .max_access_size = 4,

You said in v1 "The datasheet suggests it's either 8-bit or
16-bit accesses.", so we want max_access_size = 2 here, right?

+        .unaligned = false,
+    },
+    .impl = {
+        .min_access_size = 2,
+        .max_access_size = 2,
+        .unaligned = false,
+    },
+};





reply via email to

[Prev in Thread] Current Thread [Next in Thread]