
Hi Takahiro,
Ășt 26. 9. 2023 v 9:00 odesĂlatel AKASHI Takahiro takahiro.akashi@linaro.org napsal:
This command, "scmi", may provide a command line interface to various SCMI protocols. It supports at least initially SCMI base protocol and is intended mainly for debug purpose.
Signed-off-by: AKASHI Takahiro takahiro.akashi@linaro.org Reviewed-by: Simon Glass sjg@chromium.org Reviewed-by: Etienne Carriere etienne.carriere@foss.st.com
v3
- describe that arguments are in hex at a help message
- modify the code for dynamically allocated agent names
v2
- remove sub command category, 'scmi base', for simplicity
cmd/Kconfig | 9 ++ cmd/Makefile | 1 + cmd/scmi.c | 337 +++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 347 insertions(+) create mode 100644 cmd/scmi.c
diff --git a/cmd/Kconfig b/cmd/Kconfig index 43ca10f69ccf..f46152ace7d8 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -2533,6 +2533,15 @@ config CMD_CROS_EC a number of sub-commands for performing EC tasks such as updating its flash, accessing a small saved context area and talking to the I2C bus behind the EC (if there is one).
+config CMD_SCMI
bool "Enable scmi command"
depends on SCMI_FIRMWARE
default n
This line above is wrong and this was removed from v6 with "drop scmi command which was intended to be used for debugging". It is fine that it shouldn't be used on production system but it doesn't mean that it should be actually removed. It is useful for bring ups. Can we get this patch merged? It was already reviewed anyway.
Thanks, Michal