From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ken Chen Subject: Re: [PATCH] sparc64: use unsigned long long for u64 Date: Tue, 23 Dec 2008 09:05:44 -0800 Message-ID: References: <20081222152247.b934ed5b.sfr@canb.auug.org.au> <20081222070426.GD29160@elte.hu> <20081222181932.5dd9514e.sfr@canb.auug.org.au> <20081222080341.GA18897@elte.hu> <20081222144319.1bfc0061.akpm@linux-foundation.org> <20081222230035.GB4074@uranus.ravnborg.org> <20081222151338.3b6ef997.akpm@linux-foundation.org> <20081223131746.GA7456@uranus.ravnborg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20081223131746.GA7456@uranus.ravnborg.org> Sender: sparclinux-owner@vger.kernel.org To: Sam Ravnborg Cc: Andrew Morton , David Miller , sparclinux , mingo@elte.hu, sfr@canb.auug.org.au, linux-kernel@vger.kernel.org, paulus@samba.org, tglx@linutronix.de, hpa@zytor.com, linux-next@vger.kernel.org List-Id: linux-next.vger.kernel.org On Tue, Dec 23, 2008 at 5:17 AM, Sam Ravnborg wrote: > Andrew Morton wrote: > > People keep on doing > > printk("%llu", some_u64); > > testing it only on x86_64 and this generates a warning storm on > powerpc, sparc64, etc. Because they use `long', not `long long'. > > Quite a few 64-bit architectures are using `long' for their > s64/u64 types. We should convert them all to `long long'. > > Update types.h so we use unsigned long long for u64 and > fix all warnings in sparc64 code. > Tested with an allnoconfig, defconfig and allmodconfig builds. > > This patch introduces additional warnings in several drivers. > These will be dealt with in separate patches. > > Signed-off-by: Sam Ravnborg > Cc: Andrew Morton > --- > > It may take a few days before the drivers gets fixed. > Christmas is approaching fast by now. > > Sam > > diff --git a/arch/sparc/include/asm/timer_64.h b/arch/sparc/include/asm/timer_64.h > index 5b779fd..ef3c368 100644 > --- a/arch/sparc/include/asm/timer_64.h > +++ b/arch/sparc/include/asm/timer_64.h > @@ -10,7 +10,7 @@ > #include > > struct sparc64_tick_ops { > - unsigned long (*get_tick)(void); > + unsigned long long (*get_tick)(void); wait, why does this need to be changed? People having problem with: u64 data; seq_printf(m, "%llu ", data); This compiles fine with arch that defines u64 to unsigned long long (like x86), but generate compile time warning on arch that defines u64 to unsigned long. Isn't that you just need to convert format string to %llu for u64 type? - Ken