Messages in this thread | | | Date | Tue, 22 Dec 2020 19:34:06 +0800 | From | Can Guo <> | Subject | Re: [PATCH v2 2/2] scsi: ufs: Relax the condition of UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL |
| |
On 2020-12-22 15:29, Stanley Chu wrote: > UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL is aimed to skip enabling > fWriteBoosterBufferFlushEn while WriteBooster is initializing. > Therefore it is better to apply the checking during WriteBooster > initialization only. > > Signed-off-by: Stanley Chu <stanley.chu@mediatek.com> > --- > drivers/scsi/ufs/ufshcd.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c > index 9d61dc3eb842..e50b19925236 100644 > --- a/drivers/scsi/ufs/ufshcd.c > +++ b/drivers/scsi/ufs/ufshcd.c > @@ -289,7 +289,8 @@ static inline void ufshcd_wb_config(struct ufs_hba > *hba) > if (ret) > dev_err(hba->dev, "%s: En WB flush during H8: failed: %d\n", > __func__, ret); > - ufshcd_wb_toggle_flush(hba, true); > + if (!(hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL)) > + ufshcd_wb_toggle_flush(hba, true); > } > > static void ufshcd_scsi_unblock_requests(struct ufs_hba *hba) > @@ -5401,9 +5402,6 @@ static int > ufshcd_wb_toggle_flush_during_h8(struct ufs_hba *hba, bool set) > > static inline void ufshcd_wb_toggle_flush(struct ufs_hba *hba, bool > enable) > { > - if (hba->quirks & UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL) > - return; > - > if (enable) > ufshcd_wb_buf_flush_enable(hba); > else
Thanks for the change, I was thinking about the same.
Reviewed-by: Can Guo <cang@codeaurora.org>
| |