Pavel Roskin <[EMAIL PROTECTED]> :
[...]
> diff --git a/drivers/net/wireless/orinoco_pci.c
> b/drivers/net/wireless/orinoco_pci.c
> index e57e92b..75df90f 100644
> --- a/drivers/net/wireless/orinoco_pci.c
> +++ b/drivers/net/wireless/orinoco_pci.c
> @@ -170,9 +170,7 @@ static int orinoco_pci_init_one(struct p
> const struct pci_device_id *ent)
> {
> int err = 0;
> - unsigned long pci_iorange;
> - u16 __iomem *pci_ioaddr = NULL;
> - unsigned long pci_iolen;
> + void __iomem *pci_ioaddr = NULL;
> struct orinoco_private *priv = NULL;
> struct orinoco_pci_card *card;
> struct net_device *dev = NULL;
(ok, the useless initializers disappear later in the serie)
[...]
> @@ -208,8 +205,8 @@ static int orinoco_pci_init_one(struct p
> priv = netdev_priv(dev);
> card = priv->card;
> card->pci_ioaddr = pci_ioaddr;
> - dev->mem_start = pci_iorange;
> - dev->mem_end = pci_iorange + pci_iolen - 1;
> + dev->mem_start = pci_resource_start(pdev, 0);
> + dev->mem_end = dev->mem_start + pci_resource_len(pdev, 0) - 1;
Is there a reason why dev->mem_{start/end} should not be removed ?
--
Ueimor
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
