]> err.no Git - linux-2.6/commitdiff
[SERIAL] serial_cs: Use clean up multiport card detection
authorRussell King <rmk@dyn-67.arm.linux.org.uk>
Sat, 16 Sep 2006 19:53:05 +0000 (20:53 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Sun, 1 Oct 2006 16:06:20 +0000 (17:06 +0100)
- Use ARRAY_SIZE() instead of home grown based version.
- use parse->manfid.card rather than le16_to_cpu(buf[1]) -
  manfid.card is already converted to this format.
- use info->prodid in subsequent tests rather than
  parse->manfid.card.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
drivers/serial/serial_cs.c

index cbf260bc225dc6c5bcd41758f22bb49bd3809550..82bb603fc1ae4089e2d772542ac24faf39f34da5 100644 (file)
@@ -95,7 +95,6 @@ static const struct multi_id multi_id[] = {
        { MANFID_INTEL,   PRODID_INTEL_DUAL_RS232,      2 },
        { MANFID_NATINST, PRODID_NATINST_QUAD_RS232,    4 }
 };
-#define MULTI_COUNT (sizeof(multi_id)/sizeof(struct multi_id))
 
 struct serial_info {
        struct pcmcia_device    *p_dev;
@@ -622,13 +621,13 @@ static int serial_config(struct pcmcia_device * link)
        tuple->DesiredTuple = CISTPL_MANFID;
        if (first_tuple(link, tuple, parse) == CS_SUCCESS) {
                info->manfid = parse->manfid.manf;
-               info->prodid = le16_to_cpu(buf[1]);
-               for (i = 0; i < MULTI_COUNT; i++)
+               info->prodid = parse->manfid.card;
+               for (i = 0; i < ARRAY_SIZE(multi_id); i++)
                        if ((info->manfid == multi_id[i].manfid) &&
-                           (parse->manfid.card == multi_id[i].prodid))
+                           (info->prodid == multi_id[i].prodid)) {
+                               info->multi = multi_id[i].multi;
                                break;
-               if (i < MULTI_COUNT)
-                       info->multi = multi_id[i].multi;
+                       }
        }
 
        /* Another check for dual-serial cards: look for either serial or