From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Thu, 10 May 2018 12:23:19 +0200
Subject: [U-Boot] [PATCH 3/4] ARM: rmobile: Enable DM capable RCar I2C
 driver on Lager
In-Reply-To: <a44cd0fc-cb74-bf0a-4cbe-93287b73785c@denx.de>
References: <20180501070335.1815-1-marek.vasut+renesas@gmail.com>
 <20180501070335.1815-3-marek.vasut+renesas@gmail.com>
 <a44cd0fc-cb74-bf0a-4cbe-93287b73785c@denx.de>
Message-ID: <71656d79-17f8-2d08-c8f7-eae00b60c49f@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

On 05/10/2018 09:23 AM, Heiko Schocher wrote:
> Hello Marek,

Hi,

> Am 01.05.2018 um 09:03 schrieb Marek Vasut:
>> Enable the DM capable driver instead of the legacy one.
>>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
>> Cc: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
>> ---
>>   configs/lager_defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/configs/lager_defconfig b/configs/lager_defconfig
>> index 1071da24d2..7dcd9b7dd8 100644
>> --- a/configs/lager_defconfig
>> +++ b/configs/lager_defconfig
>> @@ -53,6 +53,7 @@ CONFIG_CLK_RENESAS=y
>>   CONFIG_DM_GPIO=y
>>   CONFIG_RCAR_GPIO=y
>>   CONFIG_DM_I2C=y
>> +CONFIG_SYS_I2C_RCAR_I2C=y
>>   CONFIG_SYS_I2C_RCAR_IIC=y
> 
> Isn;t CONFIG_SYS_I2C_RCAR_IIC also a i2c driver?
> 
> Has this SoC 2 different i2c drivers?

Yes :)

> If so, you can add my
> 
> Reviewed-by: Heiko Schocher <hs@denx.de>
> 
> (Also to patch 4)
> 
>>   CONFIG_DM_MMC=y
>>   CONFIG_SH_MMCIF=y
>>
> 
> bye,
> Heiko


-- 
Best regards,
Marek Vasut