From: Paul M. <le...@li...> - 2007-06-21 12:47:32
|
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. |