Re: [PATCH v1 0/6] Add SCMI transport descriptors

From: Nikunj Kela
Date: Thu Aug 01 2024 - 15:18:19 EST



On 7/30/2024 7:47 AM, Cristian Marussi wrote:
> Hi,
>
> this small series is an extended version of this recent, already reviewed,
> series [1] posted by Peng to add a new arm,scmi property to describe some
> platform-specific SCMI timeout constraints.
>
> On top of that, this adds 2 more properties to describe a couple more
> platform-specific transport characteristics.
>
> To minimize conflicts, the whole series is based on top of another recent
> series, which represents a rework of the core SCMI stack to split SCMI
> transports as standalone drivers. [2]
>
> Thanks,
> Cristian
>
> [1]: https://lore.kernel.org/linux-arm-kernel/20240709140957.3171255-1-peng.fan@xxxxxxxxxxx/
> [2]: https://lore.kernel.org/linux-arm-kernel/20240730133318.1573765-1-cristian.marussi@xxxxxxx/T/#t
>
> ---

Thanks Cristian for the series. I have validated them on Qualcomm
SA8255p(to be upstreamed) Ride platform.

-Nikunj

>
> Cristian Marussi (5):
> firmware: arm_scmi: Remove const from transport descriptors
> firmware: arm_scmi: Use max-rx-timeout-ms from devicetree
> dt-bindings: firmware: arm,scmi: Introduce more transport properties
> firmware: arm_scmi: Use max_msg and max_msg_size from devicetree
> firmware: arm_scmi: Relocate atomic_threshold to scmi_desc
>
> Peng Fan (1):
> dt-bindings: firmware: arm,scmi: Introduce property max-rx-timeout-ms
>
> .../bindings/firmware/arm,scmi.yaml | 22 +++++++++
> drivers/firmware/arm_scmi/common.h | 9 +++-
> drivers/firmware/arm_scmi/driver.c | 46 ++++++++++++-------
> .../arm_scmi/scmi_transport_mailbox.c | 2 +-
> .../firmware/arm_scmi/scmi_transport_optee.c | 2 +-
> .../firmware/arm_scmi/scmi_transport_smc.c | 2 +-
> .../firmware/arm_scmi/scmi_transport_virtio.c | 2 +-
> 7 files changed, 64 insertions(+), 21 deletions(-)
>