| Message ID | 1530428560-4440-7-git-send-email-gilad@benyossef.com (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Herbert Xu |
| Headers | show |
Hi Gilad, On Sun, Jul 1, 2018 at 9:05 AM Gilad Ben-Yossef <gilad@benyossef.com> wrote: > A debug print about register status post interrupt can happen > quite often. Rate limit it to avoid cluttering the log. > > Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com> > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Thanks for your patch! > --- a/drivers/crypto/ccree/cc_driver.c > +++ b/drivers/crypto/ccree/cc_driver.c > @@ -131,8 +131,8 @@ static irqreturn_t cc_isr(int irq, void *dev_id) > } > > if (irr) { > - dev_dbg(dev, "IRR includes unknown cause bits (0x%08X)\n", > - irr); > + dev_dbg_ratelimited(dev, "IRR includes unknown cause bits (0x%08X)\n", > + irr); > /* Just warning */ > } cc_isr: 932 callbacks suppressed Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> Note that it still printed 1410 lines like: ccree e6601000.crypto: Got IRR=0x008000D8 (actual value may differ) but perhaps you do intend to see these during debugging? Gr{oetje,eeting}s, Geert
diff --git a/drivers/crypto/ccree/cc_driver.c b/drivers/crypto/ccree/cc_driver.c index bd974fe..1ff229c 100644 --- a/drivers/crypto/ccree/cc_driver.c +++ b/drivers/crypto/ccree/cc_driver.c @@ -131,8 +131,8 @@ static irqreturn_t cc_isr(int irq, void *dev_id) } if (irr) { - dev_dbg(dev, "IRR includes unknown cause bits (0x%08X)\n", - irr); + dev_dbg_ratelimited(dev, "IRR includes unknown cause bits (0x%08X)\n", + irr); /* Just warning */ }
A debug print about register status post interrupt can happen quite often. Rate limit it to avoid cluttering the log. Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com> Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> --- drivers/crypto/ccree/cc_driver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)