wimax/i2400m: remove debug containing bogus calculation of index

The subtraction of the two pointers is automatically scaled by the
size of the size of the object the pointers point to, so the division
by sizeof(*i2400m->barker) is incorrect.  This has been broken since
day one of the driver and is only debug, so remove the debug completely.

Also move && in condition to clean up a checkpatch warning.

Addresses-Coverity: ("Extra sizeof expression")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/wimax/i2400m/fw.c b/drivers/net/wimax/i2400m/fw.c
index 489cba9b284d1b3bf633853c2f1f5de57e1533c2..6c9a41bff2e0adf9046fa0dd4f8f1c6d5dd27d16 100644 (file)
--- a/drivers/net/wimax/i2400m/fw.c
+++ b/drivers/net/wimax/i2400m/fw.c
@@ -397,14 +397,9 @@ int i2400m_is_boot_barker(struct i2400m *i2400m,
 
        /* Short circuit if we have already discovered the barker
         * associated with the device. */
-       if (i2400m->barker
-           && !memcmp(buf, i2400m->barker, sizeof(i2400m->barker->data))) {
-               unsigned index = (i2400m->barker - i2400m_barker_db)
-                       / sizeof(*i2400m->barker);
-               d_printf(2, dev, "boot barker cache-confirmed #%u/%08x\n",
-                        index, le32_to_cpu(i2400m->barker->data[0]));
+       if (i2400m->barker &&
+           !memcmp(buf, i2400m->barker, sizeof(i2400m->barker->data)))
                return 0;
-       }
 
        for (i = 0; i < i2400m_barker_db_used; i++) {
                barker = &i2400m_barker_db[i];