On Sun, May 11, 2025 at 09:27:33PM -0400, Damien Riégel wrote:

Hi,

This patchset brings initial support for Silicon Labs CPC protocol, standing for Co-Processor Communication. This protocol is used by the EFR32 Series [1]. These devices offer a variety for radio protocols, such as Bluetooth, Z-Wave, Zigbee [2].

Before we get too deep into the details of the patches, please could you do a compare/contrast to Greybus.

The core of Greybus is already in the kernel, with some more bits being in staging. I don't know it too well, but at first glance it looks very similar. We should not duplicate that.

Also, this patch adds Bluetooth, you talk about Z-Wave and Zigbee. But the EFR32 is a general purpose SoC, with I2C, SPI, PWM, UART. Greybus has support for these, although the code is current in staging. But for staging code, it is actually pretty good.

Why should we add a vendor implementation when we already appear to have something which does most of what is needed?

Andrew