On 08/05, Rajendra Nayak wrote: > > > On 08/04/2015 12:54 AM, Stephen Boyd wrote: > >On 07/28, Rajendra Nayak wrote: > >>+ > >>+#define PWR_ON_MASK BIT(31) > >>+#define EN_REST_WAIT_MASK GENMASK(23, 20) > >>+#define EN_FEW_WAIT_MASK GENMASK(19, 16) > >>+#define CLK_DIS_WAIT_MASK GENMASK(15, 12) > > > >This trips up some static checker... > > > >drivers/clk/qcom/gdsc.c:234:53: warning: cast truncates bits from > >constant value (fffffffffff006 becomes fffff006) > > > >So I think we can use GENMASK_ULL here instead. > > Sure. btw, what static checker is this? I did not see it with sparse. Pretty sure this was with sparse. Which now has me concerned that my sparse is old! $ sparse --version v0.5.0-47-g0f71312959ed But I updated to $ sparse --version v0.5.0-51-ga53cea28f0db And I still see it. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
