"Nori, Sekhar" <nsekhar@xxxxxx> writes: > Hi Nicolas, > > On Tue, Nov 16, 2010 at 00:10:28, Nicolas Kaiser wrote: >> aemif_calc_rate() can return a negative error value, so all the >> variables that get tested for this value need to be signed. >> >> The maximum bit width of WSETUP(WSETUP_MAX) appears to be 30 bits >> (0xf << 26). Using a signed instead of an unsigned integer >> shouldn't make a difference here. >> >> Signed-off-by: Nicolas Kaiser <nikai@xxxxxxxxx> > > Thanks for the fix. You could use the subject: > > "davinci: signedness bug in davinci_aemif_setup_timing()" > > Other than that: > > Acked-by: Sekhar Nori <nsekhar@xxxxxx> Thanks, I fixed up the subject as Sekhar suggested. Applied, queuing for 2.6.38. Kevin -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
