Commit f273c54b authored by Bean Huo's avatar Bean Huo Committed by Martin K. Petersen
Browse files

scsi: ufs: Change ufshcd_comp_devman_upiu() to ufshcd_compose_devman_upiu() 

ufshcd_comp_devman_upiu() was poorly named leading people to think it was a
completion function. Rename it to ufshcd_compose_devman_upiu().

Link: https://lore.kernel.org/r/20200814095034.20709-2-huobean@gmail.com


Reviewed-by: default avatarStanley Chu <stanley.chu@mediatek.com>
Reviewed-by: default avatarAsutosh Das <asutoshd@codeaurora.org>
Acked-by: default avatarAvri Altman <avri.altman@wdc.com>
Signed-off-by: default avatarBean Huo <beanhuo@micron.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 3079285b

drivers/scsi/ufs/ufshcd.c

+4 −3
Original line number Diff line number Diff line
@@ -2401,12 +2401,13 @@ static inline void ufshcd_prepare_utp_nop_upiu(struct ufshcd_lrb *lrbp) 
}



/**

 * ufshcd_comp_devman_upiu - UFS Protocol Information Unit(UPIU)

 * ufshcd_compose_devman_upiu - UFS Protocol Information Unit(UPIU)

 *			     for Device Management Purposes

 * @hba: per adapter instance

 * @lrbp: pointer to local reference block

 */

static int ufshcd_comp_devman_upiu(struct ufs_hba *hba, struct ufshcd_lrb *lrbp)

static int ufshcd_compose_devman_upiu(struct ufs_hba *hba,

				      struct ufshcd_lrb *lrbp)

{

	u8 upiu_flags;

	int ret = 0;
@@ -2613,7 +2614,7 @@ static int ufshcd_compose_dev_cmd(struct ufs_hba *hba, 
	ufshcd_prepare_lrbp_crypto(NULL, lrbp);

	hba->dev_cmd.type = cmd_type;



	return ufshcd_comp_devman_upiu(hba, lrbp);

	return ufshcd_compose_devman_upiu(hba, lrbp);

}



static int