OPP layer code should be PMIC-independent. A future patch will remove
the vsel field, which is TPS/TWL-dependent.
---
arch/arm/plat-omap/include/plat/opp.h | 4 +++-
arch/arm/plat-omap/opp.c | 9 ++++++---
2 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/arch/arm/plat-omap/include/plat/opp.h
b/arch/arm/plat-omap/include/plat/opp.h
index 2a7c113..4329767 100644
--- a/arch/arm/plat-omap/include/plat/opp.h
+++ b/arch/arm/plat-omap/include/plat/opp.h
@@ -21,6 +21,7 @@
* @vsel: Voltage in volt processor level(this usage is
* DEPRECATED to use Voltage in microvolts in future)
* uV = ((vsel * 12.5) + 600) * 1000
+ * @u_volt: minimum microvolts DC required for this OPP to function
*
* This structure stores the OPP information for a given domain.
* Due to legacy reasons, this structure is currently exposed and
@@ -30,8 +31,9 @@
struct omap_opp {
bool enabled;
unsigned long rate;
+ unsigned long u_volt;
u8 __deprecated opp_id;
- u16 __deprecated vsel;
+ u8 __deprecated vsel;
};
/**
diff --git a/arch/arm/plat-omap/opp.c b/arch/arm/plat-omap/opp.c
index 2543932..2e97237 100644
--- a/arch/arm/plat-omap/opp.c
+++ b/arch/arm/plat-omap/opp.c
@@ -23,7 +23,7 @@
* This is meant to help co-exist with current SRF etc
* TODO: REMOVE!
*/
-#define OPP_TERM(opp) (!(opp)->rate && !(opp)->vsel && !(opp)->enabled)
+#define OPP_TERM(opp) (!(opp)->rate && !(opp)->u_volt && !(opp)->enabled)
unsigned long opp_get_voltage(const struct omap_opp *opp)
{
@@ -31,7 +31,7 @@ unsigned long opp_get_voltage(const struct omap_opp *opp)
pr_err("%s: Invalid parameters being passed\n", __func__);
return 0;
}
- return omap_twl_vsel_to_uv(opp->vsel);
+ return opp->u_volt;
}
unsigned long opp_get_freq(const struct omap_opp *opp)
@@ -143,7 +143,8 @@ static void omap_opp_populate(struct omap_opp *opp,
{
opp->rate = opp_def->freq;
opp->enabled = opp_def->enabled;
- opp->vsel = omap_twl_uv_to_vsel(opp_def->u_volt);
+ opp->u_volt = opp_def->u_volt;
+ opp->vsel = omap_twl_uv_to_vsel(opp_def->u_volt); /* XXX remove me */
}
struct omap_opp *opp_add(struct omap_opp *oppl,
@@ -234,6 +235,7 @@ struct omap_opp __init *opp_init_list(const struct
omap_opp_def *opp_defs)
opp->rate = 0;
opp->enabled = 0;
opp->vsel = 0;
+ opp->u_volt = 0;
opp++;
while (n) {
omap_opp_populate(opp, opp_defs);
@@ -247,6 +249,7 @@ struct omap_opp __init *opp_init_list(const struct
omap_opp_def *opp_defs)
opp->rate = 0;
opp->enabled = 0;
opp->vsel = 0;
+ opp->u_volt = 0;
return oppl;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html