* [PATCH] move MODULE_ALIAS_LDISC definition to include/linux/termio.h
@ 2003-09-06 14:24 Matthias Urlichs
2003-09-06 14:51 ` Russell King
0 siblings, 1 reply; 2+ messages in thread
From: Matthias Urlichs @ 2003-09-06 14:24 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, torvalds
Please don't add non-machine-specific macros to machine-specific include files.
Thank you.
# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
# ChangeSet 1.1105 -> 1.1106
# include/asm-i386/termios.h 1.4 -> 1.5
# include/linux/termios.h 1.1 -> 1.2
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 03/09/06 smurf@smurf.noris.de 1.1106
# Move MODULE_ALIAS_LDISC declaration to include/linux/termios.h
# --------------------------------------------
#
diff -Nru a/include/asm-i386/termios.h b/include/asm-i386/termios.h
--- a/include/asm-i386/termios.h Sat Sep 6 16:19:44 2003
+++ b/include/asm-i386/termios.h Sat Sep 6 16:19:44 2003
@@ -58,7 +58,6 @@
#define N_HCI 15 /* Bluetooth HCI UART */
#ifdef __KERNEL__
-#include <linux/module.h>
/* intr=^C quit=^\ erase=del kill=^U
eof=^D vtime=\0 vmin=\1 sxtc=\0
@@ -102,8 +101,6 @@
#define user_termios_to_kernel_termios(k, u) copy_from_user(k, u, sizeof(struct termios))
#define kernel_termios_to_user_termios(u, k) copy_to_user(u, k, sizeof(struct termios))
-#define MODULE_ALIAS_LDISC(ldisc) \
- MODULE_ALIAS("tty-ldisc-" __stringify(ldisc))
#endif /* __KERNEL__ */
#endif /* _I386_TERMIOS_H */
diff -Nru a/include/linux/termios.h b/include/linux/termios.h
--- a/include/linux/termios.h Sat Sep 6 16:19:44 2003
+++ b/include/linux/termios.h Sat Sep 6 16:19:44 2003
@@ -4,4 +4,11 @@
#include <linux/types.h>
#include <asm/termios.h>
+#ifdef __KERNEL__
+#include <linux/module.h>
+
+#define MODULE_ALIAS_LDISC(ldisc) \
+ MODULE_ALIAS("tty-ldisc-" __stringify(ldisc))
+#endif /* __KERNEL__ */
+
#endif
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] move MODULE_ALIAS_LDISC definition to include/linux/termio.h
2003-09-06 14:24 [PATCH] move MODULE_ALIAS_LDISC definition to include/linux/termio.h Matthias Urlichs
@ 2003-09-06 14:51 ` Russell King
0 siblings, 0 replies; 2+ messages in thread
From: Russell King @ 2003-09-06 14:51 UTC (permalink / raw)
To: Matthias Urlichs; +Cc: linux-kernel, akpm, torvalds
On Sat, Sep 06, 2003 at 04:24:11PM +0200, Matthias Urlichs wrote:
> Please don't add non-machine-specific macros to machine-specific
> include files.
A more complete fix has already gone in for this.
--
Russell King (rmk@arm.linux.org.uk) http://www.arm.linux.org.uk/personal/
Linux kernel maintainer of:
2.6 ARM Linux - http://www.arm.linux.org.uk/
2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/
2.6 Serial core
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-09-06 14:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-09-06 14:24 [PATCH] move MODULE_ALIAS_LDISC definition to include/linux/termio.h Matthias Urlichs
2003-09-06 14:51 ` Russell King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox