| Message ID | 20220709000027.3929970-1-bjorn.andersson@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | scsi: ufs: ufshcd: Drop loglevel of WriteBoost message | expand |
>-----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
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>
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 --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;
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(-)