scsi: ufs: ufs-exynos: Fix static checker warning
authorAlim Akhtar <alim.akhtar@samsung.com>
Thu, 19 Aug 2021 17:11:31 +0000 (22:41 +0530)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 25 Aug 2021 03:10:14 +0000 (23:10 -0400)
clk_get_rate() returns unsigned long and currently this driver stores the
return value in u32 type, resulting the below warning:

Fixed smatch warnings:

        drivers/scsi/ufs/ufs-exynos.c:286 exynos_ufs_get_clk_info()
        warn: wrong type for 'ufs->mclk_rate' (should be 'ulong')

        drivers/scsi/ufs/ufs-exynos.c:287 exynos_ufs_get_clk_info()
        warn: wrong type for 'pclk_rate' (should be 'ulong')

Link: https://lore.kernel.org/r/20210819171131.55912-1-alim.akhtar@samsung.com
Fixes: 55f4b1f73631 ("scsi: ufs: ufs-exynos: Add UFS host support for Exynos SoCs")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/ufs/ufs-exynos.c
drivers/scsi/ufs/ufs-exynos.h

index 5aa096e..a14dd8c 100644 (file)
@@ -260,7 +260,7 @@ static int exynos_ufs_get_clk_info(struct exynos_ufs *ufs)
        struct ufs_hba *hba = ufs->hba;
        struct list_head *head = &hba->clk_list_head;
        struct ufs_clk_info *clki;
-       u32 pclk_rate;
+       unsigned long pclk_rate;
        u32 f_min, f_max;
        u8 div = 0;
        int ret = 0;
@@ -299,7 +299,7 @@ static int exynos_ufs_get_clk_info(struct exynos_ufs *ufs)
        }
 
        if (unlikely(pclk_rate < f_min || pclk_rate > f_max)) {
-               dev_err(hba->dev, "not available pclk range %d\n", pclk_rate);
+               dev_err(hba->dev, "not available pclk range %lu\n", pclk_rate);
                ret = -EINVAL;
                goto out;
        }
index 67505fe..dadf4fd 100644 (file)
@@ -184,7 +184,7 @@ struct exynos_ufs {
        u32 pclk_div;
        u32 pclk_avail_min;
        u32 pclk_avail_max;
-       u32 mclk_rate;
+       unsigned long mclk_rate;
        int avail_ln_rx;
        int avail_ln_tx;
        int rx_sel_idx;