Hi Tomasz, On 19 December 2013 17:15, Tomasz Figa <t.figa@xxxxxxxxxxx> wrote: > Hi Rahul, > > On Friday 06 of December 2013 21:26:30 Rahul Sharma wrote: >> Add support for pll2650xx in samsung pll file. This pll variant >> is close to pll36xx but uses CON2 registers instead of CON1. > > If the ops are otherwise idential, why not reuse the ops for pll36xx > and use CON1 or CON2 register conditionally based on pll->type field? > (Just as it is already done for pll4600, 4650 and 4650c.) > I verified the difference and found that pll2650xx is fairly different in terms of Bit Fields, Con2 register, and additional PLL config bits than pll36xx. Due to this, I have to add lot of if-else code based on pll type which doesn't looks clean. Regards, Rahul Sharma. > Best regards, > Tomasz > -- 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