From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757194AbbJHO54 (ORCPT ); Thu, 8 Oct 2015 10:57:56 -0400 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:39033 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933191AbbJHOtj (ORCPT ); Thu, 8 Oct 2015 10:49:39 -0400 Subject: Re: [PATCH] serial: atmel: fix compiler warning on address cast To: Alexandre Belloni , Russell King - ARM Linux References: <1444064452-12285-1-git-send-email-andre.przywara@arm.com> <20151008090148.GE3485@piout.net> <20151008093749.GB32532@n2100.arm.linux.org.uk> <20151008103835.GF3485@piout.net> Cc: Greg Kroah-Hartman , Jiri Slaby , Nicolas Ferre , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= , linux-arm-kernel@lists.infradead.org From: Andre Przywara X-Enigmail-Draft-Status: N1110 Organization: ARM Ltd. Message-ID: <56168292.5030207@arm.com> Date: Thu, 8 Oct 2015 15:49:54 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20151008103835.GF3485@piout.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 08/10/15 11:38, Alexandre Belloni wrote: > On 08/10/2015 at 10:37:49 +0100, Russell King - ARM Linux wrote : >> On Thu, Oct 08, 2015 at 11:01:48AM +0200, Alexandre Belloni wrote: >>> On 05/10/2015 at 18:00:52 +0100, Andre Przywara wrote : >>>> Turning on KVM and LPAE support on top of a multi_v7_defconfig will >>>> produce a compiler warning in the Atmel serial driver: >>>> drivers/tty/serial/atmel_serial.c: In function 'atmel_verify_port': >>>> drivers/tty/serial/atmel_serial.c:2299:6: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] >>>> if ((void *)port->mapbase != ser->iomem_base) >>>> ^ >>>> >>>> Fix that by using the cast on the right hand side instead, as similar >>>> code already does in other drivers. >>>> >>>> Signed-off-by: Andre Przywara >>> Acked-by: Alexandre Belloni >> >> Hmm, this actually points at a problem, and the cast screems out about it. >> Think about it, can you guess what it is? Russell, I'd really appreciate if we wouldn't have to play "guess-what-I-am-thinking" here. Can't you just say what's bothering you? > My understanding of the issue is that port->mapbase is a 64bit > resource_size_t, and is larger than void * hence the warning. Casting > iomem_base to unsigned long seems safe to me, am I wrong? That is what I found out as well: it's comparing a physical address with a pointer. And I tried to get to the root of this mismatch and fix it there, but it turned out to be really old code to allow userspace to setup serial ports by providing the MMIO address (IIRC this was for old PC-style non-PnP serial ports some 20 years ago). This whole concept is really outdated and probably does not make sense at all for most drivers (except 8250 on i386). Fixing this is a different task, I think, since it would impact most serial drivers. I am happy to tackle this later, btw. But this patch was just to fix the compiler warning I see every time now and still meant for 4.3, so I kept it deliberately simple and contained. Is that a solution we can agree on? Cheers, Andre.