lkml.org 
[lkml]   [2013]   [Jul]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH V2 1/8] regulator: Introduce OMAP regulator to control PMIC over VC/VP
On Fri, Jun 21, 2013 at 04:25:42PM -0500, Nishanth Menon wrote:

> +static const struct omap_pmic_info omap_twl4030_vdd1 = {
> + .slave_addr = 0x12,
> + .voltage_reg_addr = 0x00,
> + .cmd_reg_addr = 0x00,
> + .i2c_timeout_us = 200,
> + .slew_rate_uV = 4000,
> + .step_size_uV = 12500,
> + .min_uV = 600000,
> + .max_uV = 1450000,
> + .voltage_selector_offset = 0,
> + .voltage_selector_mask = 0x7F,
> + .voltage_selector_setbits = 0x0,
> + .voltage_selector_zero = false,
> +};

So, this still has the thing where all the data about the PMIC is
replicated (but now in this driver). It should be possible to pull all
the above information except possibly the I2C timeout and perhaps the
I2C address (if there's a separate control interface) from the standard
regulator core data structures for the PMIC. This would allow the
driver to Just Work with any PMIC without needing to add it in two
places.

Other than that this looks good, the only issue I see is where the
driver is getting the data from.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2013-07-04 18:21    [W:1.624 / U:0.188 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site