Merge branch 'master' of master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6
[cascardo/linux.git] / drivers / net / vxge / vxge-main.c
index 0fdfd58..a6606b8 100644 (file)
@@ -54,7 +54,7 @@ MODULE_LICENSE("Dual BSD/GPL");
 MODULE_DESCRIPTION("Neterion's X3100 Series 10GbE PCIe I/O"
        "Virtualized Server Adapter");
 
-static struct pci_device_id vxge_id_table[] __devinitdata = {
+static DEFINE_PCI_DEVICE_TABLE(vxge_id_table) = {
        {PCI_VENDOR_ID_S2IO, PCI_DEVICE_ID_TITAN_WIN, PCI_ANY_ID,
        PCI_ANY_ID},
        {PCI_VENDOR_ID_S2IO, PCI_DEVICE_ID_TITAN_UNI, PCI_ANY_ID,
@@ -310,7 +310,7 @@ static int vxge_rx_map(void *dtrh, struct vxge_ring *ring)
        dma_addr = pci_map_single(ring->pdev, rx_priv->skb_data,
                                rx_priv->data_size, PCI_DMA_FROMDEVICE);
 
-       if (dma_addr == 0) {
+       if (unlikely(pci_dma_mapping_error(ring->pdev, dma_addr))) {
                ring->stats.pci_map_fail++;
                return -EIO;
        }
@@ -4297,10 +4297,8 @@ vxge_probe(struct pci_dev *pdev, const struct pci_device_id *pre)
        vxge_debug_init(VXGE_TRACE, "%s: Neterion %s Server Adapter",
                vdev->ndev->name, ll_config.device_hw_info.product_desc);
 
-       vxge_debug_init(VXGE_TRACE,
-               "%s: MAC ADDR: %02X:%02X:%02X:%02X:%02X:%02X",
-               vdev->ndev->name, macaddr[0], macaddr[1], macaddr[2],
-               macaddr[3], macaddr[4], macaddr[5]);
+       vxge_debug_init(VXGE_TRACE, "%s: MAC ADDR: %pM",
+               vdev->ndev->name, macaddr);
 
        vxge_debug_init(VXGE_TRACE, "%s: Link Width x%d",
                vdev->ndev->name, vxge_hw_device_link_width_get(hldev));