2006-12-08 09:45:33

by Al Viro

[permalink] [raw]
Subject: [PATCH] uml problems with linux/io.h

Remove useless includes of linux/io.h, don't even try to build iomap_copy
on uml (it doesn't have readb() et.al., so...)

Signed-off-by: Al Viro <[email protected]>
---
crypto/blkcipher.c | 1 -
lib/Kconfig | 5 +++++
lib/Makefile | 3 ++-
lib/ioremap.c | 1 -
4 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/crypto/blkcipher.c b/crypto/blkcipher.c
index 034c939..6e93004 100644
--- a/crypto/blkcipher.c
+++ b/crypto/blkcipher.c
@@ -17,7 +17,6 @@
#include <linux/crypto.h>
#include <linux/errno.h>
#include <linux/kernel.h>
-#include <linux/io.h>
#include <linux/module.h>
#include <linux/scatterlist.h>
#include <linux/seq_file.h>
diff --git a/lib/Kconfig b/lib/Kconfig
index 734ce95..77eb379 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -97,4 +97,9 @@ #
config PLIST
boolean

+config IOMAP_COPY
+ boolean
+ depends on !UML
+ default y
+
endmenu
diff --git a/lib/Makefile b/lib/Makefile
index fea8f90..92ca7fe 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -12,13 +12,14 @@ lib-$(CONFIG_SMP) += cpumask.o

lib-y += kobject.o kref.o kobject_uevent.o klist.o

-obj-y += sort.o parser.o halfmd4.o iomap_copy.o debug_locks.o random32.o
+obj-y += sort.o parser.o halfmd4.o debug_locks.o random32.o

ifeq ($(CONFIG_DEBUG_KOBJECT),y)
CFLAGS_kobject.o += -DDEBUG
CFLAGS_kobject_uevent.o += -DDEBUG
endif

+obj-$(CONFIG_IOMAP_COPY) += iomap_copy.o
obj-$(CONFIG_DEBUG_LOCKING_API_SELFTESTS) += locking-selftest.o
obj-$(CONFIG_DEBUG_SPINLOCK) += spinlock_debug.o
lib-$(CONFIG_RWSEM_GENERIC_SPINLOCK) += rwsem-spinlock.o
diff --git a/lib/ioremap.c b/lib/ioremap.c
index 99fa277..a9e4415 100644
--- a/lib/ioremap.c
+++ b/lib/ioremap.c
@@ -5,7 +5,6 @@
*
* (C) Copyright 1995 1996 Linus Torvalds
*/
-#include <linux/io.h>
#include <linux/vmalloc.h>
#include <linux/mm.h>

--
1.4.2.GIT


2006-12-08 16:41:49

by Jeff Dike

[permalink] [raw]
Subject: Re: [PATCH] uml problems with linux/io.h

On Fri, Dec 08, 2006 at 09:45:30AM +0000, Al Viro wrote:
> Remove useless includes of linux/io.h, don't even try to build iomap_copy
> on uml (it doesn't have readb() et.al., so...)
>
> Signed-off-by: Al Viro <[email protected]>
> ---
> crypto/blkcipher.c | 1 -
> lib/Kconfig | 5 +++++
> lib/Makefile | 3 ++-
> lib/ioremap.c | 1 -
> 4 files changed, 7 insertions(+), 3 deletions(-)

Acked-by: Jeff Dike <[email protected]>

--
Work email - jdike at linux dot intel dot com