[PATCH] linux/types.h: Do not depend on __GNUC__ for __le64 and __be64

[PATCH] linux/types.h: Do not depend on __GNUC__ for __le64 and __be64

[Detlef Riekenberg]

Hi.
<linux/cdrom.h> is used in Wine, but failed to compile with pcc.
(__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)

There is no reason to restrict the typedefs with __GNUC__.
As reference, the typedefs for __u64 and __s64 where fixed
to be available with pcc on 02. Mai 2008 by H. Peter Anvin
(commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)

Wine: http://www.winehq.org
pcc:  http://pcc.ludd.ltu.se


Please CC me personally. Thanks

-- 
By by ... Detlef


>From 009605de95d2148c8c54ff83f72ebe0177b805ac Mon Sep 17 00:00:00 2001
From: Detlef Riekenberg <wine.dev@web.de>
Date: Mon, 29 Dec 2008 23:52:48 +0100
Subject: [PATCH] linux/types.h: Do not depend on __GNUC__ for __le64 and
__be64

Signed-off-by: Detlef Riekenberg <wine.dev@web.de>
---
 include/linux/types.h |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/include/linux/types.h b/include/linux/types.h
index 1d98330..ccf6742 100644
--- a/include/linux/types.h
+++ b/include/linux/types.h
@@ -181,10 +181,9 @@ typedef __u16 __bitwise __le16;
 typedef __u16 __bitwise __be16;
 typedef __u32 __bitwise __le32;
 typedef __u32 __bitwise __be32;
-#if defined(__GNUC__)
 typedef __u64 __bitwise __le64;
 typedef __u64 __bitwise __be64;
-#endif
+
 typedef __u16 __bitwise __sum16;
 typedef __u32 __bitwise __wsum;
 
-- 
1.5.4.3

Re: [PATCH] linux/types.h: Do not depend on __GNUC__ for __le64 and __be64

[H. Peter Anvin]

Detlef Riekenberg wrote:
> Hi.
> <linux/cdrom.h> is used in Wine, but failed to compile with pcc.
> (__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)
> 
> There is no reason to restrict the typedefs with __GNUC__.
> As reference, the typedefs for __u64 and __s64 where fixed
> to be available with pcc on 02. Mai 2008 by H. Peter Anvin
> (commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)
> 
> Wine: http://www.winehq.org
> pcc:  http://pcc.ludd.ltu.se
> 

Acked-by: H. Peter Anvin <hpa@zytor.com>

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

[PATCH] linux/types.h: Do not depend on __GNUC__ for __le64 and __be64 [synced to git]

[Detlef Riekenberg]

Hi.
<linux/cdrom.h> is used in Wine, but failed to compile with pcc.
(__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)

There is no reason to restrict the typedefs with __GNUC__.
As reference, the typedefs for __u64 and __s64 where fixed
to be available with pcc on 02. Mai 2008 by H. Peter Anvin
(commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)

Wine: http://www.winehq.org
pcc:  http://pcc.ludd.ltu.se

This patch is a resync to current git.

Please CC me personally. Thanks
-- 
By by ... Detlef




>From 83199a23ff35af6f10d16ebcb7ca283b3f5f05e3 Mon Sep 17 00:00:00 2001
From: Detlef Riekenberg <wine.dev@web.de>
Date: Thu, 1 Jan 2009 22:55:21 +0100
Subject: [PATCH] linux/types.h: Do not depend on __GNUC__ for
__le64/__be64


Acked-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Detlef Riekenberg <wine.dev@web.de>

---
 include/linux/types.h |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/include/linux/types.h b/include/linux/types.h
index 121f349..e2a2788 100644
--- a/include/linux/types.h
+++ b/include/linux/types.h
@@ -176,10 +176,9 @@ typedef __u16 __bitwise __le16;
 typedef __u16 __bitwise __be16;
 typedef __u32 __bitwise __le32;
 typedef __u32 __bitwise __be32;
-#if defined(__GNUC__)
 typedef __u64 __bitwise __le64;
 typedef __u64 __bitwise __be64;
-#endif
+
 typedef __u16 __bitwise __sum16;
 typedef __u32 __bitwise __wsum;
 
-- 
1.5.4.3



 

