From: Sudeep Holla <sudeep.holla@kernel.org>
To: Fayssal Benmlih <Fayssal.Benmlih@arm.com>
Cc: Cristian Marussi <Cristian.Marussi@arm.com>,
Sudeep Holla <sudeep.holla@kernel.org>,
"arm-scmi@vger.kernel.org" <arm-scmi@vger.kernel.org>,
"brauner@kernel.org" <brauner@kernel.org>,
"d-gole@ti.com" <d-gole@ti.com>,
"david@kernel.org" <david@kernel.org>,
Elif Topuz <Elif.Topuz@arm.com>,
"etienne.carriere@st.com" <etienne.carriere@st.com>,
"f.fainelli@gmail.com" <f.fainelli@gmail.com>,
"james.quinlan@broadcom.com" <james.quinlan@broadcom.com>,
"jic23@kernel.org" <jic23@kernel.org>,
"kas@kernel.org" <kas@kernel.org>,
"kernel-team@meta.com" <kernel-team@meta.com>,
"leitao@kernel.org" <leitao@kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Lukasz Luba <Lukasz.Luba@arm.com>,
"michal.simek@amd.com" <michal.simek@amd.com>,
"peng.fan@oss.nxp.com" <peng.fan@oss.nxp.com>,
Philip Radford <Philip.Radford@arm.com>,
"puranjay@kernel.org" <puranjay@kernel.org>,
Souvik Chakravarty <Souvik.Chakravarty@arm.com>,
"subrahmanya.lingappa@oss.qualcomm.com"
<subrahmanya.lingappa@oss.qualcomm.com>,
"usama.arif@linux.dev" <usama.arif@linux.dev>,
"vincent.guittot@linaro.org" <vincent.guittot@linaro.org>
Subject: Re: [PATCH v6 00/22] Introduce SCMI Telemetry support
Date: Mon, 27 Jul 2026 15:28:03 +0100 [thread overview]
Message-ID: <20260727-bouncy-slim-badger-bad63e@sudeepholla> (raw)
In-Reply-To: <DB9PR08MB86510C934D2EF25F625A5F38FFCC2@DB9PR08MB8651.eurprd08.prod.outlook.com>
Hi Fayssal,
On Mon, Jul 27, 2026 at 01:53:45PM +0000, Fayssal Benmlih wrote:
> Hi Cristian,
>
> Thanks for V6. It addresses a useful subset of the V5 feedback: the
> uninitialized feature flags, signed exponent, group bounds, bool casts,
> SHMTI enumeration loop, scoped ioctl magic, compat support, binary-only
> interface, ABI information, reset, and oversized mmap requests are fixed.
>
Appreciate your efforts in reviewing this series. But the below format
of dumping all the review comments in one place is neither the normal
mailing list practice nor much useful for other reviewer to follow as
one need to map/follow these when reading corresponding code which is
painful IMO.
You can look at other threads on the list for reference or we even have
an AI(sashiko) which does inline reviewing[1].
It will be really helpful if you can switch to that mode for future versions
or comments.
--
Regards,
Sudeep
[1] https://sashiko.dev/#/patchset/20260724144430.1440170-1-cristian.marussi@arm.com
next parent reply other threads:[~2026-07-27 14:28 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <DB9PR08MB86510C934D2EF25F625A5F38FFCC2@DB9PR08MB8651.eurprd08.prod.outlook.com>
2026-07-27 14:28 ` Sudeep Holla [this message]
2026-08-03 6:17 ` [PATCH v6 00/22] Introduce SCMI Telemetry support Cristian Marussi
2026-07-27 14:54 Fayssal Benmlih
-- strict thread matches above, loose matches on Subject: below --
2026-07-24 14:44 Cristian Marussi
2026-07-29 9:12 ` David Hildenbrand (Arm)
2026-07-29 16:36 ` Cristian Marussi
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=20260727-bouncy-slim-badger-bad63e@sudeepholla \
--to=sudeep.holla@kernel.org \
--cc=Cristian.Marussi@arm.com \
--cc=Elif.Topuz@arm.com \
--cc=Fayssal.Benmlih@arm.com \
--cc=Lukasz.Luba@arm.com \
--cc=Philip.Radford@arm.com \
--cc=Souvik.Chakravarty@arm.com \
--cc=arm-scmi@vger.kernel.org \
--cc=brauner@kernel.org \
--cc=d-gole@ti.com \
--cc=david@kernel.org \
--cc=etienne.carriere@st.com \
--cc=f.fainelli@gmail.com \
--cc=james.quinlan@broadcom.com \
--cc=jic23@kernel.org \
--cc=kas@kernel.org \
--cc=kernel-team@meta.com \
--cc=leitao@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michal.simek@amd.com \
--cc=peng.fan@oss.nxp.com \
--cc=puranjay@kernel.org \
--cc=subrahmanya.lingappa@oss.qualcomm.com \
--cc=usama.arif@linux.dev \
--cc=vincent.guittot@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox