Skip to content

Commit

Permalink
[SERIAL] serial_cs: Use clean up multiport card detection
Browse files Browse the repository at this point in the history
- 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>
  • Loading branch information
Russell King authored and Russell King committed Oct 1, 2006
1 parent de6cc84 commit 43549ad
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions drivers/serial/serial_cs.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 43549ad

Please sign in to comment.