Commit 8224ca19 authored by Haavard Skinnemoen's avatar Haavard Skinnemoen

[AVR32] Fix compile error with gcc 4.1

gcc 4.1 doesn't seem to like const variables as inline assembly
outputs. Drop support for reading 64-bit values using get_user() so
that we can use an unsigned long to hold the result regardless of the
actual size. This should be safe since many architectures, including
i386, doesn't support reading 64-bit values with get_user().
Signed-off-by: default avatarHaavard Skinnemoen <hskinnemoen@atmel.com>
parent a4022b0d
...@@ -181,24 +181,23 @@ extern int __put_user_bad(void); ...@@ -181,24 +181,23 @@ extern int __put_user_bad(void);
#define __get_user_nocheck(x, ptr, size) \ #define __get_user_nocheck(x, ptr, size) \
({ \ ({ \
typeof(*(ptr)) __gu_val = (typeof(*(ptr)) __force)0; \ unsigned long __gu_val = 0; \
int __gu_err = 0; \ int __gu_err = 0; \
\ \
switch (size) { \ switch (size) { \
case 1: __get_user_asm("ub", __gu_val, ptr, __gu_err); break; \ case 1: __get_user_asm("ub", __gu_val, ptr, __gu_err); break; \
case 2: __get_user_asm("uh", __gu_val, ptr, __gu_err); break; \ case 2: __get_user_asm("uh", __gu_val, ptr, __gu_err); break; \
case 4: __get_user_asm("w", __gu_val, ptr, __gu_err); break; \ case 4: __get_user_asm("w", __gu_val, ptr, __gu_err); break; \
case 8: __get_user_asm("d", __gu_val, ptr, __gu_err); break; \
default: __gu_err = __get_user_bad(); break; \ default: __gu_err = __get_user_bad(); break; \
} \ } \
\ \
x = __gu_val; \ x = (typeof(*(ptr)))__gu_val; \
__gu_err; \ __gu_err; \
}) })
#define __get_user_check(x, ptr, size) \ #define __get_user_check(x, ptr, size) \
({ \ ({ \
typeof(*(ptr)) __gu_val = (typeof(*(ptr)) __force)0; \ unsigned long __gu_val = 0; \
const typeof(*(ptr)) __user * __gu_addr = (ptr); \ const typeof(*(ptr)) __user * __gu_addr = (ptr); \
int __gu_err = 0; \ int __gu_err = 0; \
\ \
...@@ -216,10 +215,6 @@ extern int __put_user_bad(void); ...@@ -216,10 +215,6 @@ extern int __put_user_bad(void);
__get_user_asm("w", __gu_val, __gu_addr, \ __get_user_asm("w", __gu_val, __gu_addr, \
__gu_err); \ __gu_err); \
break; \ break; \
case 8: \
__get_user_asm("d", __gu_val, __gu_addr, \
__gu_err); \
break; \
default: \ default: \
__gu_err = __get_user_bad(); \ __gu_err = __get_user_bad(); \
break; \ break; \
...@@ -227,7 +222,7 @@ extern int __put_user_bad(void); ...@@ -227,7 +222,7 @@ extern int __put_user_bad(void);
} else { \ } else { \
__gu_err = -EFAULT; \ __gu_err = -EFAULT; \
} \ } \
x = __gu_val; \ x = (typeof(*(ptr)))__gu_val; \
__gu_err; \ __gu_err; \
}) })
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment