"Nori, Sekhar" <[email protected]> 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 <[email protected]> > > Thanks for the fix. You could use the subject: > > "davinci: signedness bug in davinci_aemif_setup_timing()" > > Other than that: > > Acked-by: Sekhar Nori <[email protected]>
Thanks, I fixed up the subject as Sekhar suggested. Applied, queuing for 2.6.38. Kevin _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
