Sylwester Nawrocki wrote:
>
> Add regulator supplies required for NOON010PC30 CIF sensor.
>
> Signed-off-by: Sylwester Nawrocki <[email protected]>
> Signed-off-by: Kyungmin Park <[email protected]>
> ---
> 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?
+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.
+ .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 <[email protected]>, 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
