Title: [9002] trunk/net/dsa: ksz8893m: clean up style a bit
Revision
9002
Author
vapier
Date
2010-07-21 09:07:26 -0400 (Wed, 21 Jul 2010)

Log Message

ksz8893m: clean up style a bit

Modified Paths


Diff

Modified: trunk/net/dsa/ksz8893m.c (9001 => 9002)


--- trunk/net/dsa/ksz8893m.c	2010-07-21 13:01:35 UTC (rev 9001)
+++ trunk/net/dsa/ksz8893m.c	2010-07-21 13:07:26 UTC (rev 9002)
@@ -3,14 +3,13 @@
  *
  * - KSZ8893M support
  *
- * Copyright 2008-2009 Analog Devices Inc.
+ * Copyright 2008-2010 Analog Devices Inc.
  *
- * Enter bugs at http://blackfin.uclinux.org/
- *
  * Licensed under the GPL-2 or later.
- *
  */
 
+#define pr_fmt(fmt) "ksz8893m: " fmt
+
 #include <linux/list.h>
 #include <linux/netdevice.h>
 #include <linux/phy.h>
@@ -46,7 +45,7 @@
 	if (!ret)
 		return message.status;
 
-	printk(KERN_ERR "ksz8893m: read reg%d failed, ret=%d.\n", reg, ret);
+	pr_err("read reg%d failed, ret=%d\n", reg, ret);
 	return ret;
 }
 
@@ -69,7 +68,7 @@
 	if (!ret)
 		return message.status;
 
-	printk(KERN_ERR "ksz8893m: write reg%d failed, ret=%d.\n", reg, ret);
+	pr_err("write reg%d failed, ret=%d\n", reg, ret);
 	return ret;
 }
 
@@ -217,7 +216,7 @@
 	if (port >= 1 && port <= KSZ8893M_PORT_NUM)
 		return port;
 
-	printk(KERN_WARNING "ksz8893m: use default phy addr 3\n");
+	pr_warning("use default phy addr 3\n");
 	return 3;
 }
 
@@ -236,7 +235,7 @@
 	return mdiobus_write(ds->master_mii_bus, phy_addr, regnum, val);
 }
 
-void ksz8893m_poll_link(struct dsa_switch *ds)
+static void ksz8893m_poll_link(struct dsa_switch *ds)
 {
 	int i;
 
@@ -299,7 +298,7 @@
 static int __devinit spi_switch_probe(struct spi_device *spi)
 {
 	if (sw.dev) {
-		printk(KERN_ERR "ksz8893m: only one instance supported at a time\n");
+		pr_err("only one instance supported at a time\n");
 		return 1;
 	}
 	memset(&sw.xfer, 0, sizeof(sw.xfer));
@@ -333,12 +332,13 @@
 	.poll_link		= ksz8893m_poll_link,
 };
 
-int __init ksz8893m_init(void)
+static int __init ksz8893m_init(void)
 {
 	int ret;
+
 	ret = spi_register_driver(&spi_switch_driver);
 	if (ret) {
-		printk(KERN_ERR "ksz8893m: Can't register driver!\n");
+		pr_err("can't register driver\n");
 		return ret;
 	}
 
@@ -347,14 +347,13 @@
 }
 module_init(ksz8893m_init);
 
-void __exit ksz8893m_cleanup(void)
+static void __exit ksz8893m_cleanup(void)
 {
 	spi_unregister_driver(&spi_switch_driver);
 	unregister_switch_driver(&ksz8893m_switch_driver);
 }
 module_exit(ksz8893m_cleanup);
 
-MODULE_AUTHOR("Graf.Yang <[email protected]>");
+MODULE_AUTHOR("Graf Yang <[email protected]>");
 MODULE_DESCRIPTION("KSZ8893M driver for DSA");
 MODULE_LICENSE("GPL");
-

Modified: trunk/net/dsa/ksz8893m.h (9001 => 9002)


--- trunk/net/dsa/ksz8893m.h	2010-07-21 13:01:35 UTC (rev 9001)
+++ trunk/net/dsa/ksz8893m.h	2010-07-21 13:07:26 UTC (rev 9002)
@@ -1,5 +1,16 @@
-#ifndef __KSZ8893M_H
-#define __KSZ8893M_H
+/*
+ * Integrated 3-Port 10/100 Managed Switch with PHYs
+ *
+ * - KSZ8893M support
+ *
+ * Copyright 2008-2010 Analog Devices Inc.
+ *
+ * Licensed under the GPL-2 or later.
+ */
+
+#ifndef __KSZ8893M_H__
+#define __KSZ8893M_H__
+
 #include <linux/netdevice.h>
 
 #define KSZ8893M_PORT_NUM 3
@@ -10,7 +21,6 @@
 #define SPI_READ          3
 #define SPI_WRITE         2
 
-
 /* PHYID High */
 #define PHYID_HIGH        0x22
 /* PHYID Low */
@@ -209,4 +219,5 @@
 	QMDebug1,
 	QMDebug2,
 };
+
 #endif
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to