Hi Takahiro, út 26. 9. 2023 v 9:00 odesílatel AKASHI Takahiro <takahiro.aka...@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.aka...@linaro.org> > Reviewed-by: Simon Glass <s...@chromium.org> > Reviewed-by: Etienne Carriere <etienne.carri...@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
