[U-Boot] [PATCH 05/18] compulab: eeprom: add support for defining eeprom i2c bus

[Nikita Kiryanov <nikita@compulab.co.il>]

On 03/08/14 16:48, Marek Vasut wrote:
> On Sunday, August 03, 2014 at 09:34:35 AM, Nikita Kiryanov wrote:
>> Create CONFIG_SYS_I2C_EEPROM_BUS #define to tell the EEPROM
>> module what I2C bus the EEPROM is located at. Make cl_eeprom_read()
>> switch to that bus when reading EEPROM.
>>
>> Cc: Igor Grinberg <grinberg@compulab.co.il>
>> Cc: Dmitry Lifshitz <lifshitz@compulab.co.il>
>> Cc: Tom Rini <trini@ti.com>
>> Acked-by: Igor Grinberg <grinberg@compulab.co.il>
>> Acked-by: Dmitry Lifshitz <lifshitz@compulab.co.il>
>> Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il>
>> ---
>>   board/compulab/common/eeprom.c | 10 +++++++++-
>>   include/configs/cm_t335.h      |  1 +
>>   include/configs/cm_t35.h       |  1 +
>>   include/configs/cm_t54.h       |  1 +
>>   4 files changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/board/compulab/common/eeprom.c
>> b/board/compulab/common/eeprom.c index 20fe3e1..b5c1c2a 100644
>> --- a/board/compulab/common/eeprom.c
>> +++ b/board/compulab/common/eeprom.c
>> @@ -31,8 +31,16 @@ static int cl_eeprom_layout; /* Implicitly
>> LAYOUT_INVALID */
>>
>>   static int cl_eeprom_read(uint offset, uchar *buf, int len)
>>   {
>> -	return i2c_read(CONFIG_SYS_I2C_EEPROM_ADDR, offset,
>> +	int res;
>> +	unsigned int current_i2c_bus = i2c_get_bus_num();
>> +
>> +	i2c_set_bus_num(CONFIG_SYS_I2C_EEPROM_BUS);
>
> Please handle the return value of this function too.

Will do..

> [...]
> BR,MV
>