diff options
author | Joe Perches <joe@perches.com> | 2010-08-12 08:22:51 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-08-18 17:26:30 -0400 |
commit | 636b8116d4e116eca1ac978f546b16fadfc15a6c (patch) | |
tree | 287364907b68208a473472e9f6314b64ced16d3d /drivers/net/pcmcia/smc91c92_cs.c | |
parent | 00093fab980d0a8950a64bdf9e346d0497b9a7e4 (diff) |
drivers/net/pcmcia: Use pr_<level> and netdev_<level>
On Mon, 2010-08-09 at 17:34 +0200, Dominik Brodowski wrote:
> look good from a PCMCIA point of view, therefore:
> Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
If PCMCIA is still being looked after, then here's
another for you, maybe for 2.6.37.
Use the more descriptive logging message styles.
There are whitespace/indentation errors in the original
sources that these changes do not modify, so checkpatch
errors were cheerfully ignored.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/pcmcia/smc91c92_cs.c')
-rw-r--r-- | drivers/net/pcmcia/smc91c92_cs.c | 99 |
1 files changed, 45 insertions, 54 deletions
diff --git a/drivers/net/pcmcia/smc91c92_cs.c b/drivers/net/pcmcia/smc91c92_cs.c index 377367d03b4..cc7a357feac 100644 --- a/drivers/net/pcmcia/smc91c92_cs.c +++ b/drivers/net/pcmcia/smc91c92_cs.c | |||
@@ -25,6 +25,8 @@ | |||
25 | 25 | ||
26 | ======================================================================*/ | 26 | ======================================================================*/ |
27 | 27 | ||
28 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
29 | |||
28 | #include <linux/module.h> | 30 | #include <linux/module.h> |
29 | #include <linux/kernel.h> | 31 | #include <linux/kernel.h> |
30 | #include <linux/init.h> | 32 | #include <linux/init.h> |
@@ -820,7 +822,7 @@ static int check_sig(struct pcmcia_device *link) | |||
820 | modconf_t mod = { | 822 | modconf_t mod = { |
821 | .Attributes = CONF_IO_CHANGE_WIDTH, | 823 | .Attributes = CONF_IO_CHANGE_WIDTH, |
822 | }; | 824 | }; |
823 | printk(KERN_INFO "smc91c92_cs: using 8-bit IO window.\n"); | 825 | pr_info("using 8-bit IO window\n"); |
824 | 826 | ||
825 | smc91c92_suspend(link); | 827 | smc91c92_suspend(link); |
826 | pcmcia_modify_configuration(link, &mod); | 828 | pcmcia_modify_configuration(link, &mod); |
@@ -881,7 +883,7 @@ static int smc91c92_config(struct pcmcia_device *link) | |||
881 | if ((if_port >= 0) && (if_port <= 2)) | 883 | if ((if_port >= 0) && (if_port <= 2)) |
882 | dev->if_port = if_port; | 884 | dev->if_port = if_port; |
883 | else | 885 | else |
884 | printk(KERN_NOTICE "smc91c92_cs: invalid if_port requested\n"); | 886 | dev_notice(&link->dev, "invalid if_port requested\n"); |
885 | 887 | ||
886 | switch (smc->manfid) { | 888 | switch (smc->manfid) { |
887 | case MANFID_OSITECH: | 889 | case MANFID_OSITECH: |
@@ -899,7 +901,7 @@ static int smc91c92_config(struct pcmcia_device *link) | |||
899 | } | 901 | } |
900 | 902 | ||
901 | if (i != 0) { | 903 | if (i != 0) { |
902 | printk(KERN_NOTICE "smc91c92_cs: Unable to find hardware address.\n"); | 904 | dev_notice(&link->dev, "Unable to find hardware address.\n"); |
903 | goto config_failed; | 905 | goto config_failed; |
904 | } | 906 | } |
905 | 907 | ||
@@ -952,30 +954,28 @@ static int smc91c92_config(struct pcmcia_device *link) | |||
952 | SET_NETDEV_DEV(dev, &link->dev); | 954 | SET_NETDEV_DEV(dev, &link->dev); |
953 | 955 | ||
954 | if (register_netdev(dev) != 0) { | 956 | if (register_netdev(dev) != 0) { |
955 | printk(KERN_ERR "smc91c92_cs: register_netdev() failed\n"); | 957 | dev_err(&link->dev, "register_netdev() failed\n"); |
956 | goto config_undo; | 958 | goto config_undo; |
957 | } | 959 | } |
958 | 960 | ||
959 | printk(KERN_INFO "%s: smc91c%s rev %d: io %#3lx, irq %d, " | 961 | netdev_info(dev, "smc91c%s rev %d: io %#3lx, irq %d, hw_addr %pM\n", |
960 | "hw_addr %pM\n", | 962 | name, (rev & 0x0f), dev->base_addr, dev->irq, dev->dev_addr); |
961 | dev->name, name, (rev & 0x0f), dev->base_addr, dev->irq, | ||
962 | dev->dev_addr); | ||
963 | 963 | ||
964 | if (rev > 0) { | 964 | if (rev > 0) { |
965 | if (mir & 0x3ff) | 965 | if (mir & 0x3ff) |
966 | printk(KERN_INFO " %lu byte", mir); | 966 | netdev_info(dev, " %lu byte", mir); |
967 | else | 967 | else |
968 | printk(KERN_INFO " %lu kb", mir>>10); | 968 | netdev_info(dev, " %lu kb", mir>>10); |
969 | printk(" buffer, %s xcvr\n", (smc->cfg & CFG_MII_SELECT) ? | 969 | pr_cont(" buffer, %s xcvr\n", |
970 | "MII" : if_names[dev->if_port]); | 970 | (smc->cfg & CFG_MII_SELECT) ? "MII" : if_names[dev->if_port]); |
971 | } | 971 | } |
972 | 972 | ||
973 | if (smc->cfg & CFG_MII_SELECT) { | 973 | if (smc->cfg & CFG_MII_SELECT) { |
974 | if (smc->mii_if.phy_id != -1) { | 974 | if (smc->mii_if.phy_id != -1) { |
975 | dev_dbg(&link->dev, " MII transceiver at index %d, status %x.\n", | 975 | netdev_dbg(dev, " MII transceiver at index %d, status %x\n", |
976 | smc->mii_if.phy_id, j); | 976 | smc->mii_if.phy_id, j); |
977 | } else { | 977 | } else { |
978 | printk(KERN_NOTICE " No MII transceivers found!\n"); | 978 | netdev_notice(dev, " No MII transceivers found!\n"); |
979 | } | 979 | } |
980 | } | 980 | } |
981 | return 0; | 981 | return 0; |
@@ -1081,10 +1081,10 @@ static void smc_dump(struct net_device *dev) | |||
1081 | save = inw(ioaddr + BANK_SELECT); | 1081 | save = inw(ioaddr + BANK_SELECT); |
1082 | for (w = 0; w < 4; w++) { | 1082 | for (w = 0; w < 4; w++) { |
1083 | SMC_SELECT_BANK(w); | 1083 | SMC_SELECT_BANK(w); |
1084 | printk(KERN_DEBUG "bank %d: ", w); | 1084 | netdev_printk(KERN_DEBUG, dev, "bank %d: ", w); |
1085 | for (i = 0; i < 14; i += 2) | 1085 | for (i = 0; i < 14; i += 2) |
1086 | printk(" %04x", inw(ioaddr + i)); | 1086 | pr_cont(" %04x", inw(ioaddr + i)); |
1087 | printk("\n"); | 1087 | pr_cont("\n"); |
1088 | } | 1088 | } |
1089 | outw(save, ioaddr + BANK_SELECT); | 1089 | outw(save, ioaddr + BANK_SELECT); |
1090 | } | 1090 | } |
@@ -1106,7 +1106,7 @@ static int smc_open(struct net_device *dev) | |||
1106 | return -ENODEV; | 1106 | return -ENODEV; |
1107 | /* Physical device present signature. */ | 1107 | /* Physical device present signature. */ |
1108 | if (check_sig(link) < 0) { | 1108 | if (check_sig(link) < 0) { |
1109 | printk("smc91c92_cs: Yikes! Bad chip signature!\n"); | 1109 | netdev_info(dev, "Yikes! Bad chip signature!\n"); |
1110 | return -ENODEV; | 1110 | return -ENODEV; |
1111 | } | 1111 | } |
1112 | link->open++; | 1112 | link->open++; |
@@ -1172,7 +1172,7 @@ static void smc_hardware_send_packet(struct net_device * dev) | |||
1172 | u_char packet_no; | 1172 | u_char packet_no; |
1173 | 1173 | ||
1174 | if (!skb) { | 1174 | if (!skb) { |
1175 | printk(KERN_ERR "%s: In XMIT with no packet to send.\n", dev->name); | 1175 | netdev_err(dev, "In XMIT with no packet to send\n"); |
1176 | return; | 1176 | return; |
1177 | } | 1177 | } |
1178 | 1178 | ||
@@ -1180,8 +1180,8 @@ static void smc_hardware_send_packet(struct net_device * dev) | |||
1180 | packet_no = inw(ioaddr + PNR_ARR) >> 8; | 1180 | packet_no = inw(ioaddr + PNR_ARR) >> 8; |
1181 | if (packet_no & 0x80) { | 1181 | if (packet_no & 0x80) { |
1182 | /* If not, there is a hardware problem! Likely an ejected card. */ | 1182 | /* If not, there is a hardware problem! Likely an ejected card. */ |
1183 | printk(KERN_WARNING "%s: 91c92 hardware Tx buffer allocation" | 1183 | netdev_warn(dev, "hardware Tx buffer allocation failed, status %#2.2x\n", |
1184 | " failed, status %#2.2x.\n", dev->name, packet_no); | 1184 | packet_no); |
1185 | dev_kfree_skb_irq(skb); | 1185 | dev_kfree_skb_irq(skb); |
1186 | smc->saved_skb = NULL; | 1186 | smc->saved_skb = NULL; |
1187 | netif_start_queue(dev); | 1187 | netif_start_queue(dev); |
@@ -1200,8 +1200,7 @@ static void smc_hardware_send_packet(struct net_device * dev) | |||
1200 | u_char *buf = skb->data; | 1200 | u_char *buf = skb->data; |
1201 | u_int length = skb->len; /* The chip will pad to ethernet min. */ | 1201 | u_int length = skb->len; /* The chip will pad to ethernet min. */ |
1202 | 1202 | ||
1203 | pr_debug("%s: Trying to xmit packet of length %d.\n", | 1203 | netdev_dbg(dev, "Trying to xmit packet of length %d\n", length); |
1204 | dev->name, length); | ||
1205 | 1204 | ||
1206 | /* send the packet length: +6 for status word, length, and ctl */ | 1205 | /* send the packet length: +6 for status word, length, and ctl */ |
1207 | outw(0, ioaddr + DATA_1); | 1206 | outw(0, ioaddr + DATA_1); |
@@ -1233,9 +1232,8 @@ static void smc_tx_timeout(struct net_device *dev) | |||
1233 | struct smc_private *smc = netdev_priv(dev); | 1232 | struct smc_private *smc = netdev_priv(dev); |
1234 | unsigned int ioaddr = dev->base_addr; | 1233 | unsigned int ioaddr = dev->base_addr; |
1235 | 1234 | ||
1236 | printk(KERN_NOTICE "%s: SMC91c92 transmit timed out, " | 1235 | netdev_notice(dev, "transmit timed out, Tx_status %2.2x status %4.4x.\n", |
1237 | "Tx_status %2.2x status %4.4x.\n", | 1236 | inw(ioaddr)&0xff, inw(ioaddr + 2)); |
1238 | dev->name, inw(ioaddr)&0xff, inw(ioaddr + 2)); | ||
1239 | dev->stats.tx_errors++; | 1237 | dev->stats.tx_errors++; |
1240 | smc_reset(dev); | 1238 | smc_reset(dev); |
1241 | dev->trans_start = jiffies; /* prevent tx timeout */ | 1239 | dev->trans_start = jiffies; /* prevent tx timeout */ |
@@ -1254,14 +1252,14 @@ static netdev_tx_t smc_start_xmit(struct sk_buff *skb, | |||
1254 | 1252 | ||
1255 | netif_stop_queue(dev); | 1253 | netif_stop_queue(dev); |
1256 | 1254 | ||
1257 | pr_debug("%s: smc_start_xmit(length = %d) called," | 1255 | netdev_dbg(dev, "smc_start_xmit(length = %d) called, status %04x\n", |
1258 | " status %4.4x.\n", dev->name, skb->len, inw(ioaddr + 2)); | 1256 | skb->len, inw(ioaddr + 2)); |
1259 | 1257 | ||
1260 | if (smc->saved_skb) { | 1258 | if (smc->saved_skb) { |
1261 | /* THIS SHOULD NEVER HAPPEN. */ | 1259 | /* THIS SHOULD NEVER HAPPEN. */ |
1262 | dev->stats.tx_aborted_errors++; | 1260 | dev->stats.tx_aborted_errors++; |
1263 | printk(KERN_DEBUG "%s: Internal error -- sent packet while busy.\n", | 1261 | netdev_printk(KERN_DEBUG, dev, |
1264 | dev->name); | 1262 | "Internal error -- sent packet while busy\n"); |
1265 | return NETDEV_TX_BUSY; | 1263 | return NETDEV_TX_BUSY; |
1266 | } | 1264 | } |
1267 | smc->saved_skb = skb; | 1265 | smc->saved_skb = skb; |
@@ -1269,7 +1267,7 @@ static netdev_tx_t smc_start_xmit(struct sk_buff *skb, | |||
1269 | num_pages = skb->len >> 8; | 1267 | num_pages = skb->len >> 8; |
1270 | 1268 | ||
1271 | if (num_pages > 7) { | 1269 | if (num_pages > 7) { |
1272 | printk(KERN_ERR "%s: Far too big packet error.\n", dev->name); | 1270 | netdev_err(dev, "Far too big packet error: %d pages\n", num_pages); |
1273 | dev_kfree_skb (skb); | 1271 | dev_kfree_skb (skb); |
1274 | smc->saved_skb = NULL; | 1272 | smc->saved_skb = NULL; |
1275 | dev->stats.tx_dropped++; | 1273 | dev->stats.tx_dropped++; |
@@ -1339,8 +1337,7 @@ static void smc_tx_err(struct net_device * dev) | |||
1339 | } | 1337 | } |
1340 | 1338 | ||
1341 | if (tx_status & TS_SUCCESS) { | 1339 | if (tx_status & TS_SUCCESS) { |
1342 | printk(KERN_NOTICE "%s: Successful packet caused error " | 1340 | netdev_notice(dev, "Successful packet caused error interrupt?\n"); |
1343 | "interrupt?\n", dev->name); | ||
1344 | } | 1341 | } |
1345 | /* re-enable transmit */ | 1342 | /* re-enable transmit */ |
1346 | SMC_SELECT_BANK(0); | 1343 | SMC_SELECT_BANK(0); |
@@ -1530,8 +1527,7 @@ static void smc_rx(struct net_device *dev) | |||
1530 | /* Assertion: we are in Window 2. */ | 1527 | /* Assertion: we are in Window 2. */ |
1531 | 1528 | ||
1532 | if (inw(ioaddr + FIFO_PORTS) & FP_RXEMPTY) { | 1529 | if (inw(ioaddr + FIFO_PORTS) & FP_RXEMPTY) { |
1533 | printk(KERN_ERR "%s: smc_rx() with nothing on Rx FIFO.\n", | 1530 | netdev_err(dev, "smc_rx() with nothing on Rx FIFO\n"); |
1534 | dev->name); | ||
1535 | return; | 1531 | return; |
1536 | } | 1532 | } |
1537 | 1533 | ||
@@ -1646,8 +1642,7 @@ static int s9k_config(struct net_device *dev, struct ifmap *map) | |||
1646 | else if (map->port > 2) | 1642 | else if (map->port > 2) |
1647 | return -EINVAL; | 1643 | return -EINVAL; |
1648 | dev->if_port = map->port; | 1644 | dev->if_port = map->port; |
1649 | printk(KERN_INFO "%s: switched to %s port\n", | 1645 | netdev_info(dev, "switched to %s port\n", if_names[dev->if_port]); |
1650 | dev->name, if_names[dev->if_port]); | ||
1651 | smc_reset(dev); | 1646 | smc_reset(dev); |
1652 | } | 1647 | } |
1653 | return 0; | 1648 | return 0; |
@@ -1798,7 +1793,7 @@ static void media_check(u_long arg) | |||
1798 | this, we can limp along even if the interrupt is blocked */ | 1793 | this, we can limp along even if the interrupt is blocked */ |
1799 | if (smc->watchdog++ && ((i>>8) & i)) { | 1794 | if (smc->watchdog++ && ((i>>8) & i)) { |
1800 | if (!smc->fast_poll) | 1795 | if (!smc->fast_poll) |
1801 | printk(KERN_INFO "%s: interrupt(s) dropped!\n", dev->name); | 1796 | netdev_info(dev, "interrupt(s) dropped!\n"); |
1802 | local_irq_save(flags); | 1797 | local_irq_save(flags); |
1803 | smc_interrupt(dev->irq, dev); | 1798 | smc_interrupt(dev->irq, dev); |
1804 | local_irq_restore(flags); | 1799 | local_irq_restore(flags); |
@@ -1822,7 +1817,7 @@ static void media_check(u_long arg) | |||
1822 | SMC_SELECT_BANK(3); | 1817 | SMC_SELECT_BANK(3); |
1823 | link = mdio_read(dev, smc->mii_if.phy_id, 1); | 1818 | link = mdio_read(dev, smc->mii_if.phy_id, 1); |
1824 | if (!link || (link == 0xffff)) { | 1819 | if (!link || (link == 0xffff)) { |
1825 | printk(KERN_INFO "%s: MII is missing!\n", dev->name); | 1820 | netdev_info(dev, "MII is missing!\n"); |
1826 | smc->mii_if.phy_id = -1; | 1821 | smc->mii_if.phy_id = -1; |
1827 | goto reschedule; | 1822 | goto reschedule; |
1828 | } | 1823 | } |
@@ -1830,15 +1825,13 @@ static void media_check(u_long arg) | |||
1830 | link &= 0x0004; | 1825 | link &= 0x0004; |
1831 | if (link != smc->link_status) { | 1826 | if (link != smc->link_status) { |
1832 | u_short p = mdio_read(dev, smc->mii_if.phy_id, 5); | 1827 | u_short p = mdio_read(dev, smc->mii_if.phy_id, 5); |
1833 | printk(KERN_INFO "%s: %s link beat\n", dev->name, | 1828 | netdev_info(dev, "%s link beat\n", link ? "found" : "lost"); |
1834 | (link) ? "found" : "lost"); | ||
1835 | smc->duplex = (((p & 0x0100) || ((p & 0x1c0) == 0x40)) | 1829 | smc->duplex = (((p & 0x0100) || ((p & 0x1c0) == 0x40)) |
1836 | ? TCR_FDUPLX : 0); | 1830 | ? TCR_FDUPLX : 0); |
1837 | if (link) { | 1831 | if (link) { |
1838 | printk(KERN_INFO "%s: autonegotiation complete: " | 1832 | netdev_info(dev, "autonegotiation complete: " |
1839 | "%sbaseT-%cD selected\n", dev->name, | 1833 | "%dbaseT-%cD selected\n", |
1840 | ((p & 0x0180) ? "100" : "10"), | 1834 | (p & 0x0180) ? 100 : 10, smc->duplex ? 'F' : 'H'); |
1841 | (smc->duplex ? 'F' : 'H')); | ||
1842 | } | 1835 | } |
1843 | SMC_SELECT_BANK(0); | 1836 | SMC_SELECT_BANK(0); |
1844 | outw(inw(ioaddr + TCR) | smc->duplex, ioaddr + TCR); | 1837 | outw(inw(ioaddr + TCR) | smc->duplex, ioaddr + TCR); |
@@ -1857,25 +1850,23 @@ static void media_check(u_long arg) | |||
1857 | if (media != smc->media_status) { | 1850 | if (media != smc->media_status) { |
1858 | if ((media & smc->media_status & 1) && | 1851 | if ((media & smc->media_status & 1) && |
1859 | ((smc->media_status ^ media) & EPH_LINK_OK)) | 1852 | ((smc->media_status ^ media) & EPH_LINK_OK)) |
1860 | printk(KERN_INFO "%s: %s link beat\n", dev->name, | 1853 | netdev_info(dev, "%s link beat\n", |
1861 | (smc->media_status & EPH_LINK_OK ? "lost" : "found")); | 1854 | smc->media_status & EPH_LINK_OK ? "lost" : "found"); |
1862 | else if ((media & smc->media_status & 2) && | 1855 | else if ((media & smc->media_status & 2) && |
1863 | ((smc->media_status ^ media) & EPH_16COL)) | 1856 | ((smc->media_status ^ media) & EPH_16COL)) |
1864 | printk(KERN_INFO "%s: coax cable %s\n", dev->name, | 1857 | netdev_info(dev, "coax cable %s\n", |
1865 | (media & EPH_16COL ? "problem" : "ok")); | 1858 | media & EPH_16COL ? "problem" : "ok"); |
1866 | if (dev->if_port == 0) { | 1859 | if (dev->if_port == 0) { |
1867 | if (media & 1) { | 1860 | if (media & 1) { |
1868 | if (media & EPH_LINK_OK) | 1861 | if (media & EPH_LINK_OK) |
1869 | printk(KERN_INFO "%s: flipped to 10baseT\n", | 1862 | netdev_info(dev, "flipped to 10baseT\n"); |
1870 | dev->name); | ||
1871 | else | 1863 | else |
1872 | smc_set_xcvr(dev, 2); | 1864 | smc_set_xcvr(dev, 2); |
1873 | } else { | 1865 | } else { |
1874 | if (media & EPH_16COL) | 1866 | if (media & EPH_16COL) |
1875 | smc_set_xcvr(dev, 1); | 1867 | smc_set_xcvr(dev, 1); |
1876 | else | 1868 | else |
1877 | printk(KERN_INFO "%s: flipped to 10base2\n", | 1869 | netdev_info(dev, "flipped to 10base2\n"); |
1878 | dev->name); | ||
1879 | } | 1870 | } |
1880 | } | 1871 | } |
1881 | smc->media_status = media; | 1872 | smc->media_status = media; |