From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v2 04/31] arm64: MMU definitions Date: Wed, 15 Aug 2012 18:34:46 +0200 Message-ID: References: <1344966752-16102-1-git-send-email-catalin.marinas@arm.com> <1344966752-16102-5-git-send-email-catalin.marinas@arm.com> <201208151330.02112.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-qc0-f174.google.com ([209.85.216.174]:60382 "EHLO mail-qc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755693Ab2HOQer (ORCPT ); Wed, 15 Aug 2012 12:34:47 -0400 In-Reply-To: <201208151330.02112.arnd@arndb.de> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arnd Bergmann Cc: Catalin Marinas , linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Will Deacon On Wed, Aug 15, 2012 at 3:30 PM, Arnd Bergmann wrote: >> +#define TCR_IPS_40BIT (2 << 32) By default, constants are int, i.e. 32-bit. So you must write 2ULL << 32 >> +#define TCR_ASID16 (1 << 36) 1ULL > As a matter of coding style, I would much prefer tables like this to be > written as > > #define TCR_IRGN_MASK 0x0000000003000300 0x0000000003000300ULL, to be safe Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds