[PATCH] hostap: Use void *hw_priv instead of #ifdef in local data

Replace hardware model specific #ifdef's in struct local_info with
void *hw_priv that is pointing to cs/pci/plx specific data
structure. This removes unneeded #ifdef's and as such, is a step
towards making it possible to share objects for hostap_hw.c and
hostap_download.c with cs/pci/plx drivers without having to compile
and link the same code separately for each one.
Signed-off-by: Jouni Malinen <jkmaline@cc.hut.fi>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>

drivers/net/wireless/hostap/hostap_pci.c

@@ -34,6 +34,12 @@ MODULE_LICENSE("GPL");
 MODULE_VERSION(PRISM2_VERSION);
 
 
+/* struct local_info::hw_priv */
+struct hostap_pci_priv {
+	void __iomem *mem_start;
+};
+
+
 /* FIX: do we need mb/wmb/rmb with memory operations? */
 
 
@@ -61,7 +67,7 @@ static inline void hfa384x_outb_debug(struct net_device *dev, int a, u8 v)
 
 	spin_lock_irqsave(&local->lock, flags);
 	prism2_io_debug_add(dev, PRISM2_IO_DEBUG_CMD_OUTB, a, v);
-	writeb(v, local->mem_start + a);
+	writeb(v, hw_priv->mem_start + a);
 	spin_unlock_irqrestore(&local->lock, flags);
 }
 
@@ -76,7 +82,7 @@ static inline u8 hfa384x_inb_debug(struct net_device *dev, int a)
 	local = iface->local;
 
 	spin_lock_irqsave(&local->lock, flags);
-	v = readb(local->mem_start + a);
+	v = readb(hw_priv->mem_start + a);
 	prism2_io_debug_add(dev, PRISM2_IO_DEBUG_CMD_INB, a, v);
 	spin_unlock_irqrestore(&local->lock, flags);
 	return v;
@@ -93,7 +99,7 @@ static inline void hfa384x_outw_debug(struct net_device *dev, int a, u16 v)
 
 	spin_lock_irqsave(&local->lock, flags);
 	prism2_io_debug_add(dev, PRISM2_IO_DEBUG_CMD_OUTW, a, v);
-	writew(v, local->mem_start + a);
+	writew(v, hw_priv->mem_start + a);
 	spin_unlock_irqrestore(&local->lock, flags);
 }
 
@@ -108,7 +114,7 @@ static inline u16 hfa384x_inw_debug(struct net_device *dev, int a)
 	local = iface->local;
 
 	spin_lock_irqsave(&local->lock, flags);
-	v = readw(local->mem_start + a);
+	v = readw(hw_priv->mem_start + a);
 	prism2_io_debug_add(dev, PRISM2_IO_DEBUG_CMD_INW, a, v);
 	spin_unlock_irqrestore(&local->lock, flags);
 	return v;
@@ -126,37 +132,37 @@ static inline u16 hfa384x_inw_debug(struct net_device *dev, int a)
 static inline void hfa384x_outb(struct net_device *dev, int a, u8 v)
 {
 	struct hostap_interface *iface;
-	local_info_t *local;
+	struct hostap_pci_priv *hw_priv;
 	iface = netdev_priv(dev);
-	local = iface->local;
-	writeb(v, local->mem_start + a);
+	hw_priv = iface->local->hw_priv;
+	writeb(v, hw_priv->mem_start + a);
 }
 
 static inline u8 hfa384x_inb(struct net_device *dev, int a)
 {
 	struct hostap_interface *iface;
-	local_info_t *local;
+	struct hostap_pci_priv *hw_priv;
 	iface = netdev_priv(dev);
-	local = iface->local;
-	return readb(local->mem_start + a);
+	hw_priv = iface->local->hw_priv;
+	return readb(hw_priv->mem_start + a);
 }
 
 static inline void hfa384x_outw(struct net_device *dev, int a, u16 v)
 {
 	struct hostap_interface *iface;
-	local_info_t *local;
+	struct hostap_pci_priv *hw_priv;
 	iface = netdev_priv(dev);
-	local = iface->local;
-	writew(v, local->mem_start + a);
+	hw_priv = iface->local->hw_priv;
+	writew(v, hw_priv->mem_start + a);
 }
 
 static inline u16 hfa384x_inw(struct net_device *dev, int a)
 {
 	struct hostap_interface *iface;
-	local_info_t *local;
+	struct hostap_pci_priv *hw_priv;
 	iface = netdev_priv(dev);
-	local = iface->local;
-	return readw(local->mem_start + a);
+	hw_priv = iface->local->hw_priv;
+	return readw(hw_priv->mem_start + a);
 }
 
 #define HFA384X_OUTB(v,a) hfa384x_outb(dev, (a), (v))
