[PATCH 2/2] arm: exynos: allow platform-lcd driver to control lcd regulator source on origen

Thomas Abraham thomas.abraham at linaro.org
Mon Dec 5 05:47:16 EST 2011


Dear Mr. Park,

On 5 December 2011 15:44, Kyungmin Park <kmpark at infradead.org> wrote:

[...]

>>  static struct regulator_init_data __initdata max8997_ldo1_data = {
>> @@ -379,11 +379,11 @@ static struct regulator_init_data __initdata
>> max8997_buck5_data = {
>>  static struct regulator_init_data __initdata max8997_buck7_data = {
>>       .constraints    = {
>>               .name           = "VDD_LCD_3.3V",
>> -             .min_uV         = 3300000,
>> -             .max_uV         = 3300000,
>> +             .min_uV         = 750000,
>> +             .max_uV         = 3900000,
> It can support the voltage change at buck itself, but in board it's
> fixed. I'm not sure it's correct usage at board file.
>
> I think original code is better to understand and use it as name, it's
> fixed v3.3 voltage.

Yes, that is true. If the board cannot support any other voltage for
buck7, there is no need to change min_uV and max_uV. I will revert it
to the original values.

Thanks,
Thomas.

>
> Thank you,
> Kyungmin Park

[...]



More information about the linux-arm-kernel mailing list