[U-Boot] [PATCH 9/9] mx6qsabrelite: Add splaschscreen support

[Fabio Estevam]

Hi Dirk,

On Sat, Jun 2, 2012 at 3:52 PM, Dirk Behme <dirk.behme at googlemail.com> wrote:

> Maybe you could check [1]
>
> -#define CONFIG_SYS_I2C_SLAVE            0xfe
> +#define CONFIG_SYS_I2C_SLAVE    0x1f
>
> then? I'm no I2C expert and I wasn't sure what to use for
> CONFIG_SYS_I2C_SLAVE.

Yes, 0x1f is the correct address for the Hannstar LVDS I2C slave address.

I was doing:

i2c_read(0x1f, 3, 1, &value, 1);

,which was actually using the 0x1f value and discarded the
CONFIG_SYS_I2C_SLAVE definition.

I can use CONFIG_SYS_I2C_SLAVE instead, which makes the code cleaner.

Thanks,

Fabio Estevam