linux-kernel - RE: [PATCH 1/8] omap3evm: Enable regulators for camera interface

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ADF30F4D7BDE934D9B632CE7D5C7ACA4047C4D090831@dbde03.ent.ti.com>
Date:	Tue, 13 Sep 2011 12:36:20 +0530
From:	"Ravi, Deepthy" <deepthy.ravi@...com>
To:	Laurent Pinchart <laurent.pinchart@...asonboard.com>
CC:	"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>,
	"tony@...mide.com" <tony@...mide.com>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"mchehab@...radead.org" <mchehab@...radead.org>,
	"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
	"g.liakhovetski@....de" <g.liakhovetski@....de>,
	"Hiremath, Vaibhav" <hvaibhav@...com>
Subject: RE: [PATCH 1/8] omap3evm: Enable regulators for camera interface

Hi,
> ________________________________________
> From: Laurent Pinchart [laurent.pinchart@...asonboard.com]
> Sent: Thursday, September 08, 2011 10:21 PM
> To: Ravi, Deepthy
> Cc: linux-omap@...r.kernel.org; tony@...mide.com; linux@....linux.org.uk; linux-arm-kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; mchehab@...radead.org; linux-media@...r.kernel.org; g.liakhovetski@....de; Hiremath, Vaibhav
> Subject: Re: [PATCH 1/8] omap3evm: Enable regulators for camera interface
>
> Hi,
>
> On Thursday 08 September 2011 15:33:51 Deepthy Ravi wrote:
>> From: Vaibhav Hiremath <hvaibhav@...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@...com>
>> Signed-off-by: Deepthy Ravi <deepthy.ravi@...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").
>
[Deepthy Ravi] Yes, you are right. I will modify it.
>> +};
>> +
>> +/* 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
>

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