pps: convert printk/pr_* to dev_*

Since we now have direct pointers to struct pps_device everywhere it's
easy to use dev_* functions to print messages instead of plain printks.
Where dev_* cannot be used printks are converted to pr_*.

Signed-off-by: Alexander Gordeev <lasaine@lvk.cs.msu.su>
Acked-by: Rodolfo Giometti <giometti@linux.it>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

4 files changed, 26 insertions, 21 deletions

diff --git a/drivers/pps/clients/pps-ktimer.c b/drivers/pps/clients/pps-ktimer.c
index 966ead188ee2..2728469d3884 100644
--- a/drivers/pps/clients/pps-ktimer.c
+++ b/drivers/pps/clients/pps-ktimer.c
@@ -19,6 +19,7 @@
  *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
 #include <linux/kernel.h>
 #include <linux/module.h>
@@ -45,7 +46,7 @@ static void pps_ktimer_event(unsigned long ptr)
         /* First of all we get the time stamp... */
         pps_get_ts(&ts);
 
-        pr_info("PPS event at %lu\n", jiffies);
+        dev_info(pps->dev, "PPS event at %lu\n", jiffies);
 
         pps_event(pps, &ts, PPS_CAPTUREASSERT, NULL);
 
@@ -94,7 +95,7 @@ static int __init pps_ktimer_init(void)
         pps = pps_register_source(&pps_ktimer_info,
                                 PPS_CAPTUREASSERT | PPS_OFFSETASSERT);
         if (pps == NULL) {
-                printk(KERN_ERR "cannot register ktimer source\n");
+                pr_err("cannot register PPS source\n");
                 return -ENOMEM;
         }
 
diff --git a/drivers/pps/clients/pps-ldisc.c b/drivers/pps/clients/pps-ldisc.c
index 1517f5498ec0..1b3c6eda12ba 100644
--- a/drivers/pps/clients/pps-ldisc.c
+++ b/drivers/pps/clients/pps-ldisc.c
@@ -19,6 +19,8 @@
  *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
 #include <linux/module.h>
 #include <linux/serial_core.h>
 #include <linux/tty.h>
diff --git a/drivers/pps/kapi.c b/drivers/pps/kapi.c
index 98d4012ca595..e8847c118ea3 100644
--- a/drivers/pps/kapi.c
+++ b/drivers/pps/kapi.c
@@ -19,6 +19,7 @@
  *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
 #include <linux/kernel.h>
 #include <linux/module.h>
