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 b079606..32b61ba 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) {