On Wed, April 23, 2014, Doug Anderson wrote: > Data errors are completely expected during tuning. Printing them out > is confusing people looking at the kernel logs. They see things like: > > [ 3.613296] dwmmc_exynos 12200000.dwmmc0: data error, status 0x00000088 > > ...and they think something is wrong with their hardware. > > Remove the printouts. We'll leave it up to a higher level to report > about errors. Yes, your saying makes sense. But in normal I/O, actual error information such as status below is useful, isn't it? Thanks, Seungwon Jeon > > Signed-off-by: Doug Anderson <dianders@xxxxxxxxxxxx> > --- > drivers/mmc/host/dw_mmc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c > index cced599..4c8d423 100644 > --- a/drivers/mmc/host/dw_mmc.c > +++ b/drivers/mmc/host/dw_mmc.c > @@ -1248,7 +1248,7 @@ static int dw_mci_data_complete(struct dw_mci *host, struct mmc_data *data) > data->error = -EIO; > } > > - dev_err(host->dev, "data error, status 0x%08x\n", status); > + dev_dbg(host->dev, "data error, status 0x%08x\n", status); > > /* > * After an error, there may be data lingering > -- > 1.9.1.423.g4596e3a > > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |