Re: [RFC PATCH 00/11] SCMI Notifications Support

Cristian Marussi
2020-01-23T11:02:18Z

Hi

On 20/01/2020 12:23, Cristian Marussi wrote:
> Hi all,
>
> this series wants to introduce SCMI Notification Support, built on top of
> the standard Kernel notification chain subsystem.
>
[snip]

>
> Based on scmi-next [1], on top of:
>
> commit 257d0e20ec4f ("include: trace: Add SCMI header with trace events")

Sorry but I've got this is wrong, the series is based in fact on top of the
very following commit on scmi-next [1]:

729d3530a504 drivers: firmware: scmi: Extend SCMI transport layer by trace events

Thanks Jim for reporting this.

Regards

Cristian

>
> This series has been tested on JUNO with an experimental firmware only
> supporting Perf Notifications.
>
> Any thoughts ?
>
> Thanks
>
> Cristian
> ----
>
> [1] git://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git
>
> Cristian Marussi (8):
> firmware: arm_scmi: Add core notifications support
> firmware: arm_scmi: Add notifications anti-tampering
> firmware: arm_scmi: Enable core notifications
> firmware: arm_scmi: Add Power notifications support
> firmware: arm_scmi: Add Perf notifications support
> firmware: arm_scmi: Add Sensor notifications support
> firmware: arm_scmi: Add Reset notifications support
> firmware: arm_scmi: Add Base notifications support
>
> Sudeep Holla (3):
> firmware: arm_scmi: Add receive buffer support for notifications
> firmware: arm_scmi: Update protocol commands and notification list
> firmware: arm_scmi: Add support for notifications message processing
>
> drivers/firmware/arm_scmi/Makefile | 2 +-
> drivers/firmware/arm_scmi/base.c | 132 ++++
> drivers/firmware/arm_scmi/bus.c | 3 +
> drivers/firmware/arm_scmi/common.h | 4 +
> drivers/firmware/arm_scmi/driver.c | 121 +++-
> drivers/firmware/arm_scmi/notify.c | 1047 +++++++++++++++++++++++++++
> drivers/firmware/arm_scmi/notify.h | 79 ++
> drivers/firmware/arm_scmi/perf.c | 167 ++++-
> drivers/firmware/arm_scmi/power.c | 161 +++-
> drivers/firmware/arm_scmi/reset.c | 126 +++-
> drivers/firmware/arm_scmi/sensors.c | 105 ++-
> include/linux/scmi_protocol.h | 82 +++
> 12 files changed, 1991 insertions(+), 38 deletions(-)
> create mode 100644 drivers/firmware/arm_scmi/notify.c
> create mode 100644 drivers/firmware/arm_scmi/notify.h
>