Skip to contentSkip to conversation

clk: qcom: Add support for root clock generators (RCGs)

DISCUSSION Dan Carpenter 1 participant 1 message Apr 2, 2014 Kernel janitors

lore
Share & tools
lore
#0 lore raw

Hello Stephen Boyd,

The patch bcd61c0f535a: "clk: qcom: Add support for root clock generators (RCGs)" from Jan 15, 2014, leads to the following static checker warning:

	drivers/clk/qcom/clk-rcg2.c:79 clk_rcg2_get_parent()
	warn: signedness bug returning '(-22)'

drivers/clk/qcom/clk-rcg2.c

    61  static u8 clk_rcg2_get_parent(struct clk_hw *hw)
    62  {
    63          struct clk_rcg2 *rcg = to_clk_rcg2(hw);
    64          int num_parents = __clk_get_num_parents(hw->clk);
    65          u32 cfg;
    66          int i, ret;
    67  
    68          ret = regmap_read(rcg->clkr.regmap, rcg->cmd_rcgr + CFG_REG, &cfg);
    69          if (ret)
    70                  return ret;
    71  
    72          cfg &= CFG_SRC_SEL_MASK;
    73          cfg >>= CFG_SRC_SEL_SHIFT;
    74  
    75          for (i = 0; i < num_parents; i++)
    76                  if (cfg = rcg->parent_map[i])
    77                          return i;
    78  
    79          return -EINVAL;
                ^^^^^^^^^^^^^^
    80  }

See also:

drivers/clk/qcom/clk-rcg.c:57 clk_rcg_get_parent()
warn: signedness bug returning '(-22)'

drivers/clk/qcom/clk-rcg.c:86 clk_dyn_rcg_get_parent()
warn: signedness bug returning '(-22)'

regards,
dan carpenter