Hello Dirk, On 24.04.2013 15:00, Dirk Eibach wrote: > Hello Heiko, > >> On 24.04.2013 13:43, Dirk Eibach wrote: >>> Signed-off-by: Dirk Eibach <dirk.eib...@gdsys.cc> >>> Cc: Heiko Schocher <h...@denx.de> >>> Cc: Stefan Roese <s...@denx.de> >>> --- >>> drivers/i2c/Makefile | 2 +- >>> drivers/i2c/ppc4xx_i2c.c | 193 >>> +++++++++++++++++++++++----------------------- >>> 2 files changed, 97 insertions(+), 98 deletions(-) >> >> Did you tried this patch with the mutlibus/multiadapter >> framework? > > sure. This RFC is based on 20130416_multibus_v2.
Great! :-) Also with soft_i2c driver in use? >> When switching one i2c driver to this new Framework, you >> must adapt all config files, which use this driver ... >> Sorry, some stupid work, but it must be done, without >> this I will bet, MAKEALL fails ... > > Certainly. But before starting this, I wanted to make sure, that the > basic implementation is allright. Yep, looks fine too me. >> [...] >>> diff --git a/drivers/i2c/ppc4xx_i2c.c b/drivers/i2c/ppc4xx_i2c.c >>> index 53fedd5..c924874 100644 >>> --- a/drivers/i2c/ppc4xx_i2c.c >>> +++ b/drivers/i2c/ppc4xx_i2c.c >> [...] >>> +#ifdef CONFIG_SYS_I2C_PPC4XX_CH0 >> >> You introduce here new defines, please document them in >> the README, thanks! > > OK. Thanks! >>> +U_BOOT_I2C_ADAP_COMPLETE(ppc4xx_0, ppc4xx_i2c_init, >>> ppc4xx_i2c_probe, >>> + ppc4xx_i2c_read, ppc4xx_i2c_write, >>> + ppc4xx_i2c_set_bus_speed, >>> + CONFIG_SYS_I2C_PPC4XX_SPEED_0, >>> + CONFIG_SYS_I2C_PPC4XX_SLAVE_0, 0) >>> +#endif >>> +#ifdef CONFIG_SYS_I2C_PPC4XX_CH1 >>> +U_BOOT_I2C_ADAP_COMPLETE(ppc4xx_1, ppc4xx_i2c_init, >>> ppc4xx_i2c_probe, >>> + ppc4xx_i2c_read, ppc4xx_i2c_write, >>> + ppc4xx_i2c_set_bus_speed, >>> + CONFIG_SYS_I2C_PPC4XX_SPEED_1, >>> + CONFIG_SYS_I2C_PPC4XX_SLAVE_1, 1) >>> +#endif >> >> beside of this, your patch looks good ... > > Fine, so I will start adapting all dependent boards. Should the patch > be based on 20130416_multibus_v2 or should I post a series based on > v2013.04 that includes the 20130416_multibus_v2 changes? Against 20130416_multibus_v2 and if the merge window opens I post this patch with the other patches for this new i2c framework on the ML. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot