From: "Damien Riégel" <damien.riegel@silabs.com>
To: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
Cc: <greybus-dev@lists.linaro.org>, "Johan Hovold" <johan@kernel.org>,
"Alex Elder" <elder@kernel.org>, <linux-kernel@vger.kernel.org>,
"Silicon Labs Kernel Team" <linux-devel@silabs.com>
Subject: Re: [RFC PATCH v2 00/12] greybus: introduce CPC as transport layer
Date: Mon, 01 Dec 2025 10:39:50 -0500 [thread overview]
Message-ID: <DEMZRMIAGULC.3R4S53KVQRCH8@silabs.com> (raw)
In-Reply-To: <2025112610-hankering-muscular-d120@gregkh>
On Wed Nov 26, 2025 at 7:33 AM EST, Greg Kroah-Hartman wrote:
> On Tue, Nov 25, 2025 at 10:26:36AM -0500, Damien Riégel wrote:
>> Hi Greg, Johan, and Alex,
>>
>> On Fri Nov 14, 2025 at 10:07 AM EST, Damien Riégel wrote:
>> > Hi,
>> >
>> > This patchset brings support for Silicon Labs' CPC (Co-Processor
>> > Communication) protocol as transport layer for Greybus. This is
>> > introduced as a module that sits between Greybus and CPC Host Device
>> > Driver implementations, like SDIO or SPI, which are not part of this
>> > RFC. If there's no push back with this RFC, the final patchset ready for
>> > upstream will include the SDIO driver.
>>
>> Gentle poke about this RFC, I would really appreciate any kind of
>> feedback on it.
>
> Given my workload, I don't respond to "RFC" as obviously it's not ready
> for the submitter to feel that it should be applied yet :)
Noted, I'll send a non-RFC version of the patchset very soon!
>> If it's too big I can try to make it smaller to make the review easier.
>> As we're committing to Greybus to enable the coexistence of different
>> radio stacks in one chip, we want to make sure we're heading in the
>> right direction and that our work has a chance to get upstreamed.
>
> Always make review easier for us, so yes, please make it smaller!
I might have overpromised on this one :( I've already tried to limit the
scope of the patchset to be as small as possible while still being
meaningful and my goal was to add the SDIO driver to give the full
picture of what we're trying to achieve. I'll see if I can cut it down
even further.
Side note for list administrators: I can send patchset just fine with
git-send-email but posting to the list with my email client (aerc) gives
me every single time:
The message is being held because:
Message has implicit destination
I'm subscribed to the list and my emails seem to make it in the end (I
don't know if they are manually released or not). I looked up that error
message and I didn't find anything useful. Am I the only one having that
issue? Would be happy to find a solution.
Thank you,
--
Damien
prev parent reply other threads:[~2025-12-01 15:40 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-14 15:07 [RFC PATCH v2 00/12] greybus: introduce CPC as transport layer Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 01/12] greybus: cpc: add minimal CPC Host Device infrastructure Damien Riégel
2025-11-14 16:44 ` Yacin Belmihoub-Martel
2025-11-14 16:51 ` Yacin Belmihoub-Martel
2025-11-14 16:55 ` Yacin Belmihoub-Martel
2025-11-14 15:07 ` [RFC PATCH v2 02/12] greybus: cpc: introduce CPC cport structure Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 03/12] greybus: cpc: use socket buffers instead of gb_message in TX path Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 04/12] greybus: cpc: pack cport ID in Greybus header Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 05/12] greybus: cpc: switch RX path to socket buffers Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 06/12] greybus: cpc: introduce CPC header structure Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 07/12] greybus: cpc: account for CPC header size in RX and TX path Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 08/12] greybus: cpc: add and validate sequence numbers Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 09/12] greybus: cpc: acknowledge all incoming messages Damien Riégel
2025-11-14 15:07 ` [RFC PATCH v2 10/12] greybus: cpc: use holding queue instead of sending out immediately Damien Riégel
2025-11-14 19:53 ` Yacin Belmihoub-Martel
2025-11-14 15:07 ` [RFC PATCH v2 11/12] greybus: cpc: honour remote's RX window Damien Riégel
2025-11-14 20:28 ` Yacin Belmihoub-Martel
2025-11-14 15:07 ` [RFC PATCH v2 12/12] greybus: cpc: let host device drivers dequeue TX frames Damien Riégel
2025-11-25 15:26 ` [RFC PATCH v2 00/12] greybus: introduce CPC as transport layer Damien Riégel
2025-11-26 12:33 ` Greg Kroah-Hartman
2025-12-01 15:39 ` Damien Riégel [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=DEMZRMIAGULC.3R4S53KVQRCH8@silabs.com \
--to=damien.riegel@silabs.com \
--cc=elder@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=greybus-dev@lists.linaro.org \
--cc=johan@kernel.org \
--cc=linux-devel@silabs.com \
--cc=linux-kernel@vger.kernel.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