On Mon, Mar 16, 2015 at 08:15:26AM -0000, sricharan@xxxxxxxxxxxxxx wrote: > Hi, > > <snip> > > > >> > >> > +static int adm_get_blksize(unsigned int burst) > >> > +{ > >> > + int ret; > >> > + > >> > + switch (burst) { > >> > + case 16: > >> > + ret = 0; > >> > + break; > >> > + case 32: > >> > + ret = 1; > >> > + break; > >> > + case 64: > >> > + ret = 2; > >> > + break; > >> > + case 128: > >> > + ret = 3; > >> > + break; > >> > + case 192: > >> > + ret = 4; > >> > + break; > >> > + case 256: > >> > + ret = 5; > >> > + break; > >> ffs(burst>>4) ? > > > > that should work nicely. thanks. > > > Will not work for 192, 256 ? you are right. I'll have to separate those out into 2 more cases. Good catch! -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html