On 08/24/2016 12:01 PM, Stephen Boyd wrote: > On 08/11, Rajendra Nayak wrote: >> diff --git a/drivers/clk/qcom/common.c b/drivers/clk/qcom/common.c >> index f7c226a..6bf5abd 100644 >> --- a/drivers/clk/qcom/common.c >> +++ b/drivers/clk/qcom/common.c >> @@ -25,6 +25,14 @@ >> #include "reset.h" >> #include "gdsc.h" >> >> +#define PLL_LOCK_COUNT_SHIFT 8 >> +#define PLL_LOCK_COUNT_MASK 0x3f >> +#define PLL_BIAS_COUNT_SHIFT 14 >> +#define PLL_BIAS_COUNT_MASK 0x3f >> +#define PLL_VOTE_FSM_ENA BIT(20) >> +#define PLL_DYN_FSM_ENA BIT(20) >> +#define PLL_VOTE_FSM_RESET BIT(21) > > These appear but aren't deleted from anywhere else so I suspect > now we have two copies of these defines somewhere? Perhaps this > should go into common.h as well? yes, these are used in clk-pll.c as well, so could as well move them over to common.h instead of having two copies > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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