[U-Boot] [PATCH 1/3] misc:pmic: Support for voltage to register value conversion function

[Stefano Babic <sbabic@denx.de>]

On 28/03/2012 13:26, Lukasz Majewski wrote:

Hi Lukasz,
>>
>>>  
>>> +int pmic_vol_to_reg(struct pmic *p, int uV)
>>
>> You added a new entry point to pmic, but you do not use it...I have
>> not found in your patchset why it is necessary for you.
>>
> 
> Rationale for this change is in commits following this one.
> For trats Samsung target it is necessary to change the default voltage.

Yes, I was expecting that you call pmic_vol_to_reg() in your following
patches, but I cannot find it. Am I missing something ?

> 
> I think, that providing access to such a function as a pointer is the
> best possible solution.
> 
> For example the MAX8997 and MAX8998 PMICs have different way of
> calculating the value, which represents the LDO output value.
> 
> 
>> I am not sure if we require to add this to the PMIC API or hide in the
>> pmic specific code. Other PMICs has not a a register for different
>> voltages, but only a bit inside the same register (this is the case
>> for the Freescale's PMICs we currently support in u-boot).
> 
> I think, that it is easier to define function pointer in the pmic
> structure, than separate functions and handling them in target platform
> data.

Ok, understood.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================