@@ -80,20 +81,20 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
 
         /* Sanity checks */
         if ((info->mode & default_params) != default_params) {
-                printk(KERN_ERR "pps: %s: unsupported default parameters\n",
+                pr_err("%s: unsupported default parameters\n",
                                         info->name);
                 err = -EINVAL;
                 goto pps_register_source_exit;
         }
         if ((info->mode & (PPS_ECHOASSERT | PPS_ECHOCLEAR)) != 0 &&
                         info->echo == NULL) {
-                printk(KERN_ERR "pps: %s: echo function is not defined\n",
+                pr_err("%s: echo function is not defined\n",
                                         info->name);
                 err = -EINVAL;
                 goto pps_register_source_exit;
         }
         if ((info->mode & (PPS_TSFMT_TSPEC | PPS_TSFMT_NTPFP)) == 0) {
-                printk(KERN_ERR "pps: %s: unspecified time format\n",
+                pr_err("%s: unspecified time format\n",
                                         info->name);
                 err = -EINVAL;
                 goto pps_register_source_exit;
@@ -138,7 +139,7 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
         if (id >= PPS_MAX_SOURCES) {
                 spin_unlock_irq(&pps_idr_lock);
 
-                printk(KERN_ERR "pps: %s: too many PPS sources in the system\n",
+                pr_err("%s: too many PPS sources in the system\n",
                                         info->name);
                 err = -EBUSY;
                 goto free_idr;
@@ -150,12 +151,12 @@ struct pps_device *pps_register_source(struct pps_source_info *info,
         /* Create the char device */
         err = pps_register_cdev(pps);
         if (err < 0) {
-                printk(KERN_ERR "pps: %s: unable to create char device\n",
+                pr_err("%s: unable to create char device\n",
                                         info->name);
                 goto free_idr;
         }
 
-        pr_info("new PPS source %s at ID %d\n", info->name, id);
+        dev_info(pps->dev, "new PPS source %s\n", info->name);
 
         return pps;
 
@@ -168,7 +169,7 @@ kfree_pps:
         kfree(pps);
 
 pps_register_source_exit:
-        printk(KERN_ERR "pps: %s: unable to register source\n", info->name);
+        pr_err("%s: unable to register source\n", info->name);
 
         return NULL;
 }
diff --git a/drivers/pps/pps.c b/drivers/pps/pps.c
index 1922f27a52ac..9f7c2e858dd0 100644
--- a/drivers/pps/pps.c
+++ b/drivers/pps/pps.c
@@ -19,6 +19,7 @@
  *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
 
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
 #include <linux/kernel.h>
 #include <linux/module.h>
@@ -67,7 +68,7 @@ static long pps_cdev_ioctl(struct file *file,
 
         switch (cmd) {
         case PPS_GETPARAMS:
-                pr_debug("PPS_GETPARAMS: source %d\n", pps->id);
+                dev_dbg(pps->dev, "PPS_GETPARAMS\n");
 
                 spin_lock_irq(&pps->lock);
 
@@ -83,7 +84,7 @@ static long pps_cdev_ioctl(struct file *file,
                 break;
 
         case PPS_SETPARAMS:
-                pr_debug("PPS_SETPARAMS: source %d\n", pps->id);
+                dev_dbg(pps->dev, "PPS_SETPARAMS\n");
 
                 /* Check the capabilities */
                 if (!capable(CAP_SYS_TIME))
@@ -93,14 +94,14 @@ static long pps_cdev_ioctl(struct file *file,
                 if (err)
                         return -EFAULT;
                 if (!(params.mode & (PPS_CAPTUREASSERT | PPS_CAPTURECLEAR))) {
-                        pr_debug("capture mode unspecified (%x)\n",
+                        dev_dbg(pps->dev, "capture mode unspecified (%x)\n",
                                                                 params.mode);
                         return -EINVAL;
                 }
 
                 /* Check for supported capabilities */
                 if ((params.mode & ~pps->info.mode) != 0) {
-                        pr_debug("unsupported capabilities (%x)\n",
+                        dev_dbg(pps->dev, "unsupported capabilities (%x)\n",
                                                                 params.mode);
                         return -EINVAL;
                 }
@@ -113,7 +114,7 @@ static long pps_cdev_ioctl(struct file *file,
                 /* Restore the read only parameters */
                 if ((params.mode & (PPS_TSFMT_TSPEC | PPS_TSFMT_NTPFP)) == 0) {
                         /* section 3.3 of RFC 2783 interpreted */
-                        pr_debug("time format unspecified (%x)\n",
+                        dev_dbg(pps->dev, "time format unspecified (%x)\n",
                                                                 params.mode);
                         pps->params.mode |= PPS_TSFMT_TSPEC;
                 }
@@ -126,7 +127,7 @@ static long pps_cdev_ioctl(struct file *file,
                 break;
 
         case PPS_GETCAP:
-                pr_debug("PPS_GETCAP: source %d\n", pps->id);
+                dev_dbg(pps->dev, "PPS_GETCAP\n");
 
                 err = put_user(pps->info.mode, iuarg);
                 if (err)
@@ -138,7 +139,7 @@ static long pps_cdev_ioctl(struct file *file,
                 struct pps_fdata fdata;
                 unsigned int ev;
 
-                pr_debug("PPS_FETCH: source %d\n", pps->id);
+                dev_dbg(pps->dev, "PPS_FETCH\n");
 
                 err = copy_from_user(&fdata, uarg, sizeof(struct pps_fdata));
                 if (err)
@@ -153,7 +154,7 @@ static long pps_cdev_ioctl(struct file *file,
                 else {
                         unsigned long ticks;
 
-                        pr_debug("timeout %lld.%09d\n",
+                        dev_dbg(pps->dev, "timeout %lld.%09d\n",
                                         (long long) fdata.timeout.sec,
                                         fdata.timeout.nsec);
                         ticks = fdata.timeout.sec * HZ;
@@ -171,7 +172,7 @@ static long pps_cdev_ioctl(struct file *file,
 
                 /* Check for pending signals */
                 if (err == -ERESTARTSYS) {
-                        pr_debug("pending signal caught\n");
+                        dev_dbg(pps->dev, "pending signal caught\n");
                         return -EINTR;
                 }
 
@@ -240,7 +241,7 @@ int pps_register_cdev(struct pps_device *pps)
 
         err = cdev_add(&pps->cdev, devt, 1);
         if (err) {
-                printk(KERN_ERR "pps: %s: failed to add char device %d:%d\n",
+                pr_err("%s: failed to add char device %d:%d\n",
                                 pps->info.name, MAJOR(pps_devt), pps->id);
                 return err;
         }
@@ -282,14 +283,14 @@ static int __init pps_init(void)
 
         pps_class = class_create(THIS_MODULE, "pps");
         if (!pps_class) {
-                printk(KERN_ERR "pps: failed to allocate class\n");
+                pr_err("failed to allocate class\n");
                 return -ENOMEM;
         }
         pps_class->dev_attrs = pps_attrs;
 
         err = alloc_chrdev_region(&pps_devt, 0, PPS_MAX_SOURCES, "pps");
         if (err < 0) {
-                printk(KERN_ERR "pps: failed to allocate char device region\n");
+                pr_err("failed to allocate char device region\n");
                 goto remove_class;
         }