Re: [PATCH] wilc1000: fix a loop timeout condition

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:

> If the loop fails, the "while(trials--) {" loop will exit with "trials"
> set to -1.  The test for that expects it to end with "trials" set to 0
> so the warning message will not be printed.
> 
> Fix this by changing from a post-op to a pre-op.  This does mean that
> we only make 99 attempts instead of 100 but that's okay.
> 
> Fixes: f135a1571a05 ("wilc1000: Support chip sleep over SPI")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Acked-by: Ajay Singh <ajay.kathat@xxxxxxxxxxxxx>

Patch applied to wireless-drivers-next.git, thanks.

2f51061edab9 wilc1000: fix a loop timeout condition

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/YFS5gx/gi70zlIaO@mwanda/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux