You've already forked linux-rockchip
mirror of
https://github.com/armbian/linux-rockchip.git
synced 2026-01-06 11:08:10 -08:00
scsi: ufs: ufs-exynos: Fix static checker warning
[ Upstream commit313bf281f2] 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:55f4b1f736("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> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
bda5602c1c
commit
4c0c4f7021
@@ -259,7 +259,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;
|
||||
@@ -298,7 +298,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;
|
||||
}
|
||||
|
||||
@@ -197,7 +197,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;
|
||||
|
||||
Reference in New Issue
Block a user