[U-Boot] [PATCH 05/10] exynos5: support tps65090 pmic

[Minkyu Kang]

On 27/03/14 02:56, 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>
> ---
> 
>  board/samsung/common/board.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
> index 3866495..654bdb6 100644
> --- a/board/samsung/common/board.c
> +++ b/board/samsung/common/board.c
> @@ -23,6 +23,7 @@
>  #include <power/pmic.h>
>  #include <asm/arch/sromc.h>
>  #include <power/max77686_pmic.h>
> +#include <power/tps65090_pmic.h>
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> @@ -261,6 +262,12 @@ int power_init_board(void)
>  	ret = max77686_init();
>  #endif

What is the base tree of this patch?
looks different with samsung tree.
seems to need rebase?

>  

#ifdef CONFIG_POWER_TPS65090

> +	/* The TPS65090 may not be in the device tree. If so, it is not
> +	 * an error. */

please fix this multi line comment.
It should be
/*
 * ...
 * ...
 */

> +	ret = tps65090_init();
> +	if (ret == 0 || ret == -ENODEV)
> +		return 0;

#endif

> +
>  	return ret;
>  }
>  #endif
> 

Thanks,
Minkyu Kang.