On Thu, 2020-05-28 at 13:41 +0000, Avri Altman wrote: > > +++ b/drivers/scsi/ufs/ufshcd.c > > @@ -6881,8 +6881,7 @@ static int ufs_get_device_desc(struct ufs_hba > > *hba) > > u8 *desc_buf; > > struct ufs_dev_info *dev_info = &hba->dev_info; > > > > - buff_len = max_t(size_t, hba->desc_size.dev_desc, > > - QUERY_DESC_MAX_SIZE + 1); > > + buff_len = QUERY_DESC_MAX_SIZE + 1; > > So why the +1? > Originally it was used for the '\0' terminator, which is not needed > anymore. you are correct, now not need +1, I will change it. thanks, Bean
