sh-pfc: Convert message printing from pr_* to dev_*
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Sun, 10 Mar 2013 17:00:02 +0000 (18:00 +0100)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Fri, 15 Mar 2013 12:33:54 +0000 (13:33 +0100)
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/sh-pfc/core.c
drivers/pinctrl/sh-pfc/gpio.c
drivers/pinctrl/sh-pfc/pinctrl.c

index a04c497..feef897 100644 (file)
@@ -10,7 +10,6 @@
  */
 
 #define DRV_NAME "sh-pfc"
-#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
 #include <linux/bitops.h>
 #include <linux/err.h>
@@ -173,9 +172,9 @@ static void sh_pfc_write_config_reg(struct sh_pfc *pfc,
 
        sh_pfc_config_reg_helper(pfc, crp, field, &mapped_reg, &mask, &pos);
 
-       pr_debug("write_reg addr = %lx, value = %ld, field = %ld, "
-                "r_width = %ld, f_width = %ld\n",
-                crp->reg, value, field, crp->reg_width, crp->field_width);
+       dev_dbg(pfc->dev, "write_reg addr = %lx, value = %ld, field = %ld, "
+               "r_width = %ld, f_width = %ld\n",
+               crp->reg, value, field, crp->reg_width, crp->field_width);
 
        mask = ~(mask << pos);
        value = value << pos;
@@ -254,7 +253,8 @@ static int sh_pfc_mark_to_enum(struct sh_pfc *pfc, pinmux_enum_t mark, int pos,
                }
        }
 
-       pr_err("cannot locate data/mark enum_id for mark %d\n", mark);
+       dev_err(pfc->dev, "cannot locate data/mark enum_id for mark %d\n",
+               mark);
        return -EINVAL;
 }
 
@@ -396,13 +396,13 @@ static int sh_pfc_probe(struct platform_device *pdev)
                 * PFC state as it is, given that there are already
                 * extant users of it that have succeeded by this point.
                 */
-               pr_notice("failed to init GPIO chip, ignoring...\n");
+               dev_notice(pfc->dev, "failed to init GPIO chip, ignoring...\n");
        }
 #endif
 
        platform_set_drvdata(pdev, pfc);
 
-       pr_info("%s support registered\n", info->name);
+       dev_info(pfc->dev, "%s support registered\n", info->name);
 
        return 0;
 }
index e299f14..d7acb06 100644 (file)
@@ -9,8 +9,6 @@
  * for more details.
  */
 
-#define pr_fmt(fmt) KBUILD_MODNAME " gpio: " fmt
-
 #include <linux/device.h>
 #include <linux/gpio.h>
 #include <linux/init.h>
@@ -273,12 +271,18 @@ static int gpio_pin_setup(struct sh_pfc_chip *chip)
 
 static int gpio_function_request(struct gpio_chip *gc, unsigned offset)
 {
+       static bool __print_once;
        struct sh_pfc *pfc = gpio_to_pfc(gc);
        unsigned int mark = pfc->info->func_gpios[offset].enum_id;
        unsigned long flags;
        int ret;
 
-       pr_notice_once("Use of GPIO API for function requests is deprecated, convert to pinctrl\n");
+       if (!__print_once) {
+               dev_notice(pfc->dev,
+                          "Use of GPIO API for function requests is deprecated."
+                          " Convert to pinctrl\n");
+               __print_once = true;
+       }
 
        if (mark == 0)
                return -EINVAL;
@@ -334,9 +338,9 @@ sh_pfc_add_gpiochip(struct sh_pfc *pfc, int(*setup)(struct sh_pfc_chip *))
        if (unlikely(ret < 0))
                return ERR_PTR(ret);
 
-       pr_info("%s handling gpio %u -> %u\n",
-               chip->gpio_chip.label, chip->gpio_chip.base,
-               chip->gpio_chip.base + chip->gpio_chip.ngpio - 1);
+       dev_info(pfc->dev, "%s handling gpio %u -> %u\n",
+                chip->gpio_chip.label, chip->gpio_chip.base,
+                chip->gpio_chip.base + chip->gpio_chip.ngpio - 1);
 
        return chip;
 }
index 82e4fb2..52179bb 100644 (file)
@@ -9,7 +9,6 @@
  */
 
 #define DRV_NAME "sh-pfc"
-#define pr_fmt(fmt) KBUILD_MODNAME " pinctrl: " fmt
 
 #include <linux/device.h>
 #include <linux/err.h>
@@ -176,7 +175,9 @@ static int sh_pfc_gpio_request_enable(struct pinctrl_dev *pctldev,
        spin_lock_irqsave(&pfc->lock, flags);
 
        if (cfg->type != PINMUX_TYPE_NONE) {
-               pr_err("Pin %u is busy, can't configure it as GPIO.\n", offset);
+               dev_err(pfc->dev,
+                       "Pin %u is busy, can't configure it as GPIO.\n",
+                       offset);
                ret = -EBUSY;
                goto done;
        }