linux-renesas-soc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] pinctrl: rza1: reduce printed messages
@ 2019-12-12 13:53 Chris Brandt
  2019-12-16  9:41 ` Linus Walleij
  0 siblings, 1 reply; 3+ messages in thread
From: Chris Brandt @ 2019-12-12 13:53 UTC (permalink / raw)
  To: Geert Uytterhoeven, Linus Walleij
  Cc: linux-gpio, linux-renesas-soc, Chris Brandt, Jacopo Mondi

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(-)

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;
 }
-- 
2.23.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] pinctrl: rza1: reduce printed messages
  2019-12-12 13:53 [PATCH v2] pinctrl: rza1: reduce printed messages Chris Brandt
@ 2019-12-16  9:41 ` Linus Walleij
  2019-12-20 14:50   ` Geert Uytterhoeven
  0 siblings, 1 reply; 3+ messages in thread
From: Linus Walleij @ 2019-12-16  9:41 UTC (permalink / raw)
  To: Chris Brandt
  Cc: Geert Uytterhoeven, open list:GPIO SUBSYSTEM, Linux-Renesas,
	Jacopo Mondi

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] pinctrl: rza1: reduce printed messages
  2019-12-16  9:41 ` Linus Walleij
@ 2019-12-20 14:50   ` Geert Uytterhoeven
  0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2019-12-20 14:50 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Chris Brandt, Geert Uytterhoeven, open list:GPIO SUBSYSTEM,
	Linux-Renesas, Jacopo Mondi

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

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-12-20 14:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-12 13:53 [PATCH v2] pinctrl: rza1: reduce printed messages Chris Brandt
2019-12-16  9:41 ` Linus Walleij
2019-12-20 14:50   ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).