All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Golle <daniel@makrotopia.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Matthew Jackson" <matthew@pq.io>,
	"Alexander Graf" <agraf@csgraf.de>,
	"Philippe Mathieu-Daudé" <philmd@oss.qualcomm.com>,
	"Pierrick Bouvier" <pierrick.bouvier@oss.qualcomm.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Cédric Le Goater" <clg@redhat.com>,
	Kane-Chen-AS <kane_chen@aspeedtech.com>,
	"jack wang" <163wangjack@gmail.com>,
	qemu-devel@nongnu.org
Subject: Re: [PATCH 0/4] hw/misc/applesmc: Implement writes, the shutdown watchdog and get-key-type
Date: Wed, 19 Aug 2026 18:34:01 +0100	[thread overview]
Message-ID: <20260819144000.a1b2c3@makrotopia.org> (raw)
In-Reply-To: <CAFEAcA-kkKrduX6r85vLTFixRZ8pgvXpZDjR_-DTSz0h_+kDOg@mail.gmail.com>

On Wed, 19 Aug 2026 at 14:29, Peter Maydell <peter.maydell@linaro.org> wrote:
> How does this series fit in with Matthew Jackson's set of patches
> https://patchew.org/QEMU/20260720164930.68383-1-matthew@pq.io/

They do overlap, and Matthew's is the more complete of the two, so I
am dropping the overlapping parts of mine in favour of his.

We independently implemented the same two missing SMC commands (write
and get-key-type). On top of that Matthew's v4 also implements
get-key-by-index and populates a full key table, whereas my series
left get-key-by-index out (the macOS 26 guest I traced doesn't seem to
emitted them). So his series supersedes patches 2 and 4 of mine.

The only parts of my series not covered by his are the SMC shutdown
watchdog and the trace events. I have rebased those two patches on top
of his v4 and will send them as a short follow-up ("[PATCH 0/2]
hw/misc/ applesmc: SMC shutdown watchdog and trace events").
Reviewed-by/Tested-by for his series are in my replies to it.


Cheers and thanks for connecting the two threads


Daniel


      reply	other threads:[~2026-08-19 17:34 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-19  6:41 [PATCH 0/4] hw/misc/applesmc: Implement writes, the shutdown watchdog and get-key-type Daniel Golle
2026-08-19  6:41 ` [PATCH 1/4] hw/misc/applesmc: Add trace events Daniel Golle
2026-08-19  6:41 ` [PATCH 2/4] hw/misc/applesmc: Implement the write command Daniel Golle
2026-08-19  6:41 ` [PATCH 3/4] hw/misc/applesmc: Emulate the SMC shutdown watchdog Daniel Golle
2026-08-19  6:41 ` [PATCH 4/4] hw/misc/applesmc: Implement the get-key-type command Daniel Golle
2026-08-19 13:29 ` [PATCH 0/4] hw/misc/applesmc: Implement writes, the shutdown watchdog and get-key-type Peter Maydell
2026-08-19 17:34   ` Daniel Golle [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=20260819144000.a1b2c3@makrotopia.org \
    --to=daniel@makrotopia.org \
    --cc=163wangjack@gmail.com \
    --cc=agraf@csgraf.de \
    --cc=clg@redhat.com \
    --cc=kane_chen@aspeedtech.com \
    --cc=matthew@pq.io \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@oss.qualcomm.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.