[U-Boot] [PATCH 14/25] power domain: Add support for multiple powerdomains per device

[Lokesh Vutla <lokeshvutla@ti.com>]

On Tuesday 21 August 2018 08:01 PM, Lokesh Vutla wrote:
> There are cases where there are more than one power domain
> attached to the device inorder to get the device functional.
> So add support for enabling power domain based on the index.
> 
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
>  drivers/power/domain/power-domain-uclass.c | 11 +++++++++--
>  include/power-domain.h                     | 19 +++++++++++++++++++
>  2 files changed, 28 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/power/domain/power-domain-uclass.c b/drivers/power/domain/power-domain-uclass.c
> index 9e9ec4f419..2ea0ff24c7 100644
> --- a/drivers/power/domain/power-domain-uclass.c
> +++ b/drivers/power/domain/power-domain-uclass.c
> @@ -28,7 +28,8 @@ static int power_domain_of_xlate_default(struct power_domain *power_domain,
>  	return 0;
>  }
>  
> -int power_domain_get(struct udevice *dev, struct power_domain *power_domain)
> +int power_domain_get_by_index(struct udevice *dev,
> +			      struct power_domain *power_domain, int index)
>  {
>  	struct ofnode_phandle_args args;
>  	int ret;
> @@ -38,7 +39,8 @@ int power_domain_get(struct udevice *dev, struct power_domain *power_domain)
>  	debug("%s(dev=%p, power_domain=%p)\n", __func__, dev, power_domain);
>  
>  	ret = dev_read_phandle_with_args(dev, "power-domains",
> -					 "#power-domain-cells", 0, 0, &args);
> +					 "#power-domain-cells", 0, index,
> +					 &args);
>  	if (ret) {
>  		debug("%s: dev_read_phandle_with_args failed: %d\n",
>  		      __func__, ret);
> @@ -73,6 +75,11 @@ int power_domain_get(struct udevice *dev, struct power_domain *power_domain)
>  	return 0;
>  }
>  
> +int power_domain_get(struct udevice *dev, struct power_domain *power_domain)
> +{
> +	return power_domain_get_by_index(dev, power_domain, 0);
> +}
> +
>  int power_domain_free(struct power_domain *power_domain)
>  {
>  	struct power_domain_ops *ops = power_domain_dev_ops(power_domain->dev);
> diff --git a/include/power-domain.h b/include/power-domain.h
> index a558fbbdb2..0c6d77e813 100644
> --- a/include/power-domain.h
> +++ b/include/power-domain.h
> @@ -97,6 +97,25 @@ int power_domain_get(struct udevice *dev, struct power_domain *power_domain)
>  }
>  #endif
>  
> +/**
> + * power_domain_get_by_index - Get the indexed power domain for a device.
> + *
> + * @dev:		The client device.
> + * @power_domain:	A pointer to a power domain struct to initialize.
> + * @index:		Power domain index to be powered on.
> + *
> + * @return 0 if OK, or a negative error code.
> + */
> +#if CONFIG_IS_ENABLED(POWER_DOMAIN)
> +int power_domain_get_by_index(struct udevice *dev,
> +			      struct power_domain *power_domain, int index);
> +#else
> +int power_domain_get_by_index(struct udevice *dev,
> +			      struct power_domain *power_domain, int index)

Looks like I missed open braces "{" here. Will wait for some more time
and post a v2.

Thanks and regards,
Lokesh

> +	return -ENOSYS;
> +}
> +#endif
> +
>  /**
>   * power_domain_free - Free a previously requested power domain.
>   *
>