Re: [PATCH v2 3/3] firmware: scmi: fix inline comments and minor coding style issues

On Fri, 19 Feb 2021 at 06:34, Etienne Carriere etienne.carriere@linaro.org wrote:

Fix inline comments and empty line in scmi driver and test files.

Remove test on IS_ENABLED(CONFIG_*_SCMI) in test/dm/scmi.c since these configuration are expected enabled when CONFIG_FIRMWARE_SCMI is enabled in sandbox configuration.

Signed-off-by: Etienne Carriere etienne.carriere@linaro.org

Changes in v2:

• discard adding tests on IS_ENABLED(CONFIG_*_SCMI) in test/dm/scmi.c for the reason stated in commit log.

---

arch/sandbox/include/asm/scmi_test.h | 5 +++-- drivers/firmware/scmi/sandbox-scmi_agent.c | 4 ++-- test/dm/scmi.c | 15 ++------------- 3 files changed, 7 insertions(+), 17 deletions(-)

Reviewed-by: Simon Glass sjg@chromium.org