On Wed, Nov 4, 2015 at 9:39 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > The problem is that TLP_LOOP is 500 and the "loop" variable was a u8 so > "loop < TLP_LOOP" is always true. We only need this condition to work > if there is a problem so it would have been easy to miss this in > testing. Anyway, lets just make it a normal for loop with "int i" > instead of over thinking things and making it complicated. > > Fixes: 6bb4dd154ae8 ('PCI: altera: Add Altera PCIe host controller driver') > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Acked-by: Ley Foon Tan <lftan@xxxxxxxxxx> Thanks for the fix. Regards Ley Foon -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html