[U-Boot] [PATCH 8/8] driver/i2c/mxc: Add I2C3 and I2C4 for LS2085A

[Heiko Schocher]

Hello Yrok,

Am 19.03.2015 17:20, schrieb York Sun:
> LS2085A uses mxc I2C driver and has four I2C buses.
>
> Signed-off-by: York Sun <yorksun at freescale.com>
> CC: Heiko Schocher <hs at denx.de>
> ---
>   drivers/i2c/mxc_i2c.c |   16 +++++++++++++++-
>   1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/i2c/mxc_i2c.c b/drivers/i2c/mxc_i2c.c
> index fc5ee35..02a173e 100644
> --- a/drivers/i2c/mxc_i2c.c
> +++ b/drivers/i2c/mxc_i2c.c
> @@ -114,6 +114,9 @@ static u16 i2c_clk_div[50][2] = {
>   #ifndef CONFIG_SYS_MXC_I2C3_SPEED
>   #define CONFIG_SYS_MXC_I2C3_SPEED 100000
>   #endif
> +#ifndef CONFIG_SYS_MXC_I2C4_SPEED
> +#define CONFIG_SYS_MXC_I2C4_SPEED 100000
> +#endif
>
>   #ifndef CONFIG_SYS_MXC_I2C1_SLAVE
>   #define CONFIG_SYS_MXC_I2C1_SLAVE 0
> @@ -124,6 +127,9 @@ static u16 i2c_clk_div[50][2] = {
>   #ifndef CONFIG_SYS_MXC_I2C3_SLAVE
>   #define CONFIG_SYS_MXC_I2C3_SLAVE 0
>   #endif
> +#ifndef CONFIG_SYS_MXC_I2C4_SLAVE
> +#define CONFIG_SYS_MXC_I2C4_SLAVE 0
> +#endif
>
>
>   /*
> @@ -545,10 +551,18 @@ U_BOOT_I2C_ADAP_COMPLETE(mxc1, mxc_i2c_init, mxc_i2c_probe,
>   			 CONFIG_SYS_MXC_I2C2_SLAVE, 1)
>   #if defined(CONFIG_MX31) || defined(CONFIG_MX35) ||\
>   	defined(CONFIG_MX51) || defined(CONFIG_MX53) ||\
> -	defined(CONFIG_MX6) || defined(CONFIG_LS102XA)
> +	defined(CONFIG_MX6) || defined(CONFIG_LS102XA) ||\
> +	defined(CONFIG_LS2085A)
>   U_BOOT_I2C_ADAP_COMPLETE(mxc2, mxc_i2c_init, mxc_i2c_probe,
>   			 mxc_i2c_read, mxc_i2c_write,
>   			 mxc_i2c_set_bus_speed,
>   			 CONFIG_SYS_MXC_I2C3_SPEED,
>   			 CONFIG_SYS_MXC_I2C3_SLAVE, 2)
>   #endif
> +#ifdef CONFIG_LS2085A
> +U_BOOT_I2C_ADAP_COMPLETE(mxc3, mxc_i2c_init, mxc_i2c_probe,
> +			 mxc_i2c_read, mxc_i2c_write,
> +			 mxc_i2c_set_bus_speed,
> +			 CONFIG_SYS_MXC_I2C4_SPEED,
> +			 CONFIG_SYS_MXC_I2C4_SLAVE, 3)
> +#endif

Dummy question ... CONFIG_LS2085A (and CONFIG_LS102XA) are
board spezific defines, right?

If so, I do not want them in driver specific code... because
every board which uses this driver has to add here more defines.

Could you introduce a SoC or at least a driver specific define
here? Maybe CONFIG_SYS_I2C_MXC_I2C4 if no SoC specific code
is possible ...

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany