Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Alim Akhtar" <alim.akhtar@samsung.com>
To:     "'Bjorn Andersson'" <bjorn.andersson@linaro.org>,
        "'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>
In-Reply-To: <20220709000027.3929970-1-bjorn.andersson@linaro.org>
Subject: RE: [PATCH] scsi: ufs: ufshcd: Drop loglevel of WriteBoost message
Date:   Sat, 9 Jul 2022 15:34:09 +0530
Message-ID: <000001d8937b$3d39faa0$b7adefe0$@samsung.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Thread-Index: AQImLzBMENaA2Jj71t4ZY6FxYLPYzAJCrgA7rMiCdUA=
Content-Language: en-us
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
DLP-Filter: Pass
References: <CGME20220708235759epcas5p3705680d639a85558bdbc26d827deb563@epcas5p3.samsung.com>
        <20220709000027.3929970-1-bjorn.andersson@linaro.org>
Precedence: bulk


>-----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