@@ -288,6 +294,12 @@ static int prism2_pci_probe(struct pci_dev *pdev,
 	static int cards_found /* = 0 */;
 	int irq_registered = 0;
 	struct hostap_interface *iface;
+	struct hostap_pci_priv *hw_priv;
+
+	hw_priv = kmalloc(sizeof(*hw_priv), GFP_KERNEL);
+	if (hw_priv == NULL)
+		return -ENOMEM;
+	memset(hw_priv, 0, sizeof(*hw_priv));
 
 	if (pci_enable_device(pdev))
 		return -EIO;
@@ -311,10 +323,11 @@ static int prism2_pci_probe(struct pci_dev *pdev,
 		goto fail;
 	iface = netdev_priv(dev);
 	local = iface->local;
+	local->hw_priv = hw_priv;
 	cards_found++;
 
         dev->irq = pdev->irq;
-        local->mem_start = mem;
+        hw_priv->mem_start = mem;
 
 	prism2_pci_cor_sreset(local);
 
@@ -339,6 +352,8 @@ static int prism2_pci_probe(struct pci_dev *pdev,
 	return hostap_hw_ready(dev);
 
  fail:
+	kfree(hw_priv);
+
 	if (irq_registered && dev)
 		free_irq(dev->irq, dev);
 
@@ -349,6 +364,9 @@ static int prism2_pci_probe(struct pci_dev *pdev,
 
  err_out_disable:
 	pci_disable_device(pdev);
+	kfree(hw_priv);
+	if (local)
+		local->hw_priv = NULL;
 	prism2_free_local_data(dev);
 
 	return -ENODEV;
@@ -360,9 +378,11 @@ static void prism2_pci_remove(struct pci_dev *pdev)
 	struct net_device *dev;
 	struct hostap_interface *iface;
 	void __iomem *mem_start;
+	struct hostap_pci_priv *hw_priv;
 
 	dev = pci_get_drvdata(pdev);
 	iface = netdev_priv(dev);
+	hw_priv = iface->local->hw_priv;
 
 	/* Reset the hardware, and ensure interrupts are disabled. */
 	prism2_pci_cor_sreset(iface->local);
@@ -371,7 +391,9 @@ static void prism2_pci_remove(struct pci_dev *pdev)
 	if (dev->irq)
 		free_irq(dev->irq, dev);
 
-	mem_start = iface->local->mem_start;
+	mem_start = hw_priv->mem_start;
+	kfree(hw_priv);
+	iface->local->hw_priv = NULL;
 	prism2_free_local_data(dev);
 
 	iounmap(mem_start);

drivers/net/wireless/hostap/hostap_plx.c

@@ -42,6 +42,13 @@ module_param(ignore_cis, int, 0444);
 MODULE_PARM_DESC(ignore_cis, "Do not verify manfid information in CIS");
 
 
+/* struct local_info::hw_priv */
+struct hostap_plx_priv {
+	void __iomem *attr_mem;
+	unsigned int cor_offset;
+};
+
+
 #define PLX_MIN_ATTR_LEN 512	/* at least 2 x 256 is needed for CIS */
 #define COR_SRESET       0x80
 #define COR_LEVLREQ      0x40
@@ -261,27 +268,28 @@ static int hfa384x_to_bap(struct net_device *dev, u16 bap, void *buf, int len)
 static void prism2_plx_cor_sreset(local_info_t *local)
 {
 	unsigned char corsave;
+	struct hostap_plx_priv *hw_priv = local->hw_priv;
 
 	printk(KERN_DEBUG "%s: Doing reset via direct COR access.\n",
 	       dev_info);
 
 	/* Set sreset bit of COR and clear it after hold time */
 
-	if (local->attr_mem == NULL) {
+	if (hw_priv->attr_mem == NULL) {
 		/* TMD7160 - COR at card's first I/O addr */
-		corsave = inb(local->cor_offset);
-		outb(corsave | COR_SRESET, local->cor_offset);
+		corsave = inb(hw_priv->cor_offset);
+		outb(corsave | COR_SRESET, hw_priv->cor_offset);
 		mdelay(2);
-		outb(corsave & ~COR_SRESET, local->cor_offset);
+		outb(corsave & ~COR_SRESET, hw_priv->cor_offset);
 		mdelay(2);
 	} else {
 		/* PLX9052 */
-		corsave = readb(local->attr_mem + local->cor_offset);
+		corsave = readb(hw_priv->attr_mem + hw_priv->cor_offset);
 		writeb(corsave | COR_SRESET,
-		       local->attr_mem + local->cor_offset);
+		       hw_priv->attr_mem + hw_priv->cor_offset);
 		mdelay(2);
 		writeb(corsave & ~COR_SRESET,
-		       local->attr_mem + local->cor_offset);
+		       hw_priv->attr_mem + hw_priv->cor_offset);
 		mdelay(2);
 	}
 }
@@ -290,26 +298,27 @@ static void prism2_plx_cor_sreset(local_info_t *local)
 static void prism2_plx_genesis_reset(local_info_t *local, int hcr)
 {
 	unsigned char corsave;
+	struct hostap_plx_priv *hw_priv = local->hw_priv;
 
-	if (local->attr_mem == NULL) {
+	if (hw_priv->attr_mem == NULL) {
 		/* TMD7160 - COR at card's first I/O addr */
-		corsave = inb(local->cor_offset);
-		outb(corsave | COR_SRESET, local->cor_offset);
+		corsave = inb(hw_priv->cor_offset);
+		outb(corsave | COR_SRESET, hw_priv->cor_offset);
 		mdelay(10);
-		outb(hcr, local->cor_offset + 2);
+		outb(hcr, hw_priv->cor_offset + 2);
 		mdelay(10);
-		outb(corsave & ~COR_SRESET, local->cor_offset);
+		outb(corsave & ~COR_SRESET, hw_priv->cor_offset);
 		mdelay(10);
 	} else {
 		/* PLX9052 */
-		corsave = readb(local->attr_mem + local->cor_offset);
+		corsave = readb(hw_priv->attr_mem + hw_priv->cor_offset);
 		writeb(corsave | COR_SRESET,
-		       local->attr_mem + local->cor_offset);
+		       hw_priv->attr_mem + hw_priv->cor_offset);
 		mdelay(10);
-		writeb(hcr, local->attr_mem + local->cor_offset + 2);
+		writeb(hcr, hw_priv->attr_mem + hw_priv->cor_offset + 2);
 		mdelay(10);
 		writeb(corsave & ~COR_SRESET,
-		       local->attr_mem + local->cor_offset);
+		       hw_priv->attr_mem + hw_priv->cor_offset);
 		mdelay(10);
 	}
 }
@@ -438,6 +447,12 @@ static int prism2_plx_probe(struct pci_dev *pdev,
 	static int cards_found /* = 0 */;
 	int irq_registered = 0;
 	int tmd7160;
+	struct hostap_plx_priv *hw_priv;
+
+	hw_priv = kmalloc(sizeof(*hw_priv), GFP_KERNEL);
+	if (hw_priv == NULL)
+		return -ENOMEM;
+	memset(hw_priv, 0, sizeof(*hw_priv));
 
 	if (pci_enable_device(pdev))
 		return -EIO;
@@ -529,12 +544,13 @@ static int prism2_plx_probe(struct pci_dev *pdev,
 		goto fail;
 	iface = netdev_priv(dev);
 	local = iface->local;
+	local->hw_priv = hw_priv;
 	cards_found++;
 
 	dev->irq = pdev->irq;
 	dev->base_addr = pccard_ioaddr;
-	local->attr_mem = attr_mem;
-	local->cor_offset = cor_offset;
+	hw_priv->attr_mem = attr_mem;
+	hw_priv->cor_offset = cor_offset;
 
 	pci_set_drvdata(pdev, dev);
 
@@ -554,6 +570,9 @@ static int prism2_plx_probe(struct pci_dev *pdev,
 	return hostap_hw_ready(dev);
 
  fail:
+	kfree(hw_priv);
+	if (local)
+		local->hw_priv = NULL;
 	prism2_free_local_data(dev);
 
 	if (irq_registered && dev)
@@ -572,19 +591,23 @@ static void prism2_plx_remove(struct pci_dev *pdev)
 {
 	struct net_device *dev;
 	struct hostap_interface *iface;
+	struct hostap_plx_priv *hw_priv;
 
 	dev = pci_get_drvdata(pdev);
 	iface = netdev_priv(dev);
+	hw_priv = iface->local->hw_priv;
 
 	/* Reset the hardware, and ensure interrupts are disabled. */
 	prism2_plx_cor_sreset(iface->local);
 	hfa384x_disable_interrupts(dev);
 
-	if (iface->local->attr_mem)
-		iounmap(iface->local->attr_mem);
+	if (hw_priv->attr_mem)
+		iounmap(hw_priv->attr_mem);
 	if (dev->irq)
 		free_irq(dev->irq, dev);
 
+	kfree(iface->local->hw_priv);
+	iface->local->hw_priv = NULL;
 	prism2_free_local_data(dev);
 	pci_disable_device(pdev);
 }

drivers/net/wireless/hostap/hostap_wlan.h

@@ -876,27 +876,8 @@ struct local_info {
 	int io_debug_enabled;
 #endif /* PRISM2_IO_DEBUG */
 
-	/* struct local_info is used also in hostap.o that does not define
-	 * any PRISM2_{PCCARD,PLX,PCI}. Make sure that the hardware version
-	 * specific fields are in the end of the struct (these could also be
-	 * moved to void *priv or something like that). */
-#ifdef PRISM2_PCCARD
-	dev_node_t node;
-	dev_link_t *link;
-	int sandisk_connectplus;
-#endif /* PRISM2_PCCARD */
-
-#ifdef PRISM2_PLX
-	void __iomem *attr_mem;
-	unsigned int cor_offset;
-#endif /* PRISM2_PLX */
-
-#ifdef PRISM2_PCI
-	void __iomem *mem_start;
-#endif /* PRISM2_PCI */
-
-	/* NOTE! Do not add common entries here after hardware version
-	 * specific blocks. */
+	/* Pointer to hardware model specific (cs,pci,plx) private data. */
+	void *hw_priv;
 };