On Fri, Jan 28, 2011 at 6:32 PM, Kukjin Kim <kgene....@samsung.com> wrote:
> Sylwester Nawrocki wrote:
>>
>> Add regulator supplies required for NOON010PC30 CIF sensor.
>>
>> Signed-off-by: Sylwester Nawrocki <s.nawro...@samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
>> ---
>> arch/arm/mach-s5pv210/mach-goni.c | 24 +++++++++++++++++++++---
>> 1 files changed, 21 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/mach-s5pv210/mach-goni.c
> b/arch/arm/mach-s5pv210/mach-
>> goni.c
>> index fc34575..2051cee 100644
>> --- a/arch/arm/mach-s5pv210/mach-goni.c
>> +++ b/arch/arm/mach-s5pv210/mach-goni.c
>> @@ -273,6 +273,18 @@ static struct regulator_consumer_supply
>> goni_ldo5_consumers[] = {
>> REGULATOR_SUPPLY("vmmc", "s3c-sdhci.0"),
>> };
>>
>> +static struct regulator_consumer_supply goni_ldo11_consumers[] = {
>> + REGULATOR_SUPPLY("vddio", "0-0030"), /* "CAM_IO_2.8V" */
>> +};
>
> Need array in goni_ldo11_consumers?
Yes, it's required for consistency and most consumers use this style.
Look smdk board file
http://git.kernel.org/?p=linux/kernel/git/kgene/linux-samsung.git;a=blob;f=arch/arm/mach-s3c64xx/mach-smdk6410.c;h=e85192a86fbe1a9f51bb9da15f20899bf0b4ab3e;hb=refs/heads/for-next
303 #ifdef CONFIG_REGULATOR
304 /* ARM core */
305 static struct regulator_consumer_supply smdk6410_vddarm_consumers[] = {
306 {
307 .supply = "vddarm",
308 }
309 };
310
311 /* VDDARM, BUCK1 on J5 */
312 static struct regulator_init_data smdk6410_vddarm = {
313 .constraints = {
314 .name = "PVDD_ARM",
315 .min_uV = 1000000,
316 .max_uV = 1300000,
317 .always_on = 1,
318 .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE,
319 },
320 .num_consumer_supplies = ARRAY_SIZE(smdk6410_vddarm_consumers),
321 .consumer_supplies = smdk6410_vddarm_consumers,
322 };
>
> +static struct regulator_consumer_supply goni_ldo11_consumers =
> + REGULATOR_SUPPLY("vddio", "0-0030"); /* "CAM_IO_2.8V" */
>
>> +
>> +static struct regulator_consumer_supply goni_ldo13_consumers[] = {
>> + REGULATOR_SUPPLY("vdda", "0-0030"), /* "CAM_A_2.8V" */
>> +};
>
> Same above.
>
>> +
>> +static struct regulator_consumer_supply goni_ldo14_consumers[] = {
>> + REGULATOR_SUPPLY("vdd_core", "0-0030"), /* "CAM_CIF_1.8V" */
>> +};
>
> Same above.
>
>> +
>> static struct regulator_init_data goni_ldo2_data = {
>> .constraints = {
>> .name = "VALIVE_1.1V",
>> @@ -371,8 +383,10 @@ static struct regulator_init_data goni_ldo11_data = {
>> .min_uV = 2800000,
>> .max_uV = 2800000,
>> .apply_uV = 1,
>> - .always_on = 1,
>> + .valid_ops_mask = REGULATOR_CHANGE_STATUS,
>> },
>> + .num_consumer_supplies = ARRAY_SIZE(goni_ldo11_consumers),
>> + .consumer_supplies = goni_ldo11_consumers,
>
> If you modify as per my suggestion, then need to change this like following.
Same, I want to use the same style even though it has one consumer.
and If it has one consumer, ARRAY_SIZE(some_consumers) has 1 at
compile time.
>
> + .num_consumer_supplies = 1,
> + .consumer_supplies = &goni_ldo11_consumers,
>
>> };
>>
>> static struct regulator_init_data goni_ldo12_data = {
>> @@ -391,8 +405,10 @@ static struct regulator_init_data goni_ldo13_data = {
>> .min_uV = 2800000,
>> .max_uV = 2800000,
>> .apply_uV = 1,
>> - .always_on = 1,
>> + .valid_ops_mask = REGULATOR_CHANGE_STATUS,
>> },
>> + .num_consumer_supplies = ARRAY_SIZE(goni_ldo13_consumers),
>> + .consumer_supplies = goni_ldo13_consumers,
>
> Same above.
>
>> };
>>
>> static struct regulator_init_data goni_ldo14_data = {
>> @@ -401,8 +417,10 @@ static struct regulator_init_data goni_ldo14_data = {
>> .min_uV = 1800000,
>> .max_uV = 1800000,
>> .apply_uV = 1,
>> - .always_on = 1,
>> + .valid_ops_mask = REGULATOR_CHANGE_STATUS,
>> },
>> + .num_consumer_supplies = ARRAY_SIZE(goni_ldo14_consumers),
>> + .consumer_supplies = goni_ldo14_consumers,
>
> Same above.
>
>> };
>>
>> static struct regulator_init_data goni_ldo15_data = {
>> --
>> 1.7.3.5
>
> If required array in there, please let me know.
>
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene....@samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc"
> in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
