On 08/21/2013 07:17 AM, Yadwinder Singh Brar wrote: >> }; >> >> +static struct __initdata samsung_pll_clock exynos4210_plls[nr_plls] = { > > NIT: __initdata should be at last. It's helpful if you only quote as much of the patch as you need to in order to reply to it; it wasn't obvious in your message that you had no comments on the rest of the patch that you quoted, so people have to scroll through it looking carefully for any other comments. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html