RE: [PATCH] mmc: host: use pr_err for sdhci_dumpregs

From: Baranowska, BeataX
Date: Wed Jun 01 2016 - 01:13:29 EST


> -----Original Message-----
> From: Andy Shevchenko [mailto:andy.shevchenko@xxxxxxxxx]
> Sent: Tuesday, May 31, 2016 5:47 PM
> To: Baranowska, BeataX <beatax.baranowska@xxxxxxxxx>
> Cc: Hunter, Adrian <adrian.hunter@xxxxxxxxx>; Ulf Hansson
> <ulf.hansson@xxxxxxxxxx>; linux-mmc@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Dong, Chuanxiao <chuanxiao.dong@xxxxxxxxx>;
> Jarosz, SebastianX <sebastianx.jarosz@xxxxxxxxx>
> Subject: Re: [PATCH] mmc: host: use pr_err for sdhci_dumpregs
>
> On Mon, May 30, 2016 at 10:59 AM, Baranowska, BeataX
> <beatax.baranowska@xxxxxxxxx> wrote:
> > From: Chuanxiao Dong <chuanxiao.dong@xxxxxxxxx>
> >
> > sdhci_dumpregs is used to dump registers when error happens. Thus it
> > should use pr_err instead of pr_debug to show more information about
> the hardware.
> >
>
> Wouldn't be too noisy?
>
> --
> With Best Regards,
> Andy Shevchenko

It is when error happens so IMHO it should be well visible.

Regards,
Beata Baranowska