aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorHarvey Harrison <harvey.harrison@gmail.com>2008-02-15 16:41:32 -0500
committerJeff Garzik <jeff@garzik.org>2008-02-20 12:12:47 -0500
commit5d44b414daa8c1d8551aed6130d86d54175db43f (patch)
tree8d43b2f5fcdd228ecbf5c7e3fc550d49649ba84d /include
parent332673257056d8be8a69d759eda90a799af5472d (diff)
ata: fix sparse warning in libata.h
Avoids lots of these, also is more readable. include/linux/libata.h:1210:13: warning: potentially expensive pointer subtraction Change the subtraction to addition on the other side of the comparison. Thanks to Christer Weinigel for the suggestion. Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> Acked-by: Tejun Heo <htejun@gmail.com> Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'include')
-rw-r--r--include/linux/libata.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/libata.h b/include/linux/libata.h
index 2e098f940cec..ce7603a73156 100644
--- a/include/linux/libata.h
+++ b/include/linux/libata.h
@@ -1197,7 +1197,7 @@ static inline struct ata_link *ata_port_next_link(struct ata_link *link)
1197 return ap->pmp_link; 1197 return ap->pmp_link;
1198 } 1198 }
1199 1199
1200 if (++link - ap->pmp_link < ap->nr_pmp_links) 1200 if (++link < ap->nr_pmp_links + ap->pmp_link)
1201 return link; 1201 return link;
1202 return NULL; 1202 return NULL;
1203} 1203}