diff mbox series

[v2] pinctrl: rza1: reduce printed messages

Message ID 20191212135301.17915-1-chris.brandt@renesas.com
State New
Headers show
Series [v2] pinctrl: rza1: reduce printed messages | expand

Commit Message

Chris Brandt Dec. 12, 2019, 1:53 p.m. UTC 
Since this message is printed for each port, it creates a lot of output
during boot and would serve better only during debugging.

Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Acked-by: Jacopo Mondi <jacopo@jmondi.org>
---
v2:
 * Fixed text allignment
 * Added Acked-by
---
 drivers/pinctrl/pinctrl-rza1.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Linus Walleij Dec. 16, 2019, 9:41 a.m. UTC | #1 
On Thu, Dec 12, 2019 at 2:53 PM Chris Brandt <chris.brandt@renesas.com> wrote:

> Since this message is printed for each port, it creates a lot of output
> during boot and would serve better only during debugging.
>
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> Acked-by: Jacopo Mondi <jacopo@jmondi.org>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

I think Geert usually pick these up as well, else tell me if I should
just apply it.

Yours,
Linus Walleij
Geert Uytterhoeven Dec. 20, 2019, 2:50 p.m. UTC | #2 
Hi Linus,

On Mon, Dec 16, 2019 at 10:41 AM Linus Walleij <linus.walleij@linaro.org> wrote:
> On Thu, Dec 12, 2019 at 2:53 PM Chris Brandt <chris.brandt@renesas.com> wrote:
> > Since this message is printed for each port, it creates a lot of output
> > during boot and would serve better only during debugging.
> >
> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> > Acked-by: Jacopo Mondi <jacopo@jmondi.org>
>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
>
> I think Geert usually pick these up as well, else tell me if I should
> just apply it.

Indeed.

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
and queueing in sh-pfc-for-v5.6.

Gr{oetje,eeting}s,

                        Geert
diff mbox series

Patch

diff --git a/drivers/pinctrl/pinctrl-rza1.c b/drivers/pinctrl/pinctrl-rza1.c
index 017fc6b3e27e..e291923df13a 100644
--- a/drivers/pinctrl/pinctrl-rza1.c
+++ b/drivers/pinctrl/pinctrl-rza1.c
@@ -1235,8 +1235,8 @@  static int rza1_parse_gpiochip(struct rza1_pinctrl *rza1_pctl,
 
 	pinctrl_add_gpio_range(rza1_pctl->pctl, range);
 
-	dev_info(rza1_pctl->dev, "Parsed gpiochip %s with %d pins\n",
-		 chip->label, chip->ngpio);
+	dev_dbg(rza1_pctl->dev, "Parsed gpiochip %s with %d pins\n",
+		chip->label, chip->ngpio);
 
 	return 0;
 }