diff mbox series

scsi: ufs: ufshcd: Drop loglevel of WriteBoost message

Message ID 20220709000027.3929970-1-bjorn.andersson@linaro.org (mailing list archive)
State Accepted
Headers show
Series scsi: ufs: ufshcd: Drop loglevel of WriteBoost message | expand

Commit Message

Bjorn Andersson July 9, 2022, midnight UTC
Commit '3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration code")'
changed the log level of the write boost enable/disable notification
from debug to info, this results in a lot of noise in the kernel log
during normal operation.

Drop it back to debug level to avoid this.

Fixes: 3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration code")
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
---
 drivers/ufs/core/ufshcd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alim Akhtar July 9, 2022, 10:04 a.m. UTC | #1
>-----Original Message-----
>From: Bjorn Andersson [mailto:bjorn.andersson@linaro.org]
>Sent: Saturday, July 9, 2022 5:30 AM
>To: Alim Akhtar <alim.akhtar@samsung.com>; Avri Altman
><avri.altman@wdc.com>; Bart Van Assche <bvanassche@acm.org>; James
>E.J. Bottomley <jejb@linux.ibm.com>; Martin K. Petersen
><martin.petersen@oracle.com>
>Cc: Bean Huo <beanhuo@micron.com>; Adrian Hunter
><adrian.hunter@intel.com>; Yue Hu <huyue2@yulong.com>; linux-
>scsi@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: [PATCH] scsi: ufs: ufshcd: Drop loglevel of WriteBoost message
>
>Commit '3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration code")'
>changed the log level of the write boost enable/disable notification from
>debug to info, this results in a lot of noise in the kernel log during
normal
>operation.
>
>Drop it back to debug level to avoid this.
>
>Fixes: 3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration code")
>Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>---

Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>

> drivers/ufs/core/ufshcd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/ufs/core/ufshcd.c b/drivers/ufs/core/ufshcd.c index
>22042ee3af14..be65d5bf123e 100644
>--- a/drivers/ufs/core/ufshcd.c
>+++ b/drivers/ufs/core/ufshcd.c
>@@ -5747,7 +5747,7 @@ int ufshcd_wb_toggle(struct ufs_hba *hba, bool
>enable)
> 	}
>
> 	hba->dev_info.wb_enabled = enable;
>-	dev_info(hba->dev, "%s Write Booster %s\n",
>+	dev_dbg(hba->dev, "%s Write Booster %s\n",
> 			__func__, enable ? "enabled" : "disabled");
>
> 	return ret;
>--
>2.35.1
Bean Huo July 10, 2022, 9:04 p.m. UTC | #2
On Fri, 2022-07-08 at 17:00 -0700, Bjorn Andersson wrote:
> Commit '3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration
> code")'
> changed the log level of the write boost enable/disable notification
> from debug to info, this results in a lot of noise in the kernel log
> during normal operation.
> 
> Drop it back to debug level to avoid this.
> 
> Fixes: 3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration
> code")
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> 
Acked-by: Bean Huo <beanhuo@micron.com>
Martin K. Petersen July 14, 2022, 4:22 a.m. UTC | #3
On Fri, 8 Jul 2022 17:00:27 -0700, Bjorn Andersson wrote:

> Commit '3b5f3c0d0548 ("scsi: ufs: core: Tidy up WB configuration code")'
> changed the log level of the write boost enable/disable notification
> from debug to info, this results in a lot of noise in the kernel log
> during normal operation.
> 
> Drop it back to debug level to avoid this.
> 
> [...]

Applied to 5.19/scsi-fixes, thanks!

[1/1] scsi: ufs: ufshcd: Drop loglevel of WriteBoost message
      https://git.kernel.org/mkp/scsi/c/2ae57c995003
diff mbox series

Patch

diff --git a/drivers/ufs/core/ufshcd.c b/drivers/ufs/core/ufshcd.c
index 22042ee3af14..be65d5bf123e 100644
--- a/drivers/ufs/core/ufshcd.c
+++ b/drivers/ufs/core/ufshcd.c
@@ -5747,7 +5747,7 @@  int ufshcd_wb_toggle(struct ufs_hba *hba, bool enable)
 	}
 
 	hba->dev_info.wb_enabled = enable;
-	dev_info(hba->dev, "%s Write Booster %s\n",
+	dev_dbg(hba->dev, "%s Write Booster %s\n",
 			__func__, enable ? "enabled" : "disabled");
 
 	return ret;