Skip to content

Commit 6bfc4bf

Browse files
Chanwoo Leemartinkpetersen
authored andcommitted
scsi: ufs: Remove unnecessary return in void vops wrappers
ufshcd_vops_exit(), ufshcd_vops_setup_task_mgmt(), and ufshcd_vops_hibern8_notify() use 'return hba->vops->xxx()' while other void vops wrappers call without return. Remove the unnecessary return keywords for consistency. Signed-off-by: Chanwoo Lee <cw9316.lee@samsung.com> Reviewed-by: Peter Wang <peter.wang@mediatek.com> Reviewed-by: Bart Van Assche <bvanassche@acm.org> Link: https://patch.msgid.link/20260529061503.301182-1-cw9316.lee@samsung.com Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
1 parent 2483ae0 commit 6bfc4bf

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

drivers/ufs/core/ufshcd-priv.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ static inline const char *ufshcd_get_var_name(struct ufs_hba *hba)
132132
static inline void ufshcd_vops_exit(struct ufs_hba *hba)
133133
{
134134
if (hba->vops && hba->vops->exit)
135-
return hba->vops->exit(hba);
135+
hba->vops->exit(hba);
136136
}
137137

138138
static inline u32 ufshcd_vops_get_ufs_hci_version(struct ufs_hba *hba)
@@ -211,15 +211,15 @@ static inline void ufshcd_vops_setup_task_mgmt(struct ufs_hba *hba,
211211
int tag, u8 tm_function)
212212
{
213213
if (hba->vops && hba->vops->setup_task_mgmt)
214-
return hba->vops->setup_task_mgmt(hba, tag, tm_function);
214+
hba->vops->setup_task_mgmt(hba, tag, tm_function);
215215
}
216216

217217
static inline void ufshcd_vops_hibern8_notify(struct ufs_hba *hba,
218218
enum uic_cmd_dme cmd,
219219
enum ufs_notify_change_status status)
220220
{
221221
if (hba->vops && hba->vops->hibern8_notify)
222-
return hba->vops->hibern8_notify(hba, cmd, status);
222+
hba->vops->hibern8_notify(hba, cmd, status);
223223
}
224224

225225
static inline int ufshcd_vops_apply_dev_quirks(struct ufs_hba *hba)

0 commit comments

Comments
 (0)