lkml.org 
[lkml]   [2011]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 1/8] omap3evm: Enable regulators for camera interface
Date
Hi,

On Thursday 08 September 2011 15:33:51 Deepthy Ravi wrote:
> From: Vaibhav Hiremath <hvaibhav@ti.com>
>
> Enabled 1v8 and 2v8 regulator output, which is being used by
> camera module.

Thanks for the patch. Just one minor comment below.

> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> Signed-off-by: Deepthy Ravi <deepthy.ravi@ti.com>
> ---
> arch/arm/mach-omap2/board-omap3evm.c | 40
> ++++++++++++++++++++++++++++++++++ 1 files changed, 40 insertions(+), 0
> deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3evm.c
> b/arch/arm/mach-omap2/board-omap3evm.c index a1184b3..8333ee4 100644
> --- a/arch/arm/mach-omap2/board-omap3evm.c
> +++ b/arch/arm/mach-omap2/board-omap3evm.c
> @@ -273,6 +273,44 @@ static struct omap_dss_board_info omap3_evm_dss_data =
> { .default_device = &omap3_evm_lcd_device,
> };
>
> +static struct regulator_consumer_supply omap3evm_vaux3_supply = {
> + .supply = "cam_1v8",
> +};
> +
> +static struct regulator_consumer_supply omap3evm_vaux4_supply = {
> + .supply = "cam_2v8",
> +};
> +
> +/* VAUX3 for CAM_1V8 */
> +static struct regulator_init_data omap3evm_vaux3 = {
> + .constraints = {
> + .min_uV = 1800000,
> + .max_uV = 1800000,
> + .apply_uV = true,
> + .valid_modes_mask = REGULATOR_MODE_NORMAL
> + | REGULATOR_MODE_STANDBY,
> + .valid_ops_mask = REGULATOR_CHANGE_MODE
> + | REGULATOR_CHANGE_STATUS,
> + },
> + .num_consumer_supplies = 1,
> + .consumer_supplies = &omap3evm_vaux3_supply,

I might be wrong, but I think we're standardizing on using REGULATOR_SUPPLY
arrays as described in commit 786b01a8c1db0c0decca55d660a2a3ebd7cfb26b
("cleanup regulator supply definitions in mach-omap2").

> +};
> +
> +/* VAUX4 for CAM_2V8 */
> +static struct regulator_init_data omap3evm_vaux4 = {
> + .constraints = {
> + .min_uV = 1800000,
> + .max_uV = 1800000,
> + .apply_uV = true,
> + .valid_modes_mask = REGULATOR_MODE_NORMAL
> + | REGULATOR_MODE_STANDBY,
> + .valid_ops_mask = REGULATOR_CHANGE_MODE
> + | REGULATOR_CHANGE_STATUS,
> + },
> + .num_consumer_supplies = 1,
> + .consumer_supplies = &omap3evm_vaux4_supply,
> +};
> +
> static struct regulator_consumer_supply omap3evm_vmmc1_supply[] = {
> REGULATOR_SUPPLY("vmmc", "omap_hsmmc.0"),
> };
> @@ -499,6 +537,8 @@ static struct twl4030_platform_data omap3evm_twldata =
> { .vio = &omap3evm_vio,
> .vmmc1 = &omap3evm_vmmc1,
> .vsim = &omap3evm_vsim,
> + .vaux3 = &omap3evm_vaux3,
> + .vaux4 = &omap3evm_vaux4,
> };
>
> static int __init omap3_evm_i2c_init(void)

--
Regards,

Laurent Pinchart


\
 
 \ /
  Last update: 2011-09-09 02:01    [W:0.047 / U:1.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site