On 10/27/10 11:01, Saeed Bishara wrote: > +static void dove_mpp_conf_grp(int num, int sel, u32 *mpp_ctrl) > +{ > + int start = dove_mpp_grp[num].start; > + int end = dove_mpp_grp[num].start; > Shouldn't that be dove_mpp_grp[num].end ? oops.. thanks, will fix and resend > saeed -- Sincerely yours, Mike.