diff options
| author | Vipul Kumar <vipul.kumar@xilinx.com> | 2018-06-27 10:44:45 +0530 |
|---|---|---|
| committer | Michal Simek <michal.simek@xilinx.com> | 2018-07-19 10:49:53 +0200 |
| commit | b4f015845a5814ab56aee136bf2c7175c205c6c2 (patch) | |
| tree | 07f5918d4ad0296aa85b8cca8811eaf91bf6f6f7 /drivers/clk | |
| parent | 6da4f67ad09cd8b311d77b2b04e557b7ef65b56c (diff) | |
| download | u-boot-b4f015845a5814ab56aee136bf2c7175c205c6c2.tar.gz u-boot-b4f015845a5814ab56aee136bf2c7175c205c6c2.tar.xz u-boot-b4f015845a5814ab56aee136bf2c7175c205c6c2.zip | |
clk: zynqmp: Fixed the same if/else part error reported by coverity
This patch fixed the same if/else part error by adding the required
source select on the basis of is_pre_src check.
Signed-off-by: Vipul Kumar <vipul.kumar@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Diffstat (limited to 'drivers/clk')
| -rw-r--r-- | drivers/clk/clk_zynqmp.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/clk/clk_zynqmp.c b/drivers/clk/clk_zynqmp.c index a9e4500705..167f3f75a1 100644 --- a/drivers/clk/clk_zynqmp.c +++ b/drivers/clk/clk_zynqmp.c @@ -103,6 +103,8 @@ static const resource_size_t zynqmp_crl_apb_clkc_base = 0xff5e0020; #define PLLCTRL_BYPASS_SHFT 3 #define PLLCTRL_POST_SRC_SHFT 24 #define PLLCTRL_POST_SRC_MASK (0x7 << PLLCTRL_POST_SRC_SHFT) +#define PLLCTRL_PRE_SRC_SHFT 20 +#define PLLCTRL_PRE_SRC_MASK (0x7 << PLLCTRL_PRE_SRC_SHFT) #define NUM_MIO_PINS 77 @@ -310,8 +312,8 @@ static ulong zynqmp_clk_get_pll_src(ulong clk_ctrl, u32 src_sel; if (is_pre_src) - src_sel = (clk_ctrl & PLLCTRL_POST_SRC_MASK) >> - PLLCTRL_POST_SRC_SHFT; + src_sel = (clk_ctrl & PLLCTRL_PRE_SRC_MASK) >> + PLLCTRL_PRE_SRC_SHFT; else src_sel = (clk_ctrl & PLLCTRL_POST_SRC_MASK) >> PLLCTRL_POST_SRC_SHFT; |