Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754150Ab1EHOh4 (ORCPT ); Sun, 8 May 2011 10:37:56 -0400 Received: from netrider.rowland.org ([192.131.102.5]:53590 "HELO netrider.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1752310Ab1EHOhy (ORCPT ); Sun, 8 May 2011 10:37:54 -0400 Date: Sun, 8 May 2011 10:37:53 -0400 (EDT) From: Alan Stern X-X-Sender: stern@netrider.rowland.org To: Greg Dietsche cc: gregkh@suse.de, , , , , Subject: Re: [PATCH] usb: fix warning in usbtest module In-Reply-To: <1304825264-26853-1-git-send-email-gregory.dietsche@cuw.edu> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 761 Lines: 21 On Sat, 7 May 2011, Greg Dietsche wrote: > On amd64 unsigned is not as wide as pointer and this causes > a compiler warning. Switching to uintptr_t fixes the problem > in an arch independent manner. People tend to prefer to see non-typedef'ed type names, whenever possible. In this case, it would be enough to change the type to unsigned long. Lots of code throughout the kernel stores pointer values in unsigned long variables. I've never heard any recommendation for using uintptr_t instead. Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/