* [2.6 PATCH] Altix - console driver calls console_initcall
@ 2004-03-08 20:32 Pat Gefre
2004-03-08 21:29 ` Bjorn Helgaas
2004-03-08 23:20 ` Pat Gefre
0 siblings, 2 replies; 3+ messages in thread
From: Pat Gefre @ 2004-03-08 20:32 UTC (permalink / raw)
To: davidm; +Cc: linux-ia64, linux-kernel
Small mod to have the console driver call console_initcall() to
register.
# 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.1692 -> 1.1693
# drivers/char/sn_serial.c 1.5 -> 1.6
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 04/03/08 pfg@sgi.com 1.1693
# drivers/char/sn_serial.c
# Make console_initcall() for proper registration.
# --------------------------------------------
#
diff -Nru a/drivers/char/sn_serial.c b/drivers/char/sn_serial.c
--- a/drivers/char/sn_serial.c Mon Mar 8 14:28:53 2004
+++ b/drivers/char/sn_serial.c Mon Mar 8 14:28:53 2004
@@ -82,7 +82,6 @@
static unsigned long sn_interrupt_timeout;
extern u64 master_node_bedrock_address;
-
static int sn_debug_printf(const char *fmt, ...);
#undef DEBUG
@@ -105,7 +104,7 @@
static struct sn_sal_ops *sn_func;
/* Prototypes */
-static void __init sn_sal_serial_console_init(void);
+int __init sn_sal_serial_console_init(void);
static int snt_hw_puts(const char *, int);
static int snt_poll_getc(void);
static int snt_poll_input_pending(void);
@@ -921,9 +920,6 @@
printk(KERN_ERR "sn_serial: Unable to register tty driver\n");
return retval;
}
-#ifdef CONFIG_SGI_L1_SERIAL_CONSOLE
- sn_sal_serial_console_init();
-#endif /* CONFIG_SGI_L1_SERIAL_CONSOLE */
return 0;
}
@@ -993,7 +989,7 @@
.index = -1
};
-static void __init
+int __init
sn_sal_serial_console_init(void)
{
if (ia64_platform_is("sn2")) {
@@ -1001,6 +997,8 @@
sn_debug_printf("sn_sal_serial_console_init : register console\n");
register_console(&sal_console);
}
+ return 0;
}
+console_initcall(sn_sal_serial_console_init);
#endif /* CONFIG_SGI_L1_SERIAL_CONSOLE */
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [2.6 PATCH] Altix - console driver calls console_initcall
2004-03-08 20:32 [2.6 PATCH] Altix - console driver calls console_initcall Pat Gefre
@ 2004-03-08 21:29 ` Bjorn Helgaas
2004-03-08 23:20 ` Pat Gefre
1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Helgaas @ 2004-03-08 21:29 UTC (permalink / raw)
To: Pat Gefre, davidm; +Cc: linux-ia64, linux-kernel
On Monday 08 March 2004 1:32 pm, Pat Gefre wrote:
> -static void __init sn_sal_serial_console_init(void);
> +int __init sn_sal_serial_console_init(void);
I don't see any callers other than the console_initcall(), so
you could lose the declaration altogether.
> -static void __init
> +int __init
> sn_sal_serial_console_init(void)
console_initcall() works fine with static functions, so you should
be able to keep this static.
Bjorn
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [2.6 PATCH] Altix - console driver calls console_initcall
2004-03-08 20:32 [2.6 PATCH] Altix - console driver calls console_initcall Pat Gefre
2004-03-08 21:29 ` Bjorn Helgaas
@ 2004-03-08 23:20 ` Pat Gefre
1 sibling, 0 replies; 3+ messages in thread
From: Pat Gefre @ 2004-03-08 23:20 UTC (permalink / raw)
To: bjorn.helgaas, davidm; +Cc: linux-ia64, linux-kernel
> On Monday 08 March 2004 1:32 pm, Pat Gefre wrote:
> > -static void __init sn_sal_serial_console_init(void);
> > +int __init sn_sal_serial_console_init(void);
>
> I don't see any callers other than the console_initcall(), so
> you could lose the declaration altogether.
>
> > -static void __init
> > +int __init
> > sn_sal_serial_console_init(void)
>
> console_initcall() works fine with static functions, so you should
> be able to keep this static.
>
> Bjorn
You're right.....
Here's a new mod which goes against 1.1698 too (the 'staircase' mod) -
so it'll have the correct line numbers - in other words it'll apply
cleanly if 1.1698 has already been applied.
# 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.1693 -> 1.1694
# drivers/char/sn_serial.c 1.6 -> 1.7
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 04/03/08 pfg@sgi.com 1.1694
# drivers/char/sn_serial.c
# Make console_initcall() for proper registration.
# --------------------------------------------
#
diff -Nru a/drivers/char/sn_serial.c b/drivers/char/sn_serial.c
--- a/drivers/char/sn_serial.c Mon Mar 8 17:15:51 2004
+++ b/drivers/char/sn_serial.c Mon Mar 8 17:15:51 2004
@@ -105,7 +105,6 @@
static struct sn_sal_ops *sn_func;
/* Prototypes */
-static void __init sn_sal_serial_console_init(void);
static int snt_hw_puts(const char *, int);
static int snt_poll_getc(void);
static int snt_poll_input_pending(void);
@@ -921,9 +920,6 @@
printk(KERN_ERR "sn_serial: Unable to register tty driver\n");
return retval;
}
-#ifdef CONFIG_SGI_L1_SERIAL_CONSOLE
- sn_sal_serial_console_init();
-#endif /* CONFIG_SGI_L1_SERIAL_CONSOLE */
return 0;
}
@@ -1015,7 +1011,7 @@
.index = -1
};
-static void __init
+static int __init
sn_sal_serial_console_init(void)
{
if (ia64_platform_is("sn2")) {
@@ -1023,6 +1019,8 @@
sn_debug_printf("sn_sal_serial_console_init : register console\n");
register_console(&sal_console);
}
+ return 0;
}
+console_initcall(sn_sal_serial_console_init);
#endif /* CONFIG_SGI_L1_SERIAL_CONSOLE */
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-03-08 23:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-03-08 20:32 [2.6 PATCH] Altix - console driver calls console_initcall Pat Gefre
2004-03-08 21:29 ` Bjorn Helgaas
2004-03-08 23:20 ` Pat Gefre
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox