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 <lokeshvu...@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.
> *
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
