diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-12-04 23:21:08 +0100 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2015-12-04 23:21:08 +0100 |
commit | 7f52f31443ae9b2e6a71ec54cfc5a2c89006ea27 (patch) | |
tree | e00588408eae2d9749a0d1cf374170f60f949ef8 /arch/um | |
parent | PCI/MSI: Only use the generic MSI layer when domain is hierarchical (diff) | |
download | linux-7f52f31443ae9b2e6a71ec54cfc5a2c89006ea27.tar.xz linux-7f52f31443ae9b2e6a71ec54cfc5a2c89006ea27.zip |
PCI: altera: Fix loop in tlp_read_packet()
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.
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@oracle.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Ley Foon Tan <lftan@altera.com>
Diffstat (limited to 'arch/um')
0 files changed, 0 insertions, 0 deletions