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
next prev parent 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