diff options
| author |   Guenter Roeck <linux@roeck-us.net> | 2018-07-23 14:41:38 -0700 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-07-24 14:23:00 +0200 |
| commit | f371a7c17aaa47dd7aedc89ef3cd1d5638c03f5e (patch) | |
| tree | 72199ebd1c7fb5ed22a74b1e82fa7daef103736e /drivers/android | |
| parent | android: binder_alloc: Include asm/cacheflush.h after linux/ include files (diff) | |
| download | linux-dev-f371a7c17aaa47dd7aedc89ef3cd1d5638c03f5e.tar.xz linux-dev-f371a7c17aaa47dd7aedc89ef3cd1d5638c03f5e.zip | |
android: binder: Include asm/cacheflush.h after linux/ include files
If asm/cacheflush.h is included first, the following build warnings are
seen with sparc32 builds.
In file included from arch/sparc/include/asm/cacheflush.h:11:0,
from drivers/android/binder.c:54:
arch/sparc/include/asm/cacheflush_32.h:40:37: warning:
'struct page' declared inside parameter list will not be visible
outside of this definition or declaration
Moving the asm/ include after linux/ includes solves the problem.
Suggested-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/android')
| -rw-r--r-- | drivers/android/binder.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/android/binder.c b/drivers/android/binder.c index 95283f3bb51c..1cc2fa16af8b 100644 --- a/drivers/android/binder.c +++ b/drivers/android/binder.c @@ -51,7 +51,6 @@ #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt -#include <asm/cacheflush.h> #include <linux/fdtable.h> #include <linux/file.h> #include <linux/freezer.h> @@ -73,6 +72,9 @@ #include <linux/spinlock.h> #include <uapi/linux/android/binder.h> + +#include <asm/cacheflush.h> + #include "binder_alloc.h" #include "binder_trace.h" |