[v3] hw/riscv/k230: add IOMUX register block model

[PATCH v3 0/3] hw/riscv/k230: add IOMUX register block model

Kangjie Huang posted 3 patches 1 week, 6 days ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260813135717.2758-1-flamboyant.h.01@gmail.com

Maintainers: Chao Liu <chao.liu@processmission.com>, Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>, Paolo Bonzini <pbonzini@redhat.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Fabiano Rosas <farosas@suse.de>, Laurent Vivier <lvivier@redhat.com>

MAINTAINERS                   |   3 +
docs/system/riscv/k230.rst    |   1 +
hw/misc/Kconfig               |   3 +
hw/misc/k230_iomux.c          | 125 ++++++++++++++++++++++++++
hw/misc/meson.build           |   1 +
hw/misc/trace-events          |   4 +
hw/riscv/Kconfig              |   1 +
hw/riscv/k230.c               |  11 ++-
include/hw/misc/k230_iomux.h  |  43 +++++++++
include/hw/riscv/k230.h       |   2 +
tests/qtest/k230-iomux-test.c | 160 ++++++++++++++++++++++++++++++++++
tests/qtest/meson.build       |   5 +-
12 files changed, 355 insertions(+), 4 deletions(-)
create mode 100644 hw/misc/k230_iomux.c
create mode 100644 include/hw/misc/k230_iomux.h
create mode 100644 tests/qtest/k230-iomux-test.c

Expand all Fold all

[PATCH v3 0/3] hw/riscv/k230: add IOMUX register block model

Posted by Kangjie Huang 1 week, 6 days ago

Add a SysBus model for the 64 K230 Function IO configuration registers
documented by the K230 Technical Reference Manual. The model retains the
writable configuration fields needed by the SDK U-Boot and Linux
read-modify-write paths.

The series adds the device model, wires it into the K230 SoC at
0x91105000, and adds qtest coverage.

Changes in v3:
- Migrated the model from the deprecated device_class_set_legacy_reset()
  to the Resettable interface.
- Added a comment to the model stating that bit 31 (DI) is read-only and
  reads as zero, since the model does not simulate external pin input
  (TRM V0.3.1, section 12.9.2.2).
- Added MAINTAINERS entries for the model and the qtest.
- Defined an independent K230_IOMUX_EXPECTED_WRITABLE_MASK (0x00003fff)
  in the test instead of reusing the implementation mask, with a comment
  explaining the bit layout.

Changes in v2:
- Split the device model, SoC wiring, and qtest coverage into separate
  patches.
- Removed redundant MMIO bounds, access-size, and alignment checks.
- Limited register storage to the 64 documented Function IO registers.
- Applied the documented 0x00003fff writable mask.
- Treated offsets 0x100 through 0x7ff as reserved.
- Added tests for write masking, read-modify-write accesses, IO63,
  reserved offsets, and system reset.

Tested with:
- qtest-riscv64/k230-iomux-test
- U-Boot and Linux boot tests with no unimplemented-device warnings
  for the IOMUX region

Thanks to Alistair Francis and Chao Liu for the v1 review comments that
shaped the v2 series structure and the register semantics.

Reviewed-by: Junze Cao <caojunze424@gmail.com>

v2:
https://lore.kernel.org/qemu-devel/cover.1784022244.git.flamboyant.h.01@gmail.com/

v1:
https://lore.kernel.org/qemu-devel/20260710041845.67170-1-flamboyant.h.01@gmail.com/

Kangjie Huang (3):
  hw/misc/k230_iomux: add Kendryte K230 IOMUX model
  hw/riscv/k230: wire up the IOMUX device
  tests/qtest: add test for K230 IOMUX

 MAINTAINERS                   |   3 +
 docs/system/riscv/k230.rst    |   1 +
 hw/misc/Kconfig               |   3 +
 hw/misc/k230_iomux.c          | 125 ++++++++++++++++++++++++++
 hw/misc/meson.build           |   1 +
 hw/misc/trace-events          |   4 +
 hw/riscv/Kconfig              |   1 +
 hw/riscv/k230.c               |  11 ++-
 include/hw/misc/k230_iomux.h  |  43 +++++++++
 include/hw/riscv/k230.h       |   2 +
 tests/qtest/k230-iomux-test.c | 160 ++++++++++++++++++++++++++++++++++
 tests/qtest/meson.build       |   5 +-
 12 files changed, 355 insertions(+), 4 deletions(-)
 create mode 100644 hw/misc/k230_iomux.c
 create mode 100644 include/hw/misc/k230_iomux.h
 create mode 100644 tests/qtest/k230-iomux-test.c


base-commit: d49f87606ac1a6e15701b26c1b16c5d7e948ffcb
-- 
2.43.0