Hi Igor,

On Wed, 17 Sep 2014 17:18:49 +0300, Igor Grinberg
<grinb...@compulab.co.il> wrote:

> On 09/17/14 15:59, Nikita Kiryanov wrote:
> > Add default eeprom bus setting.
> > This addresses the trimslice compile error that was introduced
> > with the addition of this setting.
> > 
> > Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>
> > Cc: Igor Grinberg <grinb...@compulab.co.il>
> > Signed-off-by: Nikita Kiryanov <nik...@compulab.co.il>
> 
> Acked-by: Igor Grinberg <grinb...@compulab.co.il>
> 
> Albert, this should be a temporary fix to not break the trimslice support.
> We intend to rework this while switching Kconfig for all boards.
> 
> Thanks!
> 
> > ---
> >  board/compulab/common/eeprom.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/board/compulab/common/eeprom.c b/board/compulab/common/eeprom.c
> > index 85442cd..2df3ada 100644
> > --- a/board/compulab/common/eeprom.c
> > +++ b/board/compulab/common/eeprom.c
> > @@ -15,6 +15,10 @@
> >  # define CONFIG_SYS_I2C_EEPROM_ADDR_LEN    1
> >  #endif
> >  
> > +#ifndef CONFIG_SYS_I2C_EEPROM_BUS
> > +#define CONFIG_SYS_I2C_EEPROM_BUS  0
> > +#endif
> > +
> >  #define EEPROM_LAYOUT_VER_OFFSET   44
> >  #define BOARD_SERIAL_OFFSET                20
> >  #define BOARD_SERIAL_OFFSET_LEGACY 8
> > 

a 'git grep CONFIG_SYS_I2C_EEPROM_BUS' shows that all values for
CONFIG_SYS_I2C_EEPROM_BUS are set in include/configs/*.h.

Therefore I would prefer that the temporary fix be in
include/configs/trimslice.h header file too.

Amicalement,
-- 
Albert.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to