diff options
| author |  Bean Huo <beanhuo@micron.com>
| 2022-10-26 00:24:28 +0200 |
|---|---|---|
| committer |  Martin K. Petersen <martin.petersen@oracle.com>
| 2022-10-27 02:20:50 +0000 |
| commit | b43678ea5bbd92388339ecae47ed44955474f53b (patch) | |
| tree | ce56b9095942adc1d6cd9e2a4722234c35c4f06d /drivers/ufs/core/ufshcd-priv.h | |
| parent | scsi: pcmcia: nsp_cs: Remove unused variable i (diff) | |
| download | linux-b43678ea5bbd92388339ecae47ed44955474f53b.tar.gz linux-b43678ea5bbd92388339ecae47ed44955474f53b.tar.bz2 linux-b43678ea5bbd92388339ecae47ed44955474f53b.zip | |
Ccommit d3d9c4570285 ("scsi: ufs: Fix memory corruption by
ufshcd_read_desc_param()") has properly fixed stack overflow issue.
As a result, commit a2fca52ee640 ("scsi: ufs: WB is only available on LUN
#0 to #7") is no longer required. Revert it.
Cc: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Bean Huo <beanhuo@micron.com>
Link: https://lore.kernel.org/r/20221025222430.277768-2-beanhuo@iokpp.de
Reviewed-by: Arthur Simchaev <arthur.simchaev@wdc.com>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/ufs/core/ufshcd-priv.h')
| -rw-r--r-- | drivers/ufs/core/ufshcd-priv.h | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/ufs/core/ufshcd-priv.h b/drivers/ufs/core/ufshcd-priv.h index f68ca33f6ac7..a9e8e1f5afe7 100644 --- a/drivers/ufs/core/ufshcd-priv.h +++ b/drivers/ufs/core/ufshcd-priv.h @@ -293,16 +293,12 @@ static inline int ufshcd_rpm_put(struct ufs_hba *hba) * @lun: LU number to check * @return: true if the lun has a matching unit descriptor, false otherwise */ -static inline bool ufs_is_valid_unit_desc_lun(struct ufs_dev_info *dev_info, - u8 lun, u8 param_offset) +static inline bool ufs_is_valid_unit_desc_lun(struct ufs_dev_info *dev_info, u8 lun) { if (!dev_info || !dev_info->max_lu_supported) { pr_err("Max General LU supported by UFS isn't initialized\n"); return false; } - /* WB is available only for the logical unit from 0 to 7 */ - if (param_offset == UNIT_DESC_PARAM_WB_BUF_ALLOC_UNITS) - return lun < UFS_UPIU_MAX_WB_LUN_ID; return lun == UFS_UPIU_RPMB_WLUN || (lun < dev_info->max_lu_supported); } |