diff options
author | Alejandro Martinez Ruiz <alex@flawedcode.org> | 2007-10-18 10:22:02 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 15:03:38 -0800 |
commit | ef878b889907f29af1b5f34734f14eca1aa43cf6 (patch) | |
tree | a30097123359ad444ad718564168adcd716f00e3 /arch/ia64/hp | |
parent | c00acf46deb18926931ba264510353cf22b98a79 (diff) | |
download | blackbird-op-linux-ef878b889907f29af1b5f34734f14eca1aa43cf6.tar.gz blackbird-op-linux-ef878b889907f29af1b5f34734f14eca1aa43cf6.zip |
sk98lin: kill bogus check and convert to use ARRAY_SIZE()
This converts uses of ARRAY_SIZE(), and while at it also kills
unreachable code as far as I can say. I can't tell what was the author
trying to do with the following check. First we have:
PNMI_STATIC const SK_PNMI_STATADDR
StatAddr[SK_PNMI_MAX_IDX][SK_PNMI_MAC_TYPES];
and then a check goes like this:
if (SK_PNMI_MAX_IDX !=
(sizeof(StatAddr) / (sizeof(SK_PNMI_STATADDR) * SK_PNMI_MAC_TYPES)))
with the second line being just ARRAY_SIZE(StatAddr), which will always
return SK_PNMI_MAX_IDX, rendering the check useless.
Signed-off-by: Alejandro Martinez Ruiz <alex@flawedcode.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'arch/ia64/hp')
0 files changed, 0 insertions, 0 deletions