[U-Boot] [PATCH 45/55] exynos: Enable new features for exynos5 boards

[Simon Glass]

Hi Prrzemyslaw,

On 10 July 2015 at 05:55, Przemyslaw Marczak <p.marczak at samsung.com> wrote:
> Hello,
>
> On 07/03/2015 02:16 AM, Simon Glass wrote:
>>
>> Enable PMICs, regulators and the like so that new drivers will be made
>> available.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>
>> ---
>>
>>   board/samsung/smdk5420/smdk5420.c   |  2 +-
>>   configs/arndale_defconfig           |  2 ++
>>   configs/odroid-xu3_defconfig        |  5 +++++
>>   configs/peach-pi_defconfig          | 17 +++++++++++++++++
>>   configs/peach-pit_defconfig         | 17 +++++++++++++++++
>>   configs/smdk5250_defconfig          | 10 ++++++++++
>>   configs/smdk5420_defconfig          |  5 +++++
>>   configs/snow_defconfig              | 23 +++++++++++++++++++++++
>>   include/configs/arndale.h           |  4 ++++
>>   include/configs/exynos5-common.h    | 10 ++--------
>>   include/configs/exynos5-dt-common.h |  5 -----
>>   include/configs/smdk5250.h          |  3 ---
>>   include/configs/snow.h              |  3 ---
>>   13 files changed, 86 insertions(+), 20 deletions(-)
>>
>> diff --git a/board/samsung/smdk5420/smdk5420.c
>> b/board/samsung/smdk5420/smdk5420.c
>> index 88f4044..57cc92c 100644
>> --- a/board/samsung/smdk5420/smdk5420.c
>> +++ b/board/samsung/smdk5420/smdk5420.c
>> @@ -50,9 +50,9 @@ static int has_edp_bridge(void)
>>
>>   void exynos_lcd_power_on(void)
>>   {
>> +#ifdef CONFIG_POWER_TPS65090
>>         int ret;
>>
>> -#ifdef CONFIG_POWER_TPS65090
>>         ret = tps65090_init();
>>         if (ret < 0) {
>>                 printf("%s: tps65090_init() failed\n", __func__);
>> diff --git a/configs/arndale_defconfig b/configs/arndale_defconfig
>> index 43f39f2..10e9d03 100644
>> --- a/configs/arndale_defconfig
>> +++ b/configs/arndale_defconfig
>> @@ -10,3 +10,5 @@ CONFIG_I2S=y
>>   CONFIG_I2S_SAMSUNG=y
>>   CONFIG_SOUND_MAX98095=y
>>   CONFIG_SOUND_WM8994=y
>> +CONFIG_DM_I2C=y
>> +CONFIG_DM_I2C_COMPAT=y
>> diff --git a/configs/odroid-xu3_defconfig b/configs/odroid-xu3_defconfig
>> index c9c4849..9760808 100644
>> --- a/configs/odroid-xu3_defconfig
>> +++ b/configs/odroid-xu3_defconfig
>> @@ -4,3 +4,8 @@ CONFIG_TARGET_ODROID_XU3=y
>>   # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
>>   CONFIG_DEFAULT_DEVICE_TREE="exynos5422-odroidxu3"
>>   CONFIG_CMD_NET=y
>> +CONFIG_DM_I2C=y
>> +CONFIG_DM_I2C_COMPAT=y
>> +CONFIG_DM_PMIC=y
>> +CONFIG_DM_REGULATOR=y
>
>
> At present we don't have pmic/regulator driver for XU3, but probably will
> have soon, so it can be enabled.
>
> But there is no video bridge on XU3.
>
> I assume that you enable this, because of using the single board file, but
> should go this way?

That's my intent, yes. After all, downstream boards can always fiddle
with the config if needed.

Regards,
Simon