[U-Boot] [PATCH v2] samsung: board: init the s2mps11 pmic during booting time

[Lukasz Majewski <lukma@denx.de>]

Hi Jaehoon,

> Exynos5422 board has s2mps11 pmic.
> If CONFIG_PMIC_S2MPS11 is enabled, it can initialize PMIC and
> Regulators during booting time.
> 
> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
> Tested-by: Anand Moon <linux.amoon@gmail.com>
> ---
> Changelog on V2:
> - Fix the #endif location
> 
>  board/samsung/common/exynos5-dt.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/board/samsung/common/exynos5-dt.c
> b/board/samsung/common/exynos5-dt.c index a4eb351405..862644b00e
> 100644 --- a/board/samsung/common/exynos5-dt.c
> +++ b/board/samsung/common/exynos5-dt.c
> @@ -93,6 +93,9 @@ int exynos_power_init(void)
>  	struct udevice *dev;
>  	int ret;
>  
> +#ifdef CONFIG_PMIC_S2MPS11
> +	ret = pmic_get("s2mps11_pmic", &dev);
> +#else
>  	ret = pmic_get("max77686", &dev);

Cannot this be added to DT? We use #ifdefs in a file, which is supposed
to parse DT config.

>  	if (!ret) {
>  		/* TODO(sjg at chromium.org): Move into the clock/pmic
> API */ @@ -112,6 +115,7 @@ int exynos_power_init(void)
>  			s5m8767_enable_32khz_cp(dev);
>  #endif
>  	}
> +#endif	/* CONFIG_PMIC_S2MPS11 */
>  	if (ret == -ENODEV)
>  		return 0;
>  


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180129/66ef2569/attachment.sig>