* [PATCH 1/16] CRIS architecture: Correct compile errors
@ 2007-10-30 9:28 Jesper Nilsson
2007-10-30 22:14 ` Andrew Morton
0 siblings, 1 reply; 4+ messages in thread
From: Jesper Nilsson @ 2007-10-30 9:28 UTC (permalink / raw)
To: Andrew Morton, Mikael Starvik, Jesper Nilsson, linux-kernel
Remove include of asm/bitops.h, not needed here, corrects compilation error
(__le64 undefined).
Signed-off-by: Jesper Nilsson <jesper.nilsson@axis.com>
---
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 -- jesper.nilsson@axis.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/16] CRIS architecture: Correct compile errors
2007-10-30 9:28 [PATCH 1/16] CRIS architecture: Correct compile errors Jesper Nilsson
@ 2007-10-30 22:14 ` Andrew Morton
2007-10-31 10:47 ` Jesper Nilsson
0 siblings, 1 reply; 4+ messages in thread
From: Andrew Morton @ 2007-10-30 22:14 UTC (permalink / raw)
To: Jesper Nilsson; +Cc: mikael.starvik, jesper.nilsson, linux-kernel
On Tue, 30 Oct 2007 10:28:11 +0100
Jesper Nilsson <jesper.nilsson@axis.com> wrote:
> Remove include of asm/bitops.h, not needed here, corrects compilation error
> (__le64 undefined).
>
> Signed-off-by: Jesper Nilsson <jesper.nilsson@axis.com>
>
> ---
> 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.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/16] CRIS architecture: Correct compile errors
2007-10-30 22:14 ` Andrew Morton
@ 2007-10-31 10:47 ` Jesper Nilsson
2007-10-31 10:55 ` Jesper Nilsson
0 siblings, 1 reply; 4+ messages in thread
From: Jesper Nilsson @ 2007-10-31 10:47 UTC (permalink / raw)
To: Andrew Morton; +Cc: mikael.starvik, linux-kernel
On Tue, Oct 30, 2007 at 03:14:00PM -0700, Andrew Morton wrote:
> On Tue, 30 Oct 2007 10:28:11 +0100
> Jesper Nilsson <jesper.nilsson@axis.com> 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 -- jesper.nilsson@axis.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/16] CRIS architecture: Correct compile errors
2007-10-31 10:47 ` Jesper Nilsson
@ 2007-10-31 10:55 ` Jesper Nilsson
0 siblings, 0 replies; 4+ messages in thread
From: Jesper Nilsson @ 2007-10-31 10:55 UTC (permalink / raw)
To: Andrew Morton; +Cc: mikael.starvik, linux-kernel
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 <jesper.nilsson@axis.com> 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 <jesper.nilsson@axis.com>
---
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 -- jesper.nilsson@axis.com
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-10-31 10:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-30 9:28 [PATCH 1/16] CRIS architecture: Correct compile errors Jesper Nilsson
2007-10-30 22:14 ` Andrew Morton
2007-10-31 10:47 ` Jesper Nilsson
2007-10-31 10:55 ` Jesper Nilsson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).