public inbox for qemu-devel@nongnu.org
 help / color / mirror / Atom feed
From: Joel Stanley <joel@jms.id.au>
To: Peter Maydell <peter.maydell@linaro.org>, alanosong@163.com
Cc: qemu-devel@nongnu.org,
	Alistair Francis <alistair.francis@wdc.com>,
	 Daniel Henrique Barboza <dbarboza@ventanamicro.com>,
	Chris Rauer <crauer@google.com>,
	 Paolo Bonzini <pbonzini@redhat.com>,
	qemu-riscv@nongnu.org, Hao Wu <wuhaotsh@google.com>
Subject: Re: [PATCH] hw/i2c: Add designware i2c controller
Date: Wed, 11 Mar 2026 14:27:42 +1030	[thread overview]
Message-ID: <CACPK8XdK-+PYsek3JFx3cn6J1DjOaDGWJod_j4Mj__Uut7xyLw@mail.gmail.com> (raw)
In-Reply-To: <CAFEAcA9JW+1f+mRKXtFoFwGUKzBM0mypjMFc5ToukRyKqCWy0Q@mail.gmail.com>

Hello Alano and Peter,

On Fri, 16 Jan 2026 at 03:17, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Fri, 9 Jan 2026 at 12:33, Joel Stanley <joel@jms.id.au> wrote:
> >
> > From: Chris Rauer <crauer@google.com>
> >
> > Reviewed-by: Hao Wu <wuhaotsh@google.com>
> > Signed-off-by: Chris Rauer <crauer@google.com>
> > Link: https://lore.kernel.org/qemu-devel/20220110214755.810343-2-venture@google.com
> > [jms: rebase and minor build fixes for class_init and reset callback]
> > Signed-off-by: Joel Stanley <joel@jms.id.au>
> > ---
> > This is a re-submission of the model with Chris' permission, with a
> > light touch of updates to make it build with qemu master.
> >
> > It will be used by the Tenstorrent Atlantis machine, which will include
> > a functional test for i2c devices attached to the controller.
> >
> > Daniel suggested that Alistair would take it through the riscv tree as
> > a (soft) pre-req for the Atlantis machine.
>
> Hi -- what relation does this model have to the
> patch posted by Alano Song a few days earlier?
>  https://lore.kernel.org/qemu-devel/20260106131253.16192-1-AlanoSong@163.com/
> Are these independent re-implementations? Do we
> have any particular reason to prefer one over the other?

A race condition I think. I grabbed the patch from the list a few
months back, but didn't post my series until January. It seems Alano
posted this in the meantime.

Alano, is there anything missing from this version that your model
supports? Are you okay with using this version?

Cheers,

Joel


  reply	other threads:[~2026-03-11  3:58 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-09 12:32 [PATCH] hw/i2c: Add designware i2c controller Joel Stanley
2026-01-15 16:46 ` Peter Maydell
2026-03-11  3:57   ` Joel Stanley [this message]
2026-03-22  9:23 ` Alano Song

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=CACPK8XdK-+PYsek3JFx3cn6J1DjOaDGWJod_j4Mj__Uut7xyLw@mail.gmail.com \
    --to=joel@jms.id.au \
    --cc=alanosong@163.com \
    --cc=alistair.francis@wdc.com \
    --cc=crauer@google.com \
    --cc=dbarboza@ventanamicro.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=wuhaotsh@google.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