[PATCH v2 4/5] firmware: scmi: add configs to select the supported agents

[Patrick DELAUNAY]

On 10/28/21 4:57 PM, Patrick Delaunay wrote:
> Add two configs CONFIG_SCMI_AGENT_MAILBOX and CONFIG_SCMI_AGENT_SMCCC
> to select the supported agents as all the agents are not supported.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> Acked-by: Etienne Carriere <etienne.carriere at linaro.org>
> ---
>
> Changes in v2:
> - fix typo s/compatibble/compatible/
> - add acked by Etienne Carriere for the series
>
>   drivers/firmware/scmi/Kconfig  | 16 ++++++++++++++++
>   drivers/firmware/scmi/Makefile |  4 ++--
>   2 files changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/firmware/scmi/Kconfig b/drivers/firmware/scmi/Kconfig
> index c3a109beac..c33de87cd0 100644
> --- a/drivers/firmware/scmi/Kconfig
> +++ b/drivers/firmware/scmi/Kconfig
> @@ -17,3 +17,19 @@ config SCMI_FIRMWARE
>   	  based on message exchange. Messages can be exchange over tranport
>   	  channels as a mailbox device or an Arm SMCCC service with some
>   	  piece of identified shared memory.
> +
> +config SCMI_AGENT_MAILBOX
> +	bool "Enable SCMI agent mailbox"
> +	depends on SCMI_FIRMWARE && DM_MAILBOX
> +	default y
> +	help
> +	  Enable the SCMI communication channel based on mailbox
> +	  for compatible "arm,scmi".
> +
> +config SCMI_AGENT_SMCCC
> +	bool "Enable SCMI agent SMCCC"
> +	depends on SCMI_FIRMWARE && ARM_SMCCC
> +	default y
> +	help
> +	  Enable the SCMI communication channel based on Arm SMCCC service for
> +	  compatible "arm,scmi-smc".
> diff --git a/drivers/firmware/scmi/Makefile b/drivers/firmware/scmi/Makefile
> index 966475ec10..d6d6f89493 100644
> --- a/drivers/firmware/scmi/Makefile
> +++ b/drivers/firmware/scmi/Makefile
> @@ -1,5 +1,5 @@
>   obj-y	+= scmi_agent-uclass.o
>   obj-y	+= smt.o
> -obj-$(CONFIG_ARM_SMCCC)		+= smccc_agent.o
> -obj-$(CONFIG_DM_MAILBOX)	+= mailbox_agent.o
> +obj-$(SCMI_AGENT_SMCCC)		+= smccc_agent.o
> +obj-$(SCMI_AGENT_MAILBOX)	+= mailbox_agent.o


erreur here when I copy paste the configuration name

=> missing CONFIG_ in variable

+obj-$(CONFIG_SCMI_AGENT_SMCCC)		+= smccc_agent.o
+obj-$(CONFIG_SCMI_AGENT_MAILBOX)	+= mailbox_agent.o

sorry, I will sent v3 !

Patrick