From: Paul M. <pau...@re...> - 2007-07-06 17:34:24
|
On Thu, Jun 21, 2007 at 09:45:51PM +0900, Paul Mundt wrote: > On Thu, Jun 21, 2007 at 07:34:19PM +0900, Katsuya MATSUBARA wrote: > > I found an inconsistent declaration > > in arch/sh/lib/div-generic.c in 2.6.21. > > > > the function 'div64_32()' in arch/sh/lib/div64-generic.c > > returns a value of u64 type, > > but the declaration of that function is > > > > extern uint32_t __div64_32(uint64_t *dividend, uint32_t divisor); > > ^^^^^^^^ > > in include/asm-generic/div64.h > > (which is referred by include/asm-sh/div64.h). > > > Yes, div64_32() should be uint32_t, the result is passed back in r0, so > the u64 thing is pointless. I'll queue up a patch, but probably won't get > around to looking at it until after OLS. > Pushed out a fix to current git. |