From patchwork Thu Apr 13 03:35:47 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael Ellerman X-Patchwork-Id: 9678675 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id CFC4460384 for ; Thu, 13 Apr 2017 03:36:28 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C37D32862C for ; Thu, 13 Apr 2017 03:36:28 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id B53EA2864D; Thu, 13 Apr 2017 03:36:28 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 845FD2862C for ; Thu, 13 Apr 2017 03:36:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=E3DSAN3+pPWdHMEabfV/ezgUYIs7wu/ux4bPSbqpseI=; b=Rkdu1ntPE9TgZ7 D8OhvyhepNBhFladjB0jC/8VTar/uYYxrPcMI1cEOwpMlGxmp4Drg2BpPg1NIQt175I+KboMS/yzj VlEbcosRN08HuZU5hRaLnwTUVJTxNkuczyO59TB/hjdCiBAVT6RdTmDUwxLqoS+UOJI/5deQAvnhO mnZHZk9dUWzj3IOXLBrvGfWWRW2ZynmADnoy392cM/pnp05EBpDBZbGMrBWoZwQ7csZ7KBMJnQy3v tWvsoILbLqp6uS+K1LMksP2ln03beUcQxdW7AdYawyTzxYu44fEtxU8+H6lkfjrA7MEhL2llMJjvN Mr0I2+Dn1L7bmlwB9TBQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1cyVYV-0006wZ-Fd; Thu, 13 Apr 2017 03:36:23 +0000 Received: from ozlabs.org ([2401:3900:2:1::2]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cyVYS-0006tv-29 for linux-arm-kernel@lists.infradead.org; Thu, 13 Apr 2017 03:36:22 +0000 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPSA id 3w3RJl0WlMz9rxm; Thu, 13 Apr 2017 13:35:51 +1000 (AEST) From: Michael Ellerman To: Benjamin Herrenschmidt , Lorenzo Pieralisi Subject: Re: [PATCH v3 21/32] powerpc: include default ioremap_nopost() implementation In-Reply-To: <1491952528.7236.26.camel@kernel.crashing.org> References: <20170411122923.6285-1-lorenzo.pieralisi@arm.com> <20170411122923.6285-22-lorenzo.pieralisi@arm.com> <1491917928.7236.8.camel@kernel.crashing.org> <20170411142456.GB6821@red-moon> <1491952528.7236.26.camel@kernel.crashing.org> User-Agent: Notmuch/0.21 (https://notmuchmail.org) Date: Thu, 13 Apr 2017 13:35:47 +1000 Message-ID: <87shld9dyk.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170412_203620_372980_1662FC26 X-CRM114-Status: GOOD ( 10.52 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arch@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Paul Mackerras , Bjorn Helgaas , linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Benjamin Herrenschmidt writes: > On Tue, 2017-04-11 at 15:24 +0100, Lorenzo Pieralisi wrote: >> Ok, point taken. BTW, may I ask you guys to have a look into this >> please ? >> >> https://lkml.org/lkml/2017/4/6/743 >> >> It is a side effect of this thread (v2), not sure why >> on powerpc has to include . > > Not sure how we ended up with that... it's odd indeed. > > Michael ? Any reason we can't just remove it ? No ... idea. Looks like it was added in: commit b41e5fffe8b81fc939067d8c1c195cc79115d5a3 Author: Emil Medve AuthorDate: Sat May 3 06:34:04 2008 +1000 Commit: Paul Mackerras CommitDate: Mon May 5 16:47:14 2008 +1000 [POWERPC] devres: Add devm_ioremap_prot() We provide an ioremap_flags, so this provides a corresponding devm_ioremap_prot. The slight name difference is at Ben Herrenschmidt's request as he plans on changing ioremap_flags to ioremap_prot in the future. Signed-off-by: Emil Medve Signed-off-by: Kumar Gala Acked-by: Tejun Heo Cc: Benjamin Herrenschmidt Signed-off-by: Andrew Morton Signed-off-by: Paul Mackerras I'll try removing it and see what breaks. cheers diff --git a/include/asm-powerpc/io.h b/include/asm-powerpc/io.h index afae0697e8ce..e0062d73db1c 100644 --- a/include/asm-powerpc/io.h +++ b/include/asm-powerpc/io.h @@ -18,6 +18,9 @@ extern int check_legacy_ioport(unsigned long base_port); #define _PNPWRP 0xa79 #define PNPBIOS_BASE 0xf000 +#include +#include + #include #include #include @@ -744,6 +747,9 @@ static inline void * bus_to_virt(unsigned long address) #define clrsetbits_8(addr, clear, set) clrsetbits(8, addr, clear, set) +void __iomem *devm_ioremap_prot(struct device *dev, resource_size_t offset, + size_t size, unsigned long flags); + #endif /* __KERNEL__ */ #endif /* _ASM_POWERPC_IO_H */