diff options
author | Harvey Harrison <harvey.harrison@gmail.com> | 2008-02-15 16:41:32 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2008-02-20 12:12:47 -0500 |
commit | 5d44b414daa8c1d8551aed6130d86d54175db43f (patch) | |
tree | 8d43b2f5fcdd228ecbf5c7e3fc550d49649ba84d /include | |
parent | 332673257056d8be8a69d759eda90a799af5472d (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.h | 2 |
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 | } |