On 17/01/2019 20:39, Sowjanya Komatineni wrote: > increase transfer timeout to 10s to allow enough time during max > transfer size. > > Signed-off-by: Sowjanya Komatineni <skomatin...@nvidia.com> > --- > drivers/i2c/busses/i2c-tegra.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c > index e417ebf7628c..ca7c581fb4c0 100644 > --- a/drivers/i2c/busses/i2c-tegra.c > +++ b/drivers/i2c/busses/i2c-tegra.c > @@ -25,7 +25,7 @@ > > #include <asm/unaligned.h> > > -#define TEGRA_I2C_TIMEOUT (msecs_to_jiffies(1000)) > +#define TEGRA_I2C_TIMEOUT (msecs_to_jiffies(10000)) > #define BYTES_PER_FIFO_WORD 4 > > #define I2C_CNFG 0x000
Should the timeout be set depending on the max transfer size? 10s seems an age if the max transfer size is 4KB. In other words, we should this only be applied for T194+? Furthermore, in tegra_i2c_xfer_msg() we know the len of the message and so maybe it would be better to dynamically set the timeout depending on length? Cheers Jon -- nvpublic
