[U-Boot] [PATCH v1 01/18] i2c: ihs_i2c: Dual channel support
Heiko Schocher
hs at denx.de
Wed Oct 28 12:23:15 CET 2015
Hello Dirk,
Am 28.10.2015 um 11:46 schrieb dirk.eibach at gdsys.cc:
> From: Dirk Eibach <dirk.eibach at gdsys.cc>
>
> Support two i2c masters per FPGA.
>
> Signed-off-by: Dirk Eibach <dirk.eibach at gdsys.cc>
> ---
>
> README | 9 ++++++
> drivers/i2c/ihs_i2c.c | 76 ++++++++++++++++++++++++++++++++++++++++++---------
> include/gdsys_fpga.h | 12 ++++----
> 3 files changed, 79 insertions(+), 18 deletions(-)
I twould be nice to see a patch, which converts this driver to DM ;-)
As the hole patchserie goes not through the i2c tree:
Acked-by: Heiko Schocher <hs at denx.de>
bye,
Heiko
>
> diff --git a/README b/README
> index 0dc657d..25bd711 100644
> --- a/README
> +++ b/README
> @@ -2446,6 +2446,15 @@ CBFS (Coreboot Filesystem) support
> - CONFIG_SYS_I2C_IHS_CH3 activate hardware channel 3
> - CONFIG_SYS_I2C_IHS_SPEED_3 speed channel 3
> - CONFIG_SYS_I2C_IHS_SLAVE_3 slave addr channel 3
> + - activate dual channel with CONFIG_SYS_I2C_IHS_DUAL
> + - CONFIG_SYS_I2C_IHS_SPEED_0_1 speed channel 0_1
> + - CONFIG_SYS_I2C_IHS_SLAVE_0_1 slave addr channel 0_1
> + - CONFIG_SYS_I2C_IHS_SPEED_1_1 speed channel 1_1
> + - CONFIG_SYS_I2C_IHS_SLAVE_1_1 slave addr channel 1_1
> + - CONFIG_SYS_I2C_IHS_SPEED_2_1 speed channel 2_1
> + - CONFIG_SYS_I2C_IHS_SLAVE_2_1 slave addr channel 2_1
> + - CONFIG_SYS_I2C_IHS_SPEED_3_1 speed channel 3_1
> + - CONFIG_SYS_I2C_IHS_SLAVE_3_1 slave addr channel 3_1
>
> additional defines:
>
> diff --git a/drivers/i2c/ihs_i2c.c b/drivers/i2c/ihs_i2c.c
> index 19fbe59..737beaf 100644
> --- a/drivers/i2c/ihs_i2c.c
> +++ b/drivers/i2c/ihs_i2c.c
> @@ -11,6 +11,28 @@
>
> DECLARE_GLOBAL_DATA_PTR;
>
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +#define I2C_SET_REG(fld, val) \
> + { if (I2C_ADAP_HWNR & 0x10) \
> + FPGA_SET_REG(I2C_ADAP_HWNR & 0xf, i2c1.fld, val); \
> + else \
> + FPGA_SET_REG(I2C_ADAP_HWNR, i2c0.fld, val); }
> +#else
> +#define I2C_SET_REG(fld, val) \
> + FPGA_SET_REG(I2C_ADAP_HWNR, i2c0.fld, val);
> +#endif
> +
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +#define I2C_GET_REG(fld, val) \
> + { if (I2C_ADAP_HWNR & 0x10) \
> + FPGA_GET_REG(I2C_ADAP_HWNR & 0xf, i2c1.fld, val); \
> + else \
> + FPGA_GET_REG(I2C_ADAP_HWNR, i2c0.fld, val); }
> +#else
> +#define I2C_GET_REG(fld, val) \
> + FPGA_GET_REG(I2C_ADAP_HWNR, i2c0.fld, val);
> +#endif
> +
> enum {
> I2CINT_ERROR_EV = 1 << 13,
> I2CINT_TRANSMIT_EV = 1 << 14,
> @@ -29,14 +51,14 @@ static int wait_for_int(bool read)
> u16 val;
> unsigned int ctr = 0;
>
> - FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, &val);
> + I2C_GET_REG(interrupt_status, &val);
> while (!(val & (I2CINT_ERROR_EV
> | (read ? I2CINT_RECEIVE_EV : I2CINT_TRANSMIT_EV)))) {
> udelay(10);
> if (ctr++ > 5000) {
> return 1;
> }
> - FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, &val);
> + I2C_GET_REG(interrupt_status, &val);
> }
>
> return (val & I2CINT_ERROR_EV) ? 1 : 0;
> @@ -47,30 +69,30 @@ static int ihs_i2c_transfer(uchar chip, uchar *buffer, int len, bool read,
> {
> u16 val;
>
> - FPGA_SET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, I2CINT_ERROR_EV
> + I2C_SET_REG(interrupt_status, I2CINT_ERROR_EV
> | I2CINT_RECEIVE_EV | I2CINT_TRANSMIT_EV);
> - FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, &val);
> + I2C_GET_REG(interrupt_status, &val);
>
> if (!read && len) {
> val = buffer[0];
>
> if (len > 1)
> val |= buffer[1] << 8;
> - FPGA_SET_REG(I2C_ADAP_HWNR, i2c.write_mailbox_ext, val);
> + I2C_SET_REG(write_mailbox_ext, val);
> }
>
> - FPGA_SET_REG(I2C_ADAP_HWNR, i2c.write_mailbox,
> - I2CMB_NATIVE
> - | (read ? 0 : I2CMB_WRITE)
> - | (chip << 1)
> - | ((len > 1) ? I2CMB_2BYTE : 0)
> - | (is_last ? 0 : I2CMB_HOLD_BUS));
> + I2C_SET_REG(write_mailbox,
> + I2CMB_NATIVE
> + | (read ? 0 : I2CMB_WRITE)
> + | (chip << 1)
> + | ((len > 1) ? I2CMB_2BYTE : 0)
> + | (is_last ? 0 : I2CMB_HOLD_BUS));
>
> if (wait_for_int(read))
> return 1;
>
> if (read) {
> - FPGA_GET_REG(I2C_ADAP_HWNR, i2c.read_mailbox_ext, &val);
> + I2C_GET_REG(read_mailbox_ext, &val);
> buffer[0] = val & 0xff;
> if (len > 1)
> buffer[1] = val >> 8;
> @@ -163,7 +185,7 @@ static int ihs_i2c_write(struct i2c_adapter *adap, uchar chip, uint addr,
> }
>
> static unsigned int ihs_i2c_set_bus_speed(struct i2c_adapter *adap,
> - unsigned int speed)
> + unsigned int speed)
> {
> if (speed != adap->speed)
> return 1;
> @@ -179,6 +201,13 @@ U_BOOT_I2C_ADAP_COMPLETE(ihs0, ihs_i2c_init, ihs_i2c_probe,
> ihs_i2c_set_bus_speed,
> CONFIG_SYS_I2C_IHS_SPEED_0,
> CONFIG_SYS_I2C_IHS_SLAVE_0, 0)
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +U_BOOT_I2C_ADAP_COMPLETE(ihs0_1, ihs_i2c_init, ihs_i2c_probe,
> + ihs_i2c_read, ihs_i2c_write,
> + ihs_i2c_set_bus_speed,
> + CONFIG_SYS_I2C_IHS_SPEED_0_1,
> + CONFIG_SYS_I2C_IHS_SLAVE_0_1, 16)
> +#endif
> #endif
> #ifdef CONFIG_SYS_I2C_IHS_CH1
> U_BOOT_I2C_ADAP_COMPLETE(ihs1, ihs_i2c_init, ihs_i2c_probe,
> @@ -186,6 +215,13 @@ U_BOOT_I2C_ADAP_COMPLETE(ihs1, ihs_i2c_init, ihs_i2c_probe,
> ihs_i2c_set_bus_speed,
> CONFIG_SYS_I2C_IHS_SPEED_1,
> CONFIG_SYS_I2C_IHS_SLAVE_1, 1)
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +U_BOOT_I2C_ADAP_COMPLETE(ihs1_1, ihs_i2c_init, ihs_i2c_probe,
> + ihs_i2c_read, ihs_i2c_write,
> + ihs_i2c_set_bus_speed,
> + CONFIG_SYS_I2C_IHS_SPEED_1_1,
> + CONFIG_SYS_I2C_IHS_SLAVE_1_1, 17)
> +#endif
> #endif
> #ifdef CONFIG_SYS_I2C_IHS_CH2
> U_BOOT_I2C_ADAP_COMPLETE(ihs2, ihs_i2c_init, ihs_i2c_probe,
> @@ -193,6 +229,13 @@ U_BOOT_I2C_ADAP_COMPLETE(ihs2, ihs_i2c_init, ihs_i2c_probe,
> ihs_i2c_set_bus_speed,
> CONFIG_SYS_I2C_IHS_SPEED_2,
> CONFIG_SYS_I2C_IHS_SLAVE_2, 2)
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +U_BOOT_I2C_ADAP_COMPLETE(ihs2_1, ihs_i2c_init, ihs_i2c_probe,
> + ihs_i2c_read, ihs_i2c_write,
> + ihs_i2c_set_bus_speed,
> + CONFIG_SYS_I2C_IHS_SPEED_2_1,
> + CONFIG_SYS_I2C_IHS_SLAVE_2_1, 18)
> +#endif
> #endif
> #ifdef CONFIG_SYS_I2C_IHS_CH3
> U_BOOT_I2C_ADAP_COMPLETE(ihs3, ihs_i2c_init, ihs_i2c_probe,
> @@ -200,4 +243,11 @@ U_BOOT_I2C_ADAP_COMPLETE(ihs3, ihs_i2c_init, ihs_i2c_probe,
> ihs_i2c_set_bus_speed,
> CONFIG_SYS_I2C_IHS_SPEED_3,
> CONFIG_SYS_I2C_IHS_SLAVE_3, 3)
> +#ifdef CONFIG_SYS_I2C_IHS_DUAL
> +U_BOOT_I2C_ADAP_COMPLETE(ihs3_1, ihs_i2c_init, ihs_i2c_probe,
> + ihs_i2c_read, ihs_i2c_write,
> + ihs_i2c_set_bus_speed,
> + CONFIG_SYS_I2C_IHS_SPEED_3_1,
> + CONFIG_SYS_I2C_IHS_SLAVE_3_1, 19)
> +#endif
> #endif
> diff --git a/include/gdsys_fpga.h b/include/gdsys_fpga.h
> index 8a5efe7..f8d8322 100644
> --- a/include/gdsys_fpga.h
> +++ b/include/gdsys_fpga.h
> @@ -143,7 +143,7 @@ struct ihs_fpga {
> u16 reserved_2[2]; /* 0x001c */
> struct ihs_io_ep ep; /* 0x0020 */
> u16 reserved_3[9]; /* 0x002e */
> - struct ihs_i2c i2c; /* 0x0040 */
> + struct ihs_i2c i2c0; /* 0x0040 */
> u16 reserved_4[10]; /* 0x004c */
> u16 mc_int; /* 0x0060 */
> u16 mc_int_en; /* 0x0062 */
> @@ -177,7 +177,7 @@ struct ihs_fpga {
> u16 reserved_2[2]; /* 0x001c */
> struct ihs_io_ep ep; /* 0x0020 */
> u16 reserved_3[9]; /* 0x002e */
> - struct ihs_i2c i2c; /* 0x0040 */
> + struct ihs_i2c i2c0; /* 0x0040 */
> u16 reserved_4[10]; /* 0x004c */
> u16 mc_int; /* 0x0060 */
> u16 mc_int_en; /* 0x0062 */
> @@ -208,10 +208,12 @@ struct ihs_fpga {
> u16 reserved_1[29]; /* 0x001e */
> u16 mpc3w_control; /* 0x0058 */
> u16 reserved_2[3]; /* 0x005a */
> - struct ihs_i2c i2c; /* 0x0060 */
> - u16 reserved_3[205]; /* 0x0066 */
> + struct ihs_i2c i2c0; /* 0x0060 */
> + u16 reserved_3[2]; /* 0x006c */
> + struct ihs_i2c i2c1; /* 0x0070 */
> + u16 reserved_4[194]; /* 0x007c */
> struct ihs_osd osd; /* 0x0200 */
> - u16 reserved_4[761]; /* 0x020e */
> + u16 reserved_5[761]; /* 0x020e */
> u16 videomem[31736]; /* 0x0800 */
> };
> #endif
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list