[PATCH] serial: sh-sci: Use dev_dbg() for frame and parity errors

[PATCH] serial: sh-sci: Use dev_dbg() for frame and parity errors

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Adjust the console printout handling for frame and parity errors to the same
style as for break errors by using dev_dbg() instead of dev_notice().
This reduces the amount of console printouts generated by the defconfig kernel
on ARM R-Car Gen3 for certain use cases.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 drivers/tty/serial/sh-sci.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--- 0001/drivers/tty/serial/sh-sci.c
+++ work/drivers/tty/serial/sh-sci.c	2021-12-04 22:07:39.760407200 +0900
@@ -895,11 +895,11 @@ static void sci_receive_chars(struct uar
 				if (status & SCxSR_FER(port)) {
 					flag = TTY_FRAME;
 					port->icount.frame++;
-					dev_notice(port->dev, "frame error\n");
+					dev_dbg(port->dev, "frame error\n");
 				} else if (status & SCxSR_PER(port)) {
 					flag = TTY_PARITY;
 					port->icount.parity++;
-					dev_notice(port->dev, "parity error\n");
+					dev_dbg(port->dev, "parity error\n");
 				} else
 					flag = TTY_NORMAL;
 

Re: [PATCH] serial: sh-sci: Use dev_dbg() for frame and parity errors

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 595 bytes --]

Hi Magnus,

On Sat, Dec 04, 2021 at 10:46:18PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
> 
> Adjust the console printout handling for frame and parity errors to the same
> style as for break errors by using dev_dbg() instead of dev_notice().
> This reduces the amount of console printouts generated by the defconfig kernel
> on ARM R-Car Gen3 for certain use cases.
> 
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

Could we maybe just remove them because there are dedicated counters for
this?

Have a nice weekend,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] serial: sh-sci: Use dev_dbg() for frame and parity errors

[Magnus Damm]

Hi Wolfram,

On Sun, Dec 5, 2021 at 12:30 AM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
>
> Hi Magnus,
>
> On Sat, Dec 04, 2021 at 10:46:18PM +0900, Magnus Damm wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> >
> > Adjust the console printout handling for frame and parity errors to the same
> > style as for break errors by using dev_dbg() instead of dev_notice().
> > This reduces the amount of console printouts generated by the defconfig kernel
> > on ARM R-Car Gen3 for certain use cases.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
>
> Could we maybe just remove them because there are dedicated counters for
> this?

Sure, overrun and break also have counters so I'll prepare a patch
removing all four if you don't mind.

Cheers,

/ magnus

Re: [PATCH] serial: sh-sci: Use dev_dbg() for frame and parity errors

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 160 bytes --]

Hi Magnus,

> Sure, overrun and break also have counters so I'll prepare a patch
> removing all four if you don't mind.

Good idea!

All the best,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]