Thread (24 messages) flat view 24 messages, 13 authors, 2011-06-16

Re: [PATCH net-next 2/2] drivers/net: Remove casts of void *

From: Daniele Venzano <venza@brownhat.org>
Date: 2011-06-14 08:23:52
Also in: lkml

2011/6/14 Joe Perches [off-list ref]:
Unnecessary casts of void * clutter the code.
quoted hunk ↗ jump to hunk
diff --git a/drivers/net/sis900.c b/drivers/net/sis900.c
index 484f795..658a192 100644
--- a/drivers/net/sis900.c
+++ b/drivers/net/sis900.c
@@ -482,7 +482,7 @@ static int __devinit sis900_probe(struct pci_dev *pci_dev,
               ret = -ENOMEM;
               goto err_out_cleardev;
       }
-       sis_priv->tx_ring = (BufferDesc *)ring_space;
+       sis_priv->tx_ring = ring_space;
       sis_priv->tx_ring_dma = ring_dma;

       ring_space = pci_alloc_consistent(pci_dev, RX_TOTAL_SIZE, &ring_dma);
@@ -490,7 +490,7 @@ static int __devinit sis900_probe(struct pci_dev *pci_dev,
               ret = -ENOMEM;
               goto err_unmap_tx;
       }
-       sis_priv->rx_ring = (BufferDesc *)ring_space;
+       sis_priv->rx_ring = ring_space;
       sis_priv->rx_ring_dma = ring_dma;

       /* The SiS900-specific entries in the device structure. */
sis900 looks good, too.

-- 
Daniele Venzano
venza@brownhat.org
http://www.brownhat.org
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help