[patch 14/38] pcmcia: id_table for wavelan_cs
Jean Tourrilhes
jt at hpl.hp.com
Mon Mar 7 17:53:55 EST 2005
On Sun, Feb 27, 2005 at 05:13:08PM +0100, Dominik Brodowski wrote:
> Add pcmcia_device_id table.
>
> Signed-off-by: Dominik Brodowski <linux at dominikbrodowski.net>
Hi,
Sorry for the late reply...
> +static struct pcmcia_device_id wavelan_ids[] = {
> + PCMCIA_DEVICE_PROD_ID1("AT", 0x8564eba9),
This one seems bogus and won't match anything useful.
On the other hand, this one seems missing :
+ PCMCIA_DEVICE_PROD_ID12("AT&T", "WaveLAN/PCMCIA", 0x??????, 0x??????),
Also, what's up with the magic numbers ? Couldn't they be
autogenerated by the macro ? I guess they are a hash based on the
strings. Like this, it seems quite prone to mistyping and other stupid
cut'n'paste problems, and not very author friendly...
> + PCMCIA_DEVICE_PROD_ID12("Digital", "RoamAbout/DS", 0x9999ab35, 0x00d05e06),
> + PCMCIA_DEVICE_PROD_ID12("Lucent Technologies", "WaveLAN/PCMCIA", 0x23eb9949, 0x1bc50975),
> + PCMCIA_DEVICE_PROD_ID12("NCR", "WaveLAN/PCMCIA", 0x24358cd4, 0x1bc50975),
All good (I did not check the magic numbers).
Jean
More information about the linux-pcmcia
mailing list