* [PATCH v2] mmc: mtk-sd: Increase the verbosity of msdc_track_cmd_data
@ 2023-11-29 2:59 Pin-yen Lin
2023-11-29 7:53 ` Wenbin Mei (梅文彬)
0 siblings, 1 reply; 2+ messages in thread
From: Pin-yen Lin @ 2023-11-29 2:59 UTC (permalink / raw)
To: Chaotian Jing, Ulf Hansson, Matthias Brugger,
AngeloGioacchino Del Regno
Cc: linux-arm-kernel, linux-kernel, ot_shunxi.zhang, Chen-Yu Tsai,
linux-mmc, linux-mediatek, Pin-yen Lin
This log message is necessary for debugging, so enable it by default to
debug issues that are hard to reproduce locally.
Signed-off-by: Pin-yen Lin <treapking@chromium.org>
---
Changes in v2:
- Use dev_warn() instead of dev_err()
drivers/mmc/host/mtk-sd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mmc/host/mtk-sd.c b/drivers/mmc/host/mtk-sd.c
index 97f7c3d4be6e..907c13d746c1 100644
--- a/drivers/mmc/host/mtk-sd.c
+++ b/drivers/mmc/host/mtk-sd.c
@@ -1150,8 +1150,8 @@ static void msdc_recheck_sdio_irq(struct msdc_host *host)
static void msdc_track_cmd_data(struct msdc_host *host, struct mmc_command *cmd)
{
if (host->error)
- dev_dbg(host->dev, "%s: cmd=%d arg=%08X; host->error=0x%08X\n",
- __func__, cmd->opcode, cmd->arg, host->error);
+ dev_warn(host->dev, "%s: cmd=%d arg=%08X; host->error=0x%08X\n",
+ __func__, cmd->opcode, cmd->arg, host->error);
}
static void msdc_request_done(struct msdc_host *host, struct mmc_request *mrq)
--
2.43.0.rc1.413.gea7ed67945-goog
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] mmc: mtk-sd: Increase the verbosity of msdc_track_cmd_data
2023-11-29 2:59 [PATCH v2] mmc: mtk-sd: Increase the verbosity of msdc_track_cmd_data Pin-yen Lin
@ 2023-11-29 7:53 ` Wenbin Mei (梅文彬)
0 siblings, 0 replies; 2+ messages in thread
From: Wenbin Mei (梅文彬) @ 2023-11-29 7:53 UTC (permalink / raw)
To: ulf.hansson@linaro.org, matthias.bgg@gmail.com,
treapking@chromium.org, angelogioacchino.delregno@collabora.com,
Chaotian Jing (井朝天)
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, wenst@chromium.org,
linux-mediatek@lists.infradead.org, linux-mmc@vger.kernel.org,
ot_shunxi.zhang@mediatek.corp-partner.google.com
On Wed, 2023-11-29 at 10:59 +0800, Pin-yen Lin wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> This log message is necessary for debugging, so enable it by default
> to
> debug issues that are hard to reproduce locally.
>
> Signed-off-by: Pin-yen Lin <treapking@chromium.org>
>
> ---
>
> Changes in v2:
> - Use dev_warn() instead of dev_err()
>
> drivers/mmc/host/mtk-sd.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/host/mtk-sd.c b/drivers/mmc/host/mtk-sd.c
> index 97f7c3d4be6e..907c13d746c1 100644
> --- a/drivers/mmc/host/mtk-sd.c
> +++ b/drivers/mmc/host/mtk-sd.c
> @@ -1150,8 +1150,8 @@ static void msdc_recheck_sdio_irq(struct
> msdc_host *host)
> static void msdc_track_cmd_data(struct msdc_host *host, struct
> mmc_command *cmd)
> {
> if (host->error)
> - dev_dbg(host->dev, "%s: cmd=%d arg=%08X; host-
> >error=0x%08X\n",
> - __func__, cmd->opcode, cmd->arg, host->error);
> + dev_warn(host->dev, "%s: cmd=%d arg=%08X; host-
> >error=0x%08X\n",
> + __func__, cmd->opcode, cmd->arg, host->error);
Hi Pin-yen,
Thanks for your changes!
Indeed, these logs are necessary for us to debug, but the crc errors
during the tuning process
are expected and we don't want them to be printed.
The timeout errors during the tuning process are unexpected and should
be printed.
So we can change it as below:
if (host->error) {
if ((!mmc_op_tuning(cmd->opcode) && !host->hs400_tuning) ||
cmd->error == -ETIMEDOUT) {
dev_warn(host->dev, "%s: cmd=%d arg=%08X; host-
>error=0x%08X\n", __func__, cmd->opcode, cmd->arg, host->error);
}
}
Best regards,
Wenbin
> }
>
> static void msdc_request_done(struct msdc_host *host, struct
> mmc_request *mrq)
> --
> 2.43.0.rc1.413.gea7ed67945-goog
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-11-29 7:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-29 2:59 [PATCH v2] mmc: mtk-sd: Increase the verbosity of msdc_track_cmd_data Pin-yen Lin
2023-11-29 7:53 ` Wenbin Mei (梅文彬)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox