Thara Gopinath <th...@ti.com> writes:

> This patch adds an API in the voltage layer that
> can be used  during omap_device_build to register the built
> device with the voltage domain. This API is to be typically called
> only once per device during the device registeration. This approach
> makes it easy during dvfs to scale all the devices associated with
> a voltage domain and then scale the voltage domain.
>
> Signed-off-by: Thara Gopinath <th...@ti.com>

[...]

>  /* Generic voltage init functions */
> @@ -1240,6 +1251,40 @@ int omap_voltage_add_request(struct voltagedomain 
> *voltdm, struct device *dev,
>       return 0;
>  }
>  
> +int omap_voltage_add_dev(struct voltagedomain *voltdm, struct device *dev)

minor nit: I prefer this be a 'register' function,
e.g. omap_voltage_register (or _register_dev)

[...]

> diff --git a/arch/arm/plat-omap/omap_device.c 
> b/arch/arm/plat-omap/omap_device.c
> index abe933c..7c902a6 100644
> --- a/arch/arm/plat-omap/omap_device.c
> +++ b/arch/arm/plat-omap/omap_device.c
> @@ -86,6 +86,7 @@
>  
>  #include <plat/omap_device.h>
>  #include <plat/omap_hwmod.h>
> +#include <plat/voltage.h>
>  
>  /* These parameters are passed to _omap_device_{de,}activate() */
>  #define USE_WAKEUP_LAT                       0
> @@ -453,6 +454,17 @@ struct omap_device *omap_device_build_ss(const char 
> *pdev_name, int pdev_id,
>       for (i = 0; i < oh_cnt; i++) {
>               hwmods[i]->od = od;
>               _add_optional_clock_alias(od, hwmods[i]);
> +             if (hwmods[i]->vdd_name) {

Use:

                if (!is_early_device && hwmods[i]->vdd_name)              

> +                     struct omap_hwmod *oh = hwmods[i];
> +                     struct voltagedomain *voltdm;
> +
> +                     if (is_early_device)
> +                             continue;

and drop this check


> +                     voltdm = omap_voltage_domain_lookup(oh->vdd_name);
> +                     if (!omap_voltage_add_dev(voltdm, &od->pdev.dev))
> +                             oh->voltdm = voltdm;
> +             }
>       }
>  
>       if (ret)

Kevin

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to