diff mbox series

[11/15] firmware: arm_scmi: Match scmi device by both name and protocol id

Message ID 20191210145345.11616-12-sudeep.holla@arm.com
State Superseded
Headers show
Series firmware: arm_scmi: Add support for multiple device per protocol | expand

Commit Message

Sudeep Holla Dec. 10, 2019, 2:53 p.m. UTC
The scmi bus now has support to match the driver with devices not only
based on their protocol id but also based on their device name if one is
available. This was added to cater the need to support multiple devices
and drivers for the same protocol.

Let us add the name "genpd" to scmi_device_id table in the driver so
that in matches only with device with the same name and protocol id
SCMI_PROTOCOL_POWER.

Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

---
 drivers/firmware/arm_scmi/scmi_pm_domain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.17.1

Comments

Cristian Marussi Dec. 11, 2019, 6:09 p.m. UTC | #1
On 10/12/2019 14:53, Sudeep Holla wrote:
> The scmi bus now has support to match the driver with devices not only

> based on their protocol id but also based on their device name if one is

> available. This was added to cater the need to support multiple devices

> and drivers for the same protocol.

> 

> Let us add the name "genpd" to scmi_device_id table in the driver so

> that in matches only with device with the same name and protocol id

> SCMI_PROTOCOL_POWER.

> 

> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

> ---


>  drivers/firmware/arm_scmi/scmi_pm_domain.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/drivers/firmware/arm_scmi/scmi_pm_domain.c b/drivers/firmware/arm_scmi/scmi_pm_domain.c

> index 87f737e01473..bafbfe358f97 100644

> --- a/drivers/firmware/arm_scmi/scmi_pm_domain.c

> +++ b/drivers/firmware/arm_scmi/scmi_pm_domain.c

> @@ -112,7 +112,7 @@ static int scmi_pm_domain_probe(struct scmi_device *sdev)

>  }

> 

>  static const struct scmi_device_id scmi_id_table[] = {

> -	{ SCMI_PROTOCOL_POWER },

> +	{ SCMI_PROTOCOL_POWER, "genpd" },

>  	{ },

>  };

>  MODULE_DEVICE_TABLE(scmi, scmi_id_table);

> --

> 2.17.1

>
Cristian Marussi Dec. 11, 2019, 6:10 p.m. UTC | #2
On 10/12/2019 14:53, Sudeep Holla wrote:
> The scmi bus now has support to match the driver with devices not only

> based on their protocol id but also based on their device name if one is

> available. This was added to cater the need to support multiple devices

> and drivers for the same protocol.

> 

> Let us add the name "genpd" to scmi_device_id table in the driver so

> that in matches only with device with the same name and protocol id

> SCMI_PROTOCOL_POWER.

> 

> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

> ---

>  drivers/firmware/arm_scmi/scmi_pm_domain.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/drivers/firmware/arm_scmi/scmi_pm_domain.c b/drivers/firmware/arm_scmi/scmi_pm_domain.c

> index 87f737e01473..bafbfe358f97 100644

> --- a/drivers/firmware/arm_scmi/scmi_pm_domain.c

> +++ b/drivers/firmware/arm_scmi/scmi_pm_domain.c

> @@ -112,7 +112,7 @@ static int scmi_pm_domain_probe(struct scmi_device *sdev)

>  }

> 

>  static const struct scmi_device_id scmi_id_table[] = {

> -	{ SCMI_PROTOCOL_POWER },

> +	{ SCMI_PROTOCOL_POWER, "genpd" },

>  	{ },

>  };

>  MODULE_DEVICE_TABLE(scmi, scmi_id_table);

> --

> 2.17.1

> 

LGTM.

Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>



Cristian
diff mbox series

Patch

diff --git a/drivers/firmware/arm_scmi/scmi_pm_domain.c b/drivers/firmware/arm_scmi/scmi_pm_domain.c
index 87f737e01473..bafbfe358f97 100644
--- a/drivers/firmware/arm_scmi/scmi_pm_domain.c
+++ b/drivers/firmware/arm_scmi/scmi_pm_domain.c
@@ -112,7 +112,7 @@  static int scmi_pm_domain_probe(struct scmi_device *sdev)
 }

 static const struct scmi_device_id scmi_id_table[] = {
-	{ SCMI_PROTOCOL_POWER },
+	{ SCMI_PROTOCOL_POWER, "genpd" },
 	{ },
 };
 MODULE_DEVICE_TABLE(scmi, scmi_id_table);