Hi Alexey, On Wed, 2014-01-15 at 15:30 +0000, Alexey Brodkin wrote: > On Wed, 2014-01-15 at 08:58 -0600, Chin Liang See wrote: > > Enhance the DesignWare I2C driver to support address length more > > than 1 byte. This enhancement is required as some I2C slave > > device such as EEPROM chip might have 16 bit address byte. > > diff --git a/drivers/i2c/designware_i2c.c b/drivers/i2c/designware_i2c.c > > index cb2ac04..c0ac5f7 100644 > > --- a/drivers/i2c/designware_i2c.c > > +++ b/drivers/i2c/designware_i2c.c > > @@ -205,27 +205,21 @@ static int check_params(uint addr, int alen, uchar > > *buffer, int len) > > return 1; > > } > > > > - if (alen > 1) { > > - printf("addr len %d not supported\n", alen); > > - return 1; > > - } > > - > > - if (addr + len > 256) { > > - printf("address out of range\n"); > > - return 1; > > - } > > - > > return 0; > > } > > Hi Chin, > > if you strip down functionality of "check_params()" to one single check > I would recommend you to remove "check_params()" at all and do in-place > check for "buffer" existence. > > Moreover you may just use "assert" for this check because this buffer is > passed by u-boot (no need to check every parameter passed to any > function in run-time) so in production/release build it won't exist at > all.
Good suggestion. I agreed that we can strip off the check_params. Let me send the v2 patch. Thanks and have a nice day! Chin Liang > > Regards, > Alexey > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot