lkml.org 
[lkml]   [2012]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3] regulator: add new regulator driver for lp872x
2012/6/19 Kim, Milo <Milo.Kim@ti.com>:
> This driver supports TI/National LP8720, LP8725 PMIC.
>
You might want to add more commit description here.
e.g. ( I just copy-paste from your first version.)

This driver supports TI/National Semiconductor LP8720 and LP8725 PMU.
LP8720 : 5 LDOs and 1 BUCK
LP8725 : 7 LDOs and 2 BUCKS
These ICs have similar register map for controlling regulators.

Datasheet
---------
LP8720 : http://www.ti.com/litv/pdf/snvs575a
LP8725 : http://www.ti.com/lit/gpn/lp8725
I2C compatible interface
------------------------
The regmap APIs are used for accessing the registers
Supported regulator operations
------------------------------
* list_voltage/set_voltage_sel/get_voltage_sel
: voltage tables are used for selecting specific voltage
* enable/disable/is_enabled/enable_time
* set_mode/get_mode
: BUCK specific operations. Forced pwm and normal mode are selective
* set_current_limit/get_current_limit
: current limit operations for lp8725 BUCKs
Platform data
-------------
3 mandatory and 1 optional data are defined.
* general_config : value of GENERAL_CFG register is platform specific data
* regulator_data : regulator init data with id in platform side
* num_regulators : numbers of regulator_data
* get_dvs_pin_state : used for selecting buck output register
DVS input is platform specific pin for choosing buck register address

Below description should be put under --- line, so it won't display in
commit log.

> patch v3
>
> (a) replace few if-statements with switch-statements
> (b) use devm_gpio_request_one() for configuring dvs gpio
> : replace gpio_request() and gpio_direction_output() with devm_gpio_request_one()
>  remove gpio_free() code
>
> (c) enhanced code on finding matched regulator init data
> : change max loop count. fixed loop count (9) -> variable count (6 or 9)
>
> Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>

The code looks good to me,
Reviewed-by: Axel Lin <axel.lin@gmail.com>

Regards,
Axel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2012-06-19 10:41    [W:0.061 / U:1.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site