Dear [email protected],

In message <[email protected]> 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: [email protected]
"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
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to