From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: of_iomap() matched with plan iounmap() Date: Fri, 19 Aug 2011 06:26:09 -0700 (PDT) Message-ID: <20110819.062609.1865751197015675220.davem@davemloft.net> References: <20110818170226.GA16721@huya.qualcomm.com> <20110818.203447.462383652600224162.davem@davemloft.net> <201108191426.19152.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201108191426.19152.arnd@arndb.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: arnd@arndb.de Cc: davidb@codeaurora.org, devicetree-discuss@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org From: Arnd Bergmann Date: Fri, 19 Aug 2011 14:26:18 +0200 > We could of course change all existing users of of_iounmap on sparc to use > the simpler prototype, because it also just calls iounmap. Check again, on sparc64 it needs the resource to release it. Only the 32-bit version on sparc evaluates to just a plain iounmap().