On Mon, 15 Dec 2008, Wolfgang Denk wrote: Are you going to implement support for multiple I2C busses on some boards? I'm working on something like this now so it would be nice to coordinate our efforts somehow...
There is a provision for using 2 I2C busses in fsl-i2c.c now but I'm trying to make it more uniform to allow for different controllers on the same board. I have the first prototype of our MPC8548E based motherboard in PCB fab right now and I do have 3 I2C busses on it all of which are required for U-Boot. That is 2 busses off of 2 MPC8548E I2C controllers and the third one off of Silicon Motion SM502 MFD. I have SM502 I2C driver written but not tested (no real hardware to test it on yet, will be ready in something like a month) but the entire multibus multicontroller I2C framework is not in U-Boot yet. Are you working on something like this or just on moving i2c_reg_read() and i2c_reg_write() functions to inlines? > Dear Timur Tabi, > > In message <1228325310-19275-1-git-send-email-ti...@freescale.com> you > wrote: >> All implementations of the functions i2c_reg_read() and > i2c_reg_write() are >> identical. We can save space and simplify the code by converting > these >> functions into inlines and putting them in i2c.h. >> >> Signed-off-by: Timur Tabi <ti...@freescale.com> >> --- >> >> v3: add 8xx-specific code, and include PRINTDs from blackfin and pxa >> >> I'm posting this patch because I'm enhancing the I2C routines to > support >> multiple I2C busses more easily, but I need to clean up the existing > code >> first. >> >> I'm going to be on vacation when the next merge window opens, so I'm > posting >> this now in the hopes that it will be picked up when the window does > open. > > We've been discussing this long enough when the previous window was > open, so ... > >> cpu/arm920t/at91rm9200/i2c.c | 14 --------- >> cpu/arm926ejs/davinci/i2c.c | 17 ----------- >> cpu/blackfin/i2c.c | 16 ---------- >> cpu/mpc512x/i2c.c | 17 ----------- >> cpu/mpc5xxx/i2c.c | 16 ---------- >> cpu/mpc8220/i2c.c | 16 ---------- >> cpu/mpc824x/drivers/i2c/i2c.c | 14 --------- >> cpu/mpc8260/i2c.c | 16 ---------- >> cpu/mpc8xx/i2c.c | 33 --------------------- >> cpu/ppc4xx/i2c.c | 20 ------------- >> cpu/pxa/i2c.c | 15 ---------- >> drivers/i2c/fsl_i2c.c | 16 ---------- >> drivers/i2c/soft_i2c.c | 19 ------------ >> include/i2c.h | 62 > +++++++++++++++++++++++++++++++++++++++- >> 14 files changed, 60 insertions(+), 231 deletions(-) > > Applied, thanks. > > 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: w...@denx.de > All these theories, diverse as they are, have two things in common: > they explain the observed facts, and they are completeley and utterly > wrong. - Terry Pratchett, _The Light Fantastic_ > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > --- ****************************************************************** * k...@home KOI8 Net < > The impossible we do immediately. * * Las Vegas NV, USA < > Miracles require 24-hour notice. * ****************************************************************** _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot