[U-Boot] [PATCH] 8/12 Multiadapter/multibus I2C, configs part 1
Wolfgang Denk
wd at denx.de
Mon Feb 16 23:00:38 CET 2009
Dear ksi at koi8.net,
In message <Pine.LNX.4.64ksi.0902121425220.21067 at home-gw.koi8.net> you wrote:
>
> -#define CONFIG_HARD_I2C /* I2C with hardware support*/
> -#undef CONFIG_SOFT_I2C /* I2C bit-banged */
> -#define CONFIG_SYS_I2C_SPEED 400000 /* I2C speed and slave address */
> -#define CONFIG_SYS_I2C_EEPROM_ADDR 0x57
> -#define CONFIG_SYS_I2C_SLAVE 0x7F
> -#define CONFIG_SYS_I2C_NOPROBES {0x69} /* Don't probe these addrs */
> -#define CONFIG_SYS_I2C_OFFSET 0x3000
> +#define CONFIG_NEW_I2C /* New I2C code */
> +#define CONFIG_FSL_I2C /* Use FSL common I2C driver */
> +#define CONFIG_SYS_FSL_I2C_SPEED 400000 /* I2C speed and slave address */
> +#define CONFIG_SYS_FSL_I2C_EEPROM_ADDR 0x57
> +#define CONFIG_SYS_FSL_I2C_SLAVE 0x7F
> +#define CONFIG_SYS_I2C_NOPROBES {0x69} /* Don't probe these addrs */
> +#define CONFIG_SYS_FSL_I2C_OFFSET 0x3000
> +#define CONFIG_SYS_I2C_ADAPTERS {&fsl_i2c_adap[0]}
Do we really need sucheven more unreadable names?
Why do you have to add the FSL_, SOFT_, ... everywhere?
I think this should be redundant. Omit it.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"I used to think that the brain was the most wonderful organ in my
body. Then I realized who was telling me this." - Emo Phillips
More information about the U-Boot
mailing list