Re: [PATCH] scsi: ufs: Remove HPB support

[Bart Van Assche <bvanassche@xxxxxxx>]

On 7/18/23 10:02, kernel test robot wrote:
> All warnings (new ones prefixed by >>):
>
>     drivers/ufs/core/ufshcd.c: In function 'ufs_get_device_desc':
> > > drivers/ufs/core/ufshcd.c:8092:12: warning: variable 'b_ufs_feature_sup' set but not used [-Wunused-but-set-variable]
>      8092 |         u8 b_ufs_feature_sup;
>           |            ^~~~~~~~~~~~~~~~~

I think the above warning can be fixed as follows (will fold this in
when reposting this patch):

diff --git a/drivers/ufs/core/ufshcd.c b/drivers/ufs/core/ufshcd.c
index e40b71c8dbc0..c394dc50504a 100644
--- a/drivers/ufs/core/ufshcd.c
+++ b/drivers/ufs/core/ufshcd.c
@@ -8089,7 +8089,6 @@ static int ufs_get_device_desc(struct ufs_hba *hba)
 {
 	int err;
 	u8 model_index;
-	u8 b_ufs_feature_sup;
 	u8 *desc_buf;
 	struct ufs_dev_info *dev_info = &hba->dev_info;

@@ -8118,7 +8117,6 @@ static int ufs_get_device_desc(struct ufs_hba *hba)
 	dev_info->wspecversion = desc_buf[DEVICE_DESC_PARAM_SPEC_VER] << 8 |
 				      desc_buf[DEVICE_DESC_PARAM_SPEC_VER + 1];
 	dev_info->bqueuedepth = desc_buf[DEVICE_DESC_PARAM_Q_DPTH];
-	b_ufs_feature_sup = desc_buf[DEVICE_DESC_PARAM_UFS_FEAT];

 	model_index = desc_buf[DEVICE_DESC_PARAM_PRDCT_NAME];