From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: [PATCH] include/asm-generic/io.h: add 'UML' case just like 'no-MMU' Date: Wed, 26 Jun 2013 14:31:20 +0800 Message-ID: <51CA8AB8.7080504@asianux.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from intranet.asianux.com ([58.214.24.6]:28515 "EHLO intranet.asianux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751309Ab3FZGcN (ORCPT ); Wed, 26 Jun 2013 02:32:13 -0400 Sender: linux-arch-owner@vger.kernel.org List-ID: To: Jeff Dike , Richard Weinberger , Arnd Bergmann Cc: dhsharp@google.com, Mark Brown , "sfr@canb.auug.org.au" , Steven Rostedt , Ingo Molnar , uml-devel , uml-user , "linux-kernel@vger.kernel.org" , Linux-Arch =46or "User Mode Linux", it may enable 'MMU', but not need implement ioremap and iounmap, so "include/asm-generic/io.h" need notice this case to keep itself 'generic'. The related error (with allmodconfig, without pcap): CC [M] drivers/ptp/ptp_pch.o drivers/ptp/ptp_pch.c: In function =91pch_remove=92: drivers/ptp/ptp_pch.c:571:3: error: implicit declaration of function = =91iounmap=92 [-Werror=3Dimplicit-function-declaration] drivers/ptp/ptp_pch.c: In function =91pch_probe=92: drivers/ptp/ptp_pch.c:621:2: error: implicit declaration of function = =91ioremap=92 [-Werror=3Dimplicit-function-declaration] drivers/ptp/ptp_pch.c:621:13: warning: assignment makes pointer from = integer without a cast [enabled by default] cc1: some warnings being treated as errors Signed-off-by: Chen Gang --- arch/um/include/asm/Kbuild | 1 + include/asm-generic/io.h | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/um/include/asm/Kbuild b/arch/um/include/asm/Kbuild index b30f34a..a34ea5d 100644 --- a/arch/um/include/asm/Kbuild +++ b/arch/um/include/asm/Kbuild @@ -3,3 +3,4 @@ generic-y +=3D hw_irq.h irq_regs.h kdebug.h percpu.h se= ctions.h topology.h xor.h generic-y +=3D ftrace.h pci.h io.h param.h delay.h mutex.h current.h e= xec.h generic-y +=3D switch_to.h clkdev.h generic-y +=3D trace_clock.h +generic-y +=3D io.h diff --git a/include/asm-generic/io.h b/include/asm-generic/io.h index d5afe96..e80331d 100644 --- a/include/asm-generic/io.h +++ b/include/asm-generic/io.h @@ -303,10 +303,10 @@ static inline void *phys_to_virt(unsigned long ad= dress) /* * Change "struct page" to physical address. * - * This implementation is for the no-MMU case only... if you have an M= MU + * This implementation is for the no-MMU or UML case only... if you ha= ve an MMU * you'll need to provide your own definitions. */ -#ifndef CONFIG_MMU +#if !CONFIG_MMU || CONFIG_UML static inline void __iomem *ioremap(phys_addr_t offset, unsigned long = size) { return (void __iomem*) (unsigned long)offset; @@ -325,7 +325,7 @@ static inline void __iomem *ioremap(phys_addr_t off= set, unsigned long size) static inline void iounmap(void __iomem *addr) { } -#endif /* CONFIG_MMU */ +#endif /* !CONFIG_MMU || CONFIG_UML */ =20 #ifdef CONFIG_HAS_IOPORT #ifndef CONFIG_GENERIC_IOMAP --=20 1.7.7.6 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <51CA8AB8.7080504@asianux.com> Date: Wed, 26 Jun 2013 14:31:20 +0800 From: Chen Gang MIME-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Sender: linux-arch-owner@vger.kernel.org Subject: [PATCH] include/asm-generic/io.h: add 'UML' case just like 'no-MMU' To: Jeff Dike , Richard Weinberger , Arnd Bergmann Cc: dhsharp@google.com, Mark Brown , "sfr@canb.auug.org.au" , Steven Rostedt , Ingo Molnar , uml-devel , uml-user , "linux-kernel@vger.kernel.org" , Linux-Arch List-ID: =46or "User Mode Linux", it may enable 'MMU', but not need implement ioremap and iounmap, so "include/asm-generic/io.h" need notice this case to keep itself 'generic'. The related error (with allmodconfig, without pcap): CC [M] drivers/ptp/ptp_pch.o drivers/ptp/ptp_pch.c: In function =91pch_remove=92: drivers/ptp/ptp_pch.c:571:3: error: implicit declaration of function = =91iounmap=92 [-Werror=3Dimplicit-function-declaration] drivers/ptp/ptp_pch.c: In function =91pch_probe=92: drivers/ptp/ptp_pch.c:621:2: error: implicit declaration of function = =91ioremap=92 [-Werror=3Dimplicit-function-declaration] drivers/ptp/ptp_pch.c:621:13: warning: assignment makes pointer from = integer without a cast [enabled by default] cc1: some warnings being treated as errors Signed-off-by: Chen Gang --- arch/um/include/asm/Kbuild | 1 + include/asm-generic/io.h | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/um/include/asm/Kbuild b/arch/um/include/asm/Kbuild index b30f34a..a34ea5d 100644 --- a/arch/um/include/asm/Kbuild +++ b/arch/um/include/asm/Kbuild @@ -3,3 +3,4 @@ generic-y +=3D hw_irq.h irq_regs.h kdebug.h percpu.h se= ctions.h topology.h xor.h generic-y +=3D ftrace.h pci.h io.h param.h delay.h mutex.h current.h e= xec.h generic-y +=3D switch_to.h clkdev.h generic-y +=3D trace_clock.h +generic-y +=3D io.h diff --git a/include/asm-generic/io.h b/include/asm-generic/io.h index d5afe96..e80331d 100644 --- a/include/asm-generic/io.h +++ b/include/asm-generic/io.h @@ -303,10 +303,10 @@ static inline void *phys_to_virt(unsigned long ad= dress) /* * Change "struct page" to physical address. * - * This implementation is for the no-MMU case only... if you have an M= MU + * This implementation is for the no-MMU or UML case only... if you ha= ve an MMU * you'll need to provide your own definitions. */ -#ifndef CONFIG_MMU +#if !CONFIG_MMU || CONFIG_UML static inline void __iomem *ioremap(phys_addr_t offset, unsigned long = size) { return (void __iomem*) (unsigned long)offset; @@ -325,7 +325,7 @@ static inline void __iomem *ioremap(phys_addr_t off= set, unsigned long size) static inline void iounmap(void __iomem *addr) { } -#endif /* CONFIG_MMU */ +#endif /* !CONFIG_MMU || CONFIG_UML */ =20 #ifdef CONFIG_HAS_IOPORT #ifndef CONFIG_GENERIC_IOMAP --=20 1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-arch" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html