On Sat, Aug 20, 2011 at 06:01:29PM +0200, Heiko Stübner wrote: > +/* i2s-ref > + * > + * i2s bus reference clock, selectable from external, esysclk or epllref > + * > + * Note, this used to be two clocks, but was compressed into one. > +*/ > + > +struct clk *clk_i2s_srclist[] = { > + [0] = &clk_i2s_eplldiv.clk, > + [1] = &clk_i2s_ext, > + [2] = &clk_epllref.clk, > + [3] = &clk_epllref.clk, > +}; Is there any reason not to make this static (have you run your patch through checkpatch.pl ?) -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html