Remove include of asm/bitops.h, not needed here, corrects compilation error
(__le64 undefined).
Signed-off-by: Jesper Nilsson <[email protected]>
---
posix_types.h | 1 -
1 file changed, 1 deletion(-)
--- clean_linux-2.6.23/include/asm-cris/posix_types.h 2007-10-09 22:31:38.000000000 +0200
+++ linux-2.6.23/include/asm-cris/posix_types.h 2007-10-25 11:32:56.000000000 +0200
@@ -52,7 +52,6 @@
} __kernel_fsid_t;
#ifdef __KERNEL__
-#include <asm/bitops.h>
#undef __FD_SET
#define __FD_SET(fd,fdsetp) set_bit(fd, (void *)(fdsetp))
/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]
On Tue, 30 Oct 2007 10:28:11 +0100
Jesper Nilsson <[email protected]> wrote:
> Remove include of asm/bitops.h, not needed here, corrects compilation error
> (__le64 undefined).
>
> Signed-off-by: Jesper Nilsson <[email protected]>
>
> ---
> posix_types.h | 1 -
> 1 file changed, 1 deletion(-)
>
> --- clean_linux-2.6.23/include/asm-cris/posix_types.h 2007-10-09 22:31:38.000000000 +0200
> +++ linux-2.6.23/include/asm-cris/posix_types.h 2007-10-25 11:32:56.000000000 +0200
> @@ -52,7 +52,6 @@
> } __kernel_fsid_t;
>
> #ifdef __KERNEL__
> -#include <asm/bitops.h>
>
> #undef __FD_SET
> #define __FD_SET(fd,fdsetp) set_bit(fd, (void *)(fdsetp))
>
This doesn't apply and I'm not sure that it is still needed or correct.
I'll drop it.
Please always prepare and test patches against the latest kernel. 2.6.23
is very much _not_ the latest kernel - there is a 50MB diff between 2.6.23
and 2.6.24-rc1. That's a lot of difference.
On Tue, Oct 30, 2007 at 03:14:00PM -0700, Andrew Morton wrote:
> On Tue, 30 Oct 2007 10:28:11 +0100
> Jesper Nilsson <[email protected]> wrote:
>
> > Remove include of asm/bitops.h, not needed here, corrects compilation error
> > (__le64 undefined).
> [snip]
>
> This doesn't apply and I'm not sure that it is still needed or correct.
>
> I'll drop it.
Unfortunately it is still needed, I'll resend a fresh patch
against Linus git tree.
Compile error:
CC arch/cris/kernel/asm-offsets.s
In file included from include/asm/byteorder.h:23,
from include/asm-generic/bitops/le.h:5,
from include/asm-generic/bitops/ext2-non-atomic.h:4,
from include/asm/bitops.h:163,
from include/linux/bitops.h:17,
from include/asm/posix_types.h:55,
from include/linux/posix_types.h:47,
from include/linux/types.h:11,
from include/linux/capability.h:16,
from include/linux/sched.h:49,
from arch/cris/kernel/asm-offsets.c:1:
include/linux/byteorder/little_endian.h:43: parse error before "__cpu_to_le64p"
include/linux/byteorder/little_endian.h:44: warning: return type defaults to `int'
include/linux/byteorder/little_endian.h: In function `__cpu_to_le64p':
include/linux/byteorder/little_endian.h:45: `__le64' undeclared (first use in this function)
....
> Please always prepare and test patches against the latest kernel. 2.6.23
> is very much _not_ the latest kernel - there is a 50MB diff between 2.6.23
> and 2.6.24-rc1. That's a lot of difference.
I will do so from now on.
/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]
On Wed, Oct 31, 2007 at 11:47:38AM +0100, Jesper Nilsson wrote:
> On Tue, Oct 30, 2007 at 03:14:00PM -0700, Andrew Morton wrote:
> > On Tue, 30 Oct 2007 10:28:11 +0100
> > Jesper Nilsson <[email protected]> wrote:
> > > Remove include of asm/bitops.h, not needed here, corrects compilation error
> > > (__le64 undefined).
> > [snip]
> >
> > This doesn't apply and I'm not sure that it is still needed or correct.
> >
> > I'll drop it.
>
> Unfortunately it is still needed, I'll resend a fresh patch
> against Linus git tree.
>
> Compile error:
>
> CC arch/cris/kernel/asm-offsets.s
> In file included from include/asm/byteorder.h:23,
> from include/asm-generic/bitops/le.h:5,
> from include/asm-generic/bitops/ext2-non-atomic.h:4,
> from include/asm/bitops.h:163,
> from include/linux/bitops.h:17,
> from include/asm/posix_types.h:55,
> from include/linux/posix_types.h:47,
> from include/linux/types.h:11,
> from include/linux/capability.h:16,
> from include/linux/sched.h:49,
> from arch/cris/kernel/asm-offsets.c:1:
> include/linux/byteorder/little_endian.h:43: parse error before "__cpu_to_le64p"
> include/linux/byteorder/little_endian.h:44: warning: return type defaults to `int'
> include/linux/byteorder/little_endian.h: In function `__cpu_to_le64p':
> include/linux/byteorder/little_endian.h:45: `__le64' undeclared (first use in this function)
> ....
Remove include of asm/bitops.h, not needed here, corrects compilation error
(__le64 undeclared).
Signed-off-by: Jesper Nilsson <[email protected]>
---
posix_types.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/asm-cris/posix_types.h b/include/asm-cris/posix_types.h
index 92000d0..3a5e4c4 100644
--- a/include/asm-cris/posix_types.h
+++ b/include/asm-cris/posix_types.h
@@ -52,7 +52,6 @@ typedef struct {
} __kernel_fsid_t;
#ifdef __KERNEL__
-#include <linux/bitops.h>
#undef __FD_SET
#define __FD_SET(fd,fdsetp) set_bit(fd, (void *)(fdsetp))
/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]