scsi: ufs: ufs-exynos: Remove an unnecessary NULL check
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 26 Jun 2020 10:51:33 +0000 (13:51 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 30 Jun 2020 01:33:20 +0000 (21:33 -0400)
The "head" pointer can't be NULL because it points to an address in the
middle of a ufs_hba struct.  Looking at this code, probably someone would
wonder if the intent was to check whether "hba" is NULL, but "hba" isn't
NULL and the check can just be removed.

Link: https://lore.kernel.org/r/20200626105133.GF314359@mwanda
Acked-by: Alim Akhtar <alim.akhtar@samsung.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/ufs/ufs-exynos.c

index b0796066a449dcbf56581b384f570353f380e526..32b61ba77241eee587eb1c670371d15f784fca9c 100644 (file)
@@ -264,7 +264,7 @@ static int exynos_ufs_get_clk_info(struct exynos_ufs *ufs)
        u8 div = 0;
        int ret = 0;
 
-       if (!head || list_empty(head))
+       if (list_empty(head))
                goto out;
 
        list_for_each_entry(clki, head, list) {