staging: most: cdev: use dev_*() functions to print messages
authorChristian Gromm <christian.gromm@microchip.com>
Mon, 22 Jun 2020 08:48:11 +0000 (10:48 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 25 Jun 2020 13:40:28 +0000 (15:40 +0200)
This patch removes the pr_*() functions and uses dev_*() instead.

Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Link: https://lore.kernel.org/r/1592815695-19305-3-git-send-email-christian.gromm@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/cdev/cdev.c

index 093b44d095a9a4ae3eb446ebe1ec0b18796b15ae..394c7621cb51f21ec10644e6095485f92c5d8bbd 100644 (file)
@@ -5,7 +5,6 @@
  * Copyright (C) 2013-2015 Microchip Technology Germany II GmbH & Co. KG
  */
 
-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 #include <linux/module.h>
 #include <linux/sched.h>
 #include <linux/fs.h>
@@ -375,7 +374,7 @@ static int comp_rx_completion(struct mbo *mbo)
        spin_unlock(&c->unlink);
 #ifdef DEBUG_MESG
        if (kfifo_is_full(&c->fifo))
-               pr_info("WARN: Fifo is full\n");
+               dev_warn(c->dev, "Fifo is full\n");
 #endif
        wake_up_interruptible(&c->wq);
        return 0;
@@ -395,14 +394,15 @@ static int comp_tx_completion(struct most_interface *iface, int channel_id)
        if (!iface)
                return -EINVAL;
 
+       c = get_channel(iface, channel_id);
+       if (!c)
+               return -ENXIO;
+
        if ((channel_id < 0) || (channel_id >= iface->num_channels)) {
-               pr_info("Channel ID out of range\n");
+               dev_warn(c->dev, "Channel ID out of range\n");
                return -EINVAL;
        }
 
-       c = get_channel(iface, channel_id);
-       if (!c)
-               return -ENXIO;
        wake_up_interruptible(&c->wq);
        return 0;
 }