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