QEMU-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Kuan-Wei Chiu <visitorckw@gmail.com>
To: Joel Stanley <joel@jms.id.au>
Cc: "WX Chen" <wxchen0913@gmail.com>,
	"Alistair Francis" <alistair.francis@wdc.com>,
	qemu-devel@nongnu.org, "Paolo Bonzini" <pbonzini@redhat.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Chao Liu" <chao.liu.zevorn@gmail.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Weiwei Li" <liwei1518@gmail.com>,
	"Daniel Henrique Barboza" <daniel.barboza@oss.qualcomm.com>,
	"Liu Zhiwei" <zhiwei_liu@linux.alibaba.com>,
	qemu-riscv@nongnu.org, "Fabiano Rosas" <farosas@suse.de>,
	"Laurent Vivier" <lvivier@redhat.com>
Subject: Re: [PATCH v3 0/3] riscv: Add K230 DW 8250-compatible UART
Date: Fri, 14 Aug 2026 19:37:16 +0800	[thread overview]
Message-ID: <an797N3cUZj5EX0h@google.com> (raw)
In-Reply-To: <CACPK8Xfgj+AUx0h_46f=qG_UEE_o7kTZ0=Buxr7+bdHDODHpgw@mail.gmail.com>

On Thu, Aug 13, 2026 at 07:50:18PM +0930, Joel Stanley wrote:
> On Sat, 8 Aug 2026 at 12:02, WX Chen <wxchen0913@gmail.com> wrote:
> >
> > This series adds a QEMU model for the K230 SoC DesignWare 8250-compatible
> > UART controller, wires it into the K230 SoC, and adds qtest coverage.
> >
> > The implementation supports the registers and features required by the
> > Linux 8250_dw driver, providing an interactive shell over the serial port.
> 
> This is a model for the same hardware as this patch:
> 
> https://lore.kernel.org/qemu-riscv/20260616190147.1286316-2-visitorckw@gmail.com/
> 
> I'm not fussed which one we end up with in the tree, but it would be
> good to name it generically so other machines can use it.
> 
> 
Ah, I've been a bit busy lately and totally forgot to send out the next
version of the milk-v duo patchset. I will probably make the changes
and send it out soon.

As for the dw 8250 uart part, I'm not sure if I should keep my uart
patch in my series, or if I should wait for WX to respin his patchset
and get it merged into the riscv tree, and then rebase my patchset on
top of it?

Regards,
Kuan-Wei

> >
> > Signed-off-by: WX Chen <wxchen0913@gmail.com>
> > ---
> > Changes in v3:
> > - Removed all hardwired preprocessor #if conditionals from the logic
> > - Added a helper for the receiver line status interrupt check
> > - Encapsulated CPR and UCV register values in header macros
> > - Replaced g_usleep() with qtest_clock_step() in the qtest
> > - Link to v2: https://lore.kernel.org/qemu-devel/20260725-feat-k230-uart-v2-v2-0-d5fe82c47c28@gmail.com
> >
> > ---
> > WX Chen (3):
> >       hw/char: add K230 DW 8250-compatible UART
> >       hw/riscv: k230: connect DW 8250 UART
> >       tests/qtest: add K230 UART test
> >
> >  hw/char/Kconfig              |   3 +
> >  hw/char/k230_uart.c          | 816 +++++++++++++++++++++++++++++++++++++++++++
> >  hw/char/meson.build          |   1 +
> >  hw/riscv/k230.c              |  33 +-
> >  include/hw/char/k230_uart.h  | 198 +++++++++++
> >  include/hw/riscv/k230.h      |   6 +-
> >  tests/qtest/k230-uart-test.c | 514 +++++++++++++++++++++++++++
> >  tests/qtest/meson.build      |   2 +-
> >  8 files changed, 1559 insertions(+), 14 deletions(-)
> > ---
> > base-commit: 30e8a06b64aa58a3990ba39cb5d09531e7d265e0
> > change-id: 20260808-feat-k230-uart-v3-0ff3651d70ef
> >
> > Best regards,
> > --
> > WX Chen <wxchen0913@gmail.com>
> >
> >


      parent reply	other threads:[~2026-08-14 11:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-08  2:32 [PATCH v3 0/3] riscv: Add K230 DW 8250-compatible UART WX Chen
2026-08-08  2:32 ` [PATCH v3 1/3] hw/char: add " WX Chen
2026-08-08 11:35   ` Daniel Henrique Barboza
2026-08-08  2:32 ` [PATCH v3 2/3] hw/riscv: k230: connect DW 8250 UART WX Chen
2026-08-08  2:32 ` [PATCH v3 3/3] tests/qtest: add K230 UART test WX Chen
2026-08-08 11:35   ` Daniel Henrique Barboza
2026-08-11 19:12 ` [PATCH v3 0/3] riscv: Add K230 DW 8250-compatible UART Alistair
2026-08-13 10:20 ` Joel Stanley
2026-08-14  6:27   ` Bin Meng
2026-08-14 15:52     ` Alistair
2026-08-15  2:57       ` WX Chen
2026-08-14 11:37   ` Kuan-Wei Chiu [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=an797N3cUZj5EX0h@google.com \
    --to=visitorckw@gmail.com \
    --cc=alistair.francis@wdc.com \
    --cc=chao.liu.zevorn@gmail.com \
    --cc=daniel.barboza@oss.qualcomm.com \
    --cc=farosas@suse.de \
    --cc=joel@jms.id.au \
    --cc=liwei1518@gmail.com \
    --cc=lvivier@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=palmer@dabbelt.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=wxchen0913@gmail.com \
    --cc=zhiwei_liu@linux.alibaba.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox