From mboxrd@z Thu Jan 1 00:00:00 1970 From: "H. Peter Anvin" Subject: Re: [PATCH] ACPI: use ioremap_cache() Date: Mon, 27 Dec 2010 12:12:34 -0800 Message-ID: <4D18F332.5090904@zytor.com> References: <20101214220932.GA1206@sgi.com> <20101215024136.GB19601@sgi.com> <20101215164635.GA27373@sgi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from terminus.zytor.com ([198.137.202.10]:41897 "EHLO mail.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751132Ab0L0UMn (ORCPT ); Mon, 27 Dec 2010 15:12:43 -0500 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Tony Luck Cc: Len Brown , Jack Steiner , Henrique de Moraes Holschuh , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, gbeshers@sgi.com On 12/27/2010 11:42 AM, Tony Luck wrote: > On Fri, Dec 17, 2010 at 12:08 AM, Len Brown wrote: >> No change on ia64. > > Apart from breaking the build :-( > >> +++ b/arch/ia64/include/asm/io.h > ... >> +static inline void __iomem * ioremap_cache (unsigned long phys_addr, unsigned long size) >> +{ >> + return ioremap(unsigned long phys_addr, unsigned long size); >> +} > > Cut & paste issue? ... the compiler would be a lot happier > without the "unsigned long"s in the call to ioremap(). > LOL... In the long(er) run it might be worth considering having the same policy on IA64 as on x86 -- ioremap() meaning ioremap_nocache(). -hpa