On Mon, May 4, 2015 at 9:37 AM, Rhyland Klein <rklein@xxxxxxxxxx> wrote: > From: Bill Huang <bilhuang@xxxxxxxxxx> > > New SoC's may have more then 3 MISC registers, so bump up the > array size and use a #define to be more informative about the value. > > Signed-off-by: Bill Huang <bilhuang@xxxxxxxxxx> > --- > drivers/clk/tegra/clk.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/clk/tegra/clk.h b/drivers/clk/tegra/clk.h > index 5759b8bfb80e..8e7361886cf9 100644 > --- a/drivers/clk/tegra/clk.h > +++ b/drivers/clk/tegra/clk.h > @@ -156,6 +156,8 @@ struct div_nmp { > u8 override_divp_shift; > }; > > +#define MAX_PLL_MISC_REG_COUNT 6 > + > /** > * struct clk_pll_params - PLL parameters > * > @@ -213,7 +215,7 @@ struct tegra_clk_pll_params { > u32 iddq_bit_idx; > u32 aux_reg; > u32 dyn_ramp_reg; > - u32 ext_misc_reg[3]; > + u32 ext_misc_reg[MAX_PLL_MISC_REG_COUNT]; > u32 pmc_divnm_reg; > u32 pmc_divp_reg; > u32 flags; Missing kernel doc above for ext_misc_reg and some other surrounding members. Otherwise, Reviewed-by: Benson Leung <bleung@xxxxxxxxxxxx> -- Benson Leung Software Engineer, Chrom* OS bleung@xxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html