On Thu, Feb 26, 2015 at 11:41:54PM +0200, Antti Palosaari wrote: > From: Jurgen Kramer <gtmkramer@xxxxxxxxx> > > Increase si2168 cmd execute timeout to prevent firmware load failures. Tests > shows it takes up to 52ms to load the 'dvb-demod-si2168-a30-01.fw' firmware. > Increase timeout to a safe value of 70ms. > > Cc: <stable@xxxxxxxxxxxxxxx> # v3.16+ > Signed-off-by: Jurgen Kramer <gtmkramer@xxxxxxxxx> > Reviewed-by: Antti Palosaari <crope@xxxxxx> > Signed-off-by: Antti Palosaari <crope@xxxxxx> > --- > Changes since v1: > * I added my SOB > > Patch for stable 3.16+ > > That patch is already applied to master as commit 551c33e729f654ecfaed00ad399f5d2a631b72cb > There was some mistake and Cc stable tag I added to patchwork [1] was lost. > Thank you Antti, I'll queue this for the 3.16 kernel. Cheers, -- Luís > [1] https://patchwork.linuxtv.org/patch/27382/ > --- > drivers/media/dvb-frontends/si2168.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/dvb-frontends/si2168.c b/drivers/media/dvb-frontends/si2168.c > index 2e3cdcf..fbc1fa8 100644 > --- a/drivers/media/dvb-frontends/si2168.c > +++ b/drivers/media/dvb-frontends/si2168.c > @@ -39,7 +39,7 @@ static int si2168_cmd_execute(struct si2168 *s, struct si2168_cmd *cmd) > > if (cmd->rlen) { > /* wait cmd execution terminate */ > - #define TIMEOUT 50 > + #define TIMEOUT 70 > timeout = jiffies + msecs_to_jiffies(TIMEOUT); > while (!time_after(jiffies, timeout)) { > ret = i2c_master_recv(s->client, cmd->args, cmd->rlen); > -- > http://palosaari.fi/ > > -- > To unsubscribe from this list: send the line "unsubscribe stable" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
