[U-Boot] [PATCH v2 07/12] exynos5: support tps65090 pmic

[Simon Glass]

Hi Minkyu,

On 15 May 2014 00:51, Minkyu Kang <mk7.kang at samsung.com> wrote:
> On 03/04/14 08:24, Simon Glass wrote:
>> From: Aaron Durbin <adurbin at chromium.org>
>>
>> The TSP65090 is a PMIC on some exynos5 boards. The init function is
>> called for the TPS65090 pmic. If that device is not a part of the device
>> tree (returns -ENODEV) then continue. Otherwise return a failure.
>>
>> Signed-off-by: Aaron Durbin <adurbin at chromium.org>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> Reviewed-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>> Changes in v2:
>> - Move code to exynos5-dt.c
>> - Fix comment style
>> - Add #ifdef around tps65090 code
>>
>>  board/samsung/smdk5250/exynos5-dt.c | 13 +++++++++++++
>>  1 file changed, 13 insertions(+)
>>
>> diff --git a/board/samsung/smdk5250/exynos5-dt.c b/board/samsung/smdk5250/exynos5-dt.c
>> index 1a64b9b..2c1cf8a 100644
>> --- a/board/samsung/smdk5250/exynos5-dt.c
>> +++ b/board/samsung/smdk5250/exynos5-dt.c
>> @@ -20,6 +20,7 @@
>>  #include <asm/arch/sromc.h>
>>  #include <power/pmic.h>
>>  #include <power/max77686_pmic.h>
>> +#include <power/tps65090_pmic.h>
>>  #include <tmu.h>
>>
>>  DECLARE_GLOBAL_DATA_PTR;
>> @@ -164,7 +165,19 @@ int exynos_power_init(void)
>>
>>  #ifdef CONFIG_POWER_MAX77686
>>       ret = max77686_init();
>> +     if (ret)
>> +             return ret;
>>  #endif
>> +#ifdef CONFIG_POWER_TPS65090
>> +     /*
>> +      * The TPS65090 may not be in the device tree. If so, it is not
>> +      * an error.
>
> Then, how we can initialise the tps65090?

It is initialised if a suitable node is found in the device tree. If
the device tree does not have it, then the hardware is assumed to not
have this chip.

>
>> +      */
>> +     ret = tps65090_init();
>> +     if (ret == 0 || ret == -ENODEV)
>> +             return 0;
>> +#endif
>> +
>>       return ret;
>>  }
>>  #endif       /* CONFIG_POWER */
>>
>
> Thanks,
> Minkyu Kang.

Regards,
Simon