* x86: merge some trivially mergeable headers
@ 2007-10-13 20:48 Roland Dreier
2007-10-13 21:12 ` Thomas Gleixner
0 siblings, 1 reply; 4+ messages in thread
From: Roland Dreier @ 2007-10-13 20:48 UTC (permalink / raw)
To: Andi Kleen, Andrew Morton, Thomas Gleixner; +Cc: linux-kernel
Merge errno.h, resource.h, rtc.h, sections.h, serial.h and sockios.h,
where i386 and x86_64 have no or only trivial comment/include guard
differences.
Build tested on both 32-bit and 64-bit, and booted on 64-bit.
Signed-off-by: Roland Dreier <roland@digitalvampire.org>
---
Not sure who's merging this type of stuff so I just picked a grab bag
of people to put on the To: line.
Also I'm not sure if someone else is already working on this, so I'm
just sending out about 15 minutes of work. If this is helpful I'll do
more of the easy ones...
include/asm-x86/errno.h | 19 ++++++++-----------
include/asm-x86/errno_32.h | 6 ------
include/asm-x86/errno_64.h | 6 ------
include/asm-x86/resource.h | 19 ++++++++-----------
include/asm-x86/resource_32.h | 6 ------
include/asm-x86/resource_64.h | 6 ------
include/asm-x86/rtc.h | 15 ++++++++++-----
include/asm-x86/rtc_32.h | 10 ----------
include/asm-x86/rtc_64.h | 10 ----------
include/asm-x86/sections.h | 9 +++++----
include/asm-x86/sections_32.h | 7 -------
include/asm-x86/sections_64.h | 7 -------
include/asm-x86/serial.h | 30 +++++++++++++++++++++++++++---
include/asm-x86/serial_32.h | 29 -----------------------------
include/asm-x86/serial_64.h | 29 -----------------------------
include/asm-x86/sockios.h | 26 +++++++++++++++-----------
include/asm-x86/sockios_32.h | 13 -------------
include/asm-x86/sockios_64.h | 13 -------------
18 files changed, 73 insertions(+), 187 deletions(-)
delete mode 100644 include/asm-x86/errno_32.h
delete mode 100644 include/asm-x86/errno_64.h
delete mode 100644 include/asm-x86/resource_32.h
delete mode 100644 include/asm-x86/resource_64.h
delete mode 100644 include/asm-x86/rtc_32.h
delete mode 100644 include/asm-x86/rtc_64.h
delete mode 100644 include/asm-x86/sections_32.h
delete mode 100644 include/asm-x86/sections_64.h
delete mode 100644 include/asm-x86/serial_32.h
delete mode 100644 include/asm-x86/serial_64.h
delete mode 100644 include/asm-x86/sockios_32.h
delete mode 100644 include/asm-x86/sockios_64.h
diff --git a/include/asm-x86/errno.h b/include/asm-x86/errno.h
index 9d511be..e06fe53 100644
--- a/include/asm-x86/errno.h
+++ b/include/asm-x86/errno.h
@@ -1,13 +1,10 @@
#ifdef __KERNEL__
-# ifdef CONFIG_X86_32
-# include "errno_32.h"
-# else
-# include "errno_64.h"
-# endif
-#else
-# ifdef __i386__
-# include "errno_32.h"
-# else
-# include "errno_64.h"
-# endif
+
+#ifndef _ASM_X86_ERRNO_H
+#define _ASM_X86_ERRNO_H
+
+#include <asm-generic/errno.h>
+
+#endif /* _ASM_X86_ERRNO_H */
+
#endif
diff --git a/include/asm-x86/errno_32.h b/include/asm-x86/errno_32.h
deleted file mode 100644
index 969b343..0000000
--- a/include/asm-x86/errno_32.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _I386_ERRNO_H
-#define _I386_ERRNO_H
-
-#include <asm-generic/errno.h>
-
-#endif
diff --git a/include/asm-x86/errno_64.h b/include/asm-x86/errno_64.h
deleted file mode 100644
index 3111821..0000000
--- a/include/asm-x86/errno_64.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _X8664_ERRNO_H
-#define _X8664_ERRNO_H
-
-#include <asm-generic/errno.h>
-
-#endif
diff --git a/include/asm-x86/resource.h b/include/asm-x86/resource.h
index 732410a..df20199 100644
--- a/include/asm-x86/resource.h
+++ b/include/asm-x86/resource.h
@@ -1,13 +1,10 @@
#ifdef __KERNEL__
-# ifdef CONFIG_X86_32
-# include "resource_32.h"
-# else
-# include "resource_64.h"
-# endif
-#else
-# ifdef __i386__
-# include "resource_32.h"
-# else
-# include "resource_64.h"
-# endif
+
+#ifndef _ASM_X86_RESOURCE_H
+#define _ASM_X86_RESOURCE_H
+
+#include <asm-generic/resource.h>
+
+#endif /* _ASM_X86_RESOURCE_H */
+
#endif
diff --git a/include/asm-x86/resource_32.h b/include/asm-x86/resource_32.h
deleted file mode 100644
index 6c1ea37..0000000
--- a/include/asm-x86/resource_32.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _I386_RESOURCE_H
-#define _I386_RESOURCE_H
-
-#include <asm-generic/resource.h>
-
-#endif
diff --git a/include/asm-x86/resource_64.h b/include/asm-x86/resource_64.h
deleted file mode 100644
index f40b406..0000000
--- a/include/asm-x86/resource_64.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _X8664_RESOURCE_H
-#define _X8664_RESOURCE_H
-
-#include <asm-generic/resource.h>
-
-#endif
diff --git a/include/asm-x86/rtc.h b/include/asm-x86/rtc.h
index 1f0c98e..70838ea 100644
--- a/include/asm-x86/rtc.h
+++ b/include/asm-x86/rtc.h
@@ -1,5 +1,10 @@
-#ifdef CONFIG_X86_32
-# include "rtc_32.h"
-#else
-# include "rtc_64.h"
-#endif
+#ifndef _ASM_X86_RTC_H
+#define _ASM_X86_RTC_H
+
+/*
+ * x86 uses the default access methods for the RTC.
+ */
+
+#include <asm-generic/rtc.h>
+
+#endif /* _ASM_X86_RTC_H */
diff --git a/include/asm-x86/rtc_32.h b/include/asm-x86/rtc_32.h
deleted file mode 100644
index ffd0210..0000000
--- a/include/asm-x86/rtc_32.h
+++ /dev/null
@@ -1,10 +0,0 @@
-#ifndef _I386_RTC_H
-#define _I386_RTC_H
-
-/*
- * x86 uses the default access methods for the RTC.
- */
-
-#include <asm-generic/rtc.h>
-
-#endif
diff --git a/include/asm-x86/rtc_64.h b/include/asm-x86/rtc_64.h
deleted file mode 100644
index 18ed713..0000000
--- a/include/asm-x86/rtc_64.h
+++ /dev/null
@@ -1,10 +0,0 @@
-#ifndef _X86_64_RTC_H
-#define _X86_64_RTC_H
-
-/*
- * x86 uses the default access methods for the RTC.
- */
-
-#include <asm-generic/rtc.h>
-
-#endif
diff --git a/include/asm-x86/sections.h b/include/asm-x86/sections.h
index ae6c69d..bf0f2b2 100644
--- a/include/asm-x86/sections.h
+++ b/include/asm-x86/sections.h
@@ -1,5 +1,6 @@
-#ifdef CONFIG_X86_32
-# include "sections_32.h"
-#else
-# include "sections_64.h"
+#ifndef _ASM_X86_SECTIONS_H
+#define _ASM_X86_SECTIONS_H
+
+#include <asm-generic/sections.h>
+
#endif
diff --git a/include/asm-x86/sections_32.h b/include/asm-x86/sections_32.h
deleted file mode 100644
index 2dcbb92..0000000
--- a/include/asm-x86/sections_32.h
+++ /dev/null
@@ -1,7 +0,0 @@
-#ifndef _I386_SECTIONS_H
-#define _I386_SECTIONS_H
-
-/* nothing to see, move along */
-#include <asm-generic/sections.h>
-
-#endif
diff --git a/include/asm-x86/sections_64.h b/include/asm-x86/sections_64.h
deleted file mode 100644
index c746d9f..0000000
--- a/include/asm-x86/sections_64.h
+++ /dev/null
@@ -1,7 +0,0 @@
-#ifndef _X8664_SECTIONS_H
-#define _X8664_SECTIONS_H
-
-/* nothing to see, move along */
-#include <asm-generic/sections.h>
-
-#endif
diff --git a/include/asm-x86/serial.h b/include/asm-x86/serial.h
index cf1b052..628c801 100644
--- a/include/asm-x86/serial.h
+++ b/include/asm-x86/serial.h
@@ -1,5 +1,29 @@
-#ifdef CONFIG_X86_32
-# include "serial_32.h"
+#ifndef _ASM_X86_SERIAL_H
+#define _ASM_X86_SERIAL_H
+
+/*
+ * This assumes you have a 1.8432 MHz clock for your UART.
+ *
+ * It'd be nice if someone built a serial card with a 24.576 MHz
+ * clock, since the 16550A is capable of handling a top speed of 1.5
+ * megabits/second; but this requires the faster clock.
+ */
+#define BASE_BAUD ( 1843200 / 16 )
+
+/* Standard COM flags (except for COM4, because of the 8514 problem) */
+#ifdef CONFIG_SERIAL_DETECT_IRQ
+#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ)
+#define STD_COM4_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_AUTO_IRQ)
#else
-# include "serial_64.h"
+#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST)
+#define STD_COM4_FLAGS ASYNC_BOOT_AUTOCONF
#endif
+
+#define SERIAL_PORT_DFNS \
+ /* UART CLK PORT IRQ FLAGS */ \
+ { 0, BASE_BAUD, 0x3F8, 4, STD_COM_FLAGS }, /* ttyS0 */ \
+ { 0, BASE_BAUD, 0x2F8, 3, STD_COM_FLAGS }, /* ttyS1 */ \
+ { 0, BASE_BAUD, 0x3E8, 4, STD_COM_FLAGS }, /* ttyS2 */ \
+ { 0, BASE_BAUD, 0x2E8, 3, STD_COM4_FLAGS }, /* ttyS3 */
+
+#endif /* _ASM_X86_SERIAL_H */
diff --git a/include/asm-x86/serial_32.h b/include/asm-x86/serial_32.h
deleted file mode 100644
index bd67480..0000000
--- a/include/asm-x86/serial_32.h
+++ /dev/null
@@ -1,29 +0,0 @@
-/*
- * include/asm-i386/serial.h
- */
-
-
-/*
- * This assumes you have a 1.8432 MHz clock for your UART.
- *
- * It'd be nice if someone built a serial card with a 24.576 MHz
- * clock, since the 16550A is capable of handling a top speed of 1.5
- * megabits/second; but this requires the faster clock.
- */
-#define BASE_BAUD ( 1843200 / 16 )
-
-/* Standard COM flags (except for COM4, because of the 8514 problem) */
-#ifdef CONFIG_SERIAL_DETECT_IRQ
-#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ)
-#define STD_COM4_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_AUTO_IRQ)
-#else
-#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST)
-#define STD_COM4_FLAGS ASYNC_BOOT_AUTOCONF
-#endif
-
-#define SERIAL_PORT_DFNS \
- /* UART CLK PORT IRQ FLAGS */ \
- { 0, BASE_BAUD, 0x3F8, 4, STD_COM_FLAGS }, /* ttyS0 */ \
- { 0, BASE_BAUD, 0x2F8, 3, STD_COM_FLAGS }, /* ttyS1 */ \
- { 0, BASE_BAUD, 0x3E8, 4, STD_COM_FLAGS }, /* ttyS2 */ \
- { 0, BASE_BAUD, 0x2E8, 3, STD_COM4_FLAGS }, /* ttyS3 */
diff --git a/include/asm-x86/serial_64.h b/include/asm-x86/serial_64.h
deleted file mode 100644
index b0496e0..0000000
--- a/include/asm-x86/serial_64.h
+++ /dev/null
@@ -1,29 +0,0 @@
-/*
- * include/asm-x86_64/serial.h
- */
-
-
-/*
- * This assumes you have a 1.8432 MHz clock for your UART.
- *
- * It'd be nice if someone built a serial card with a 24.576 MHz
- * clock, since the 16550A is capable of handling a top speed of 1.5
- * megabits/second; but this requires the faster clock.
- */
-#define BASE_BAUD ( 1843200 / 16 )
-
-/* Standard COM flags (except for COM4, because of the 8514 problem) */
-#ifdef CONFIG_SERIAL_DETECT_IRQ
-#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ)
-#define STD_COM4_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_AUTO_IRQ)
-#else
-#define STD_COM_FLAGS (ASYNC_BOOT_AUTOCONF | ASYNC_SKIP_TEST)
-#define STD_COM4_FLAGS ASYNC_BOOT_AUTOCONF
-#endif
-
-#define SERIAL_PORT_DFNS \
- /* UART CLK PORT IRQ FLAGS */ \
- { 0, BASE_BAUD, 0x3F8, 4, STD_COM_FLAGS }, /* ttyS0 */ \
- { 0, BASE_BAUD, 0x2F8, 3, STD_COM_FLAGS }, /* ttyS1 */ \
- { 0, BASE_BAUD, 0x3E8, 4, STD_COM_FLAGS }, /* ttyS2 */ \
- { 0, BASE_BAUD, 0x2E8, 3, STD_COM4_FLAGS }, /* ttyS3 */
diff --git a/include/asm-x86/sockios.h b/include/asm-x86/sockios.h
index 5a134fc..f3b46d0 100644
--- a/include/asm-x86/sockios.h
+++ b/include/asm-x86/sockios.h
@@ -1,13 +1,17 @@
#ifdef __KERNEL__
-# ifdef CONFIG_X86_32
-# include "sockios_32.h"
-# else
-# include "sockios_64.h"
-# endif
-#else
-# ifdef __i386__
-# include "sockios_32.h"
-# else
-# include "sockios_64.h"
-# endif
+
+#ifndef _ASM_X86_SOCKIOS_H
+#define _ASM_X86_SOCKIOS_H
+
+/* Socket-level I/O control calls. */
+#define FIOSETOWN 0x8901
+#define SIOCSPGRP 0x8902
+#define FIOGETOWN 0x8903
+#define SIOCGPGRP 0x8904
+#define SIOCATMARK 0x8905
+#define SIOCGSTAMP 0x8906 /* Get stamp (timeval) */
+#define SIOCGSTAMPNS 0x8907 /* Get stamp (timespec) */
+
+#endif /* _ASM_X86_SOCKIOS_H */
+
#endif
diff --git a/include/asm-x86/sockios_32.h b/include/asm-x86/sockios_32.h
deleted file mode 100644
index ff528c7..0000000
--- a/include/asm-x86/sockios_32.h
+++ /dev/null
@@ -1,13 +0,0 @@
-#ifndef __ARCH_I386_SOCKIOS__
-#define __ARCH_I386_SOCKIOS__
-
-/* Socket-level I/O control calls. */
-#define FIOSETOWN 0x8901
-#define SIOCSPGRP 0x8902
-#define FIOGETOWN 0x8903
-#define SIOCGPGRP 0x8904
-#define SIOCATMARK 0x8905
-#define SIOCGSTAMP 0x8906 /* Get stamp (timeval) */
-#define SIOCGSTAMPNS 0x8907 /* Get stamp (timespec) */
-
-#endif
diff --git a/include/asm-x86/sockios_64.h b/include/asm-x86/sockios_64.h
deleted file mode 100644
index d726ba2..0000000
--- a/include/asm-x86/sockios_64.h
+++ /dev/null
@@ -1,13 +0,0 @@
-#ifndef __ARCH_X8664_SOCKIOS__
-#define __ARCH_X8664_SOCKIOS__
-
-/* Socket-level I/O control calls. */
-#define FIOSETOWN 0x8901
-#define SIOCSPGRP 0x8902
-#define FIOGETOWN 0x8903
-#define SIOCGPGRP 0x8904
-#define SIOCATMARK 0x8905
-#define SIOCGSTAMP 0x8906 /* Get stamp (timeval) */
-#define SIOCGSTAMPNS 0x8907 /* Get stamp (timespec) */
-
-#endif
--
1.5.3.2
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: x86: merge some trivially mergeable headers
2007-10-13 20:48 x86: merge some trivially mergeable headers Roland Dreier
@ 2007-10-13 21:12 ` Thomas Gleixner
2007-10-14 3:12 ` Roland Dreier
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Gleixner @ 2007-10-13 21:12 UTC (permalink / raw)
To: Roland Dreier; +Cc: Andi Kleen, Andrew Morton, linux-kernel
On Sat, 13 Oct 2007, Roland Dreier wrote:
> Merge errno.h, resource.h, rtc.h, sections.h, serial.h and sockios.h,
> where i386 and x86_64 have no or only trivial comment/include guard
> differences.
>
> Build tested on both 32-bit and 64-bit, and booted on 64-bit.
>
> Signed-off-by: Roland Dreier <roland@digitalvampire.org>
> ---
> Not sure who's merging this type of stuff so I just picked a grab bag
> of people to put on the To: line.
>
> Also I'm not sure if someone else is already working on this, so I'm
> just sending out about 15 minutes of work. If this is helpful I'll do
> more of the easy ones...
Roland, thanks for providing this. I have some of those already, but I
really appreciate the help. I take yours and the previously posted and
push them out into a cleanup branch on my x86 git tree.
I guess it will be unavoidable that there will be some overlap on the low
hanging fruit merges, but anyone who is tackling a more complex one should
make this public upfront so we can avoid the redundant work.
Thanks,
tglx
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: x86: merge some trivially mergeable headers
2007-10-13 21:12 ` Thomas Gleixner
@ 2007-10-14 3:12 ` Roland Dreier
2007-10-15 12:54 ` Thomas Gleixner
0 siblings, 1 reply; 4+ messages in thread
From: Roland Dreier @ 2007-10-14 3:12 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: Andi Kleen, Andrew Morton, linux-kernel
> Roland, thanks for providing this. I have some of those already, but I
> really appreciate the help. I take yours and the previously posted and
> push them out into a cleanup branch on my x86 git tree.
No problem, this merging work is the perfect thing when I need a
break-- mindless enough to be relaxing, but useful enough that I don't
feel guilty...
Anyway, I assume you meant to write "I [will] take yours", since I
don't see anything at all in tglx/linux-2.6-x86.git right now. I'll
check again on Monday and base my work on your tree.
- R.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: x86: merge some trivially mergeable headers
2007-10-14 3:12 ` Roland Dreier
@ 2007-10-15 12:54 ` Thomas Gleixner
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Gleixner @ 2007-10-15 12:54 UTC (permalink / raw)
To: Roland Dreier; +Cc: Andi Kleen, Andrew Morton, LKML, Brian Gerst
Roland,
On Sat, 13 Oct 2007, Roland Dreier wrote:
> > Roland, thanks for providing this. I have some of those already, but I
> > really appreciate the help. I take yours and the previously posted and
> > push them out into a cleanup branch on my x86 git tree.
>
> No problem, this merging work is the perfect thing when I need a
> break-- mindless enough to be relaxing, but useful enough that I don't
> feel guilty...
>
> Anyway, I assume you meant to write "I [will] take yours", since I
> don't see anything at all in tglx/linux-2.6-x86.git right now. I'll
> check again on Monday and base my work on your tree.
I meant the patch, which was posted by Brian earlier, which had some
overlap with yours.
I applied both and fixed the Kbuild entries as well. On top of those I
run my "find-duplicates-harder" script and fixed another bunch.
You can find the current state of affairs in the "cleanup" branch of
my x86 git tree.
Thanks,
tglx
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-10-15 12:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-10-13 20:48 x86: merge some trivially mergeable headers Roland Dreier
2007-10-13 21:12 ` Thomas Gleixner
2007-10-14 3:12 ` Roland Dreier
2007-10-15 12:54 ` Thomas Gleixner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®