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

[Dan Carpenter <dan.carpenter@xxxxxxxxxx>]

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
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html