On Friday 20 December 2013 05:55 PM, Russell King - ARM Linux wrote: > On Fri, Dec 20, 2013 at 05:39:25PM -0500, Santosh Shilimkar wrote: >> Russell, >> >> On Friday 13 December 2013 07:38 PM, Santosh Shilimkar wrote: >>> Building ARM with NO_BOOTMEM generates below warning. >>> >>> mm/nobootmem.c: In function _____free_pages_memory___: >>> mm/nobootmem.c:88:11: warning: comparison of distinct pointer types lacks a >>> cast >>> >>> order = min(MAX_ORDER - 1UL, __ffs(start)); >>> >>> ARM's __ffs() differs from other architectures in that it ends up being >>> an int, whereas almost everyone else is unsigned long. >>> >>> So fix ARMs __ffs() to conform to other architectures. Suggested by >>> Russell King <[email protected]> >>> >>> Some more details in below thread - >>> https://lkml.org/lkml/2013/12/9/807 >>> >>> Cc: Andrew Morton <[email protected]> >>> Cc: Russell King <[email protected]> >>> Signed-off-by: Santosh Shilimkar <[email protected]> >>> --- >> Is this patch inline with what we discussed off-list ? > > It is. > >> If you ack it, it can go into the Andrews tree to kill that one last >> warning with the memblock series. Thanks > > Acked-by: Russell King <[email protected]> > Great !! Andrew, Can you please pick the $subject patch as well in your mm tree ? Thanks
regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
