Nishanth Menon <nm@xxxxxx> writes:
> Every PMIC has it's own eccentricities, For example, one of the
> PMIC has MSB set to 1 for a specific function - voltage enable!
> using an hardcoded value specific for TWL when copied over to
> such an implementation causes the system to crash as the MSB bit
> was 0 and the voltage got disabled!.
>
> Instead we use actual values and depend on the convertion routines
> to abstract out the eccentricities of each PMIC.
>
> With this, we can now move the voltages to a common location in
> voltage.h as they are no longer dependent on PMICs and expect the
> PMIC's conversion routines to set a cap if the voltage is out of
> reach for the PMIC.
>
> Reported-by: Jon Hunter <jon-hunter@xxxxxx>
> Signed-off-by: Nishanth Menon <nm@xxxxxx>
Nice.
Minor comment for consideration while you're update this work...
[...]
> diff --git a/arch/arm/mach-omap2/voltage.h b/arch/arm/mach-omap2/voltage.h
> index f079167..7366793 100644
> --- a/arch/arm/mach-omap2/voltage.h
> +++ b/arch/arm/mach-omap2/voltage.h
> @@ -109,6 +109,24 @@ struct omap_volt_data {
> u8 vp_errgain;
> };
>
> +/* Min and max voltages from OMAP perspective */
> +#define OMAP3430_VP1_VLIMITTO_VDDMIN 850000
> +#define OMAP3430_VP1_VLIMITTO_VDDMAX 1425000
> +#define OMAP3430_VP2_VLIMITTO_VDDMIN 900000
> +#define OMAP3430_VP2_VLIMITTO_VDDMAX 1150000
> +
> +#define OMAP3630_VP1_VLIMITTO_VDDMIN 900000
> +#define OMAP3630_VP1_VLIMITTO_VDDMAX 1350000
> +#define OMAP3630_VP2_VLIMITTO_VDDMIN 900000
> +#define OMAP3630_VP2_VLIMITTO_VDDMAX 1200000
> +
> +#define OMAP4_VP_MPU_VLIMITTO_VDDMIN 830000
> +#define OMAP4_VP_MPU_VLIMITTO_VDDMAX 1410000
> +#define OMAP4_VP_IVA_VLIMITTO_VDDMIN 830000
> +#define OMAP4_VP_IVA_VLIMITTO_VDDMAX 1260000
> +#define OMAP4_VP_CORE_VLIMITTO_VDDMIN 830000
> +#define OMAP4_VP_CORE_VLIMITTO_VDDMAX 1200000
> +
Plese reference what doc/TRM/appnote etc. these values come from.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