[PATCH] [TRIVIAL] linux/types.h: Don't depend on __GNUC__ for __le64 / __be64 [synced to git]

[Detlef Riekenberg]

Hi.
<linux/cdrom.h> is used in Wine, but failed to compile with pcc.
(__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)

There is no reason to restrict the typedefs to __GNUC__.
This Patch is trivial for gcc and icc, but required for other compiler.

As reference, the typedefs for __u64 and __s64 where fixed to be
available for other compiler on 02. Mai 2008 by H. Peter Anvin
(commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)

Wine: http://www.winehq.org
pcc:  http://pcc.ludd.ltu.se

This patch is again a resync to current git.
Is there anything, that I can do for this Patch?

Please CC me personally. Thanks

-- 
Bye bye ... Detlef



>From ce469182d86b94e4e2fd4c84028369a2eefdfb0c Mon Sep 17 00:00:00 2001
From: Detlef Riekenberg <wine.dev@web.de>
Date: Wed, 7 Jan 2009 10:11:44 +0100
Subject: [PATCH] linux/types.h: Don't depend on __GNUC__ for
__le64/__be64

Acked-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Detlef Riekenberg <wine.dev@web.de>
---
 include/linux/types.h |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/include/linux/types.h b/include/linux/types.h
index 3b864f2..712ca53 100644
--- a/include/linux/types.h
+++ b/include/linux/types.h
@@ -176,10 +176,9 @@ typedef __u16 __bitwise __le16;
 typedef __u16 __bitwise __be16;
 typedef __u32 __bitwise __le32;
 typedef __u32 __bitwise __be32;
-#if defined(__GNUC__)
 typedef __u64 __bitwise __le64;
 typedef __u64 __bitwise __be64;
-#endif
+
 typedef __u16 __bitwise __sum16;
 typedef __u32 __bitwise __wsum;
 
-- 
1.5.4.3

Re: [PATCH] [TRIVIAL] linux/types.h: Don't depend on __GNUC__ for __le64 / __be64 [synced to git]

[H. Peter Anvin]

Detlef Riekenberg wrote:
> Hi.
> <linux/cdrom.h> is used in Wine, but failed to compile with pcc.
> (__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)
> 
> There is no reason to restrict the typedefs to __GNUC__.
> This Patch is trivial for gcc and icc, but required for other compiler.
> 
> As reference, the typedefs for __u64 and __s64 where fixed to be
> available for other compiler on 02. Mai 2008 by H. Peter Anvin
> (commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)
> 
> Wine: http://www.winehq.org
> pcc:  http://pcc.ludd.ltu.se
> 
> This patch is again a resync to current git.
> Is there anything, that I can do for this Patch?
> 

For what it's worth...

Acked-by: H. Peter Anvin <hpa@zytor.com>

This isn't an x86 thing so it's a bit out of scope for me to push it 
through -tip, but it's a valid bugfix.

	-hpa

Re: [PATCH] [TRIVIAL] linux/types.h: Don't depend on __GNUC__ for __le64 / __be64 [synced to git]

[Sam Ravnborg]

On Wed, Jan 07, 2009 at 11:21:54AM -0800, H. Peter Anvin wrote:
> Detlef Riekenberg wrote:
> >Hi.
> ><linux/cdrom.h> is used in Wine, but failed to compile with pcc.
> >(__le64 and __be64 are referenced in <linux/byteorder/little_endian.h>)
> >
> >There is no reason to restrict the typedefs to __GNUC__.
> >This Patch is trivial for gcc and icc, but required for other compiler.
> >
> >As reference, the typedefs for __u64 and __s64 where fixed to be
> >available for other compiler on 02. Mai 2008 by H. Peter Anvin
> >(commit edfa5cfa3dc5bfa95e6aa82a2b8904e7f6c35ed7)
> >
> >Wine: http://www.winehq.org
> >pcc:  http://pcc.ludd.ltu.se
> >
> >This patch is again a resync to current git.
> >Is there anything, that I can do for this Patch?
> >
> 
> For what it's worth...
> 
> Acked-by: H. Peter Anvin <hpa@zytor.com>
> 
> This isn't an x86 thing so it's a bit out of scope for me to push it 
> through -tip, but it's a valid bugfix.
I have applied it to kbuild-fixes.git.

Thanks,
	Sam