Skip to content

Commit

Permalink
hvc_console: use "*_console" nomenclature to avoid modpost warning.
Browse files Browse the repository at this point in the history
The use of "hvc_con_driver" as the name for a file-static "struct
console" with a ".setup" field pointing to an __init function causes
a modpost warning, since a non-initdata structure points to init code.
Using "hvc_console" as the name triggers the hacky "*_console"
workaround in modpost to silence the warning, and is the same thing
that most of the other console drivers already do.

I made the same change in hvsi.c since I happened to notice it was
likely to suffer from the same problem.

Signed-off-by: Chris Metcalf <cmetcalf@tilera.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
  • Loading branch information
Chris Metcalf authored and Benjamin Herrenschmidt committed Jul 9, 2010
1 parent 32d8ad4 commit 4455b11
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
12 changes: 6 additions & 6 deletions drivers/char/hvc_console.c
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ static int __init hvc_console_setup(struct console *co, char *options)
return 0;
}

static struct console hvc_con_driver = {
static struct console hvc_console = {
.name = "hvc",
.write = hvc_console_print,
.device = hvc_console_device,
Expand All @@ -220,7 +220,7 @@ static struct console hvc_con_driver = {
*/
static int __init hvc_console_init(void)
{
register_console(&hvc_con_driver);
register_console(&hvc_console);
return 0;
}
console_initcall(hvc_console_init);
Expand Down Expand Up @@ -276,8 +276,8 @@ int hvc_instantiate(uint32_t vtermno, int index, const struct hv_ops *ops)
* now (setup won't fail at this point). It's ok to just
* call register again if previously .setup failed.
*/
if (index == hvc_con_driver.index)
register_console(&hvc_con_driver);
if (index == hvc_console.index)
register_console(&hvc_console);

return 0;
}
Expand Down Expand Up @@ -641,7 +641,7 @@ int hvc_poll(struct hvc_struct *hp)
}
for (i = 0; i < n; ++i) {
#ifdef CONFIG_MAGIC_SYSRQ
if (hp->index == hvc_con_driver.index) {
if (hp->index == hvc_console.index) {
/* Handle the SysRq Hack */
/* XXX should support a sequence */
if (buf[i] == '\x0f') { /* ^O */
Expand Down Expand Up @@ -909,7 +909,7 @@ static void __exit hvc_exit(void)
tty_unregister_driver(hvc_driver);
/* return tty_struct instances allocated in hvc_init(). */
put_tty_driver(hvc_driver);
unregister_console(&hvc_con_driver);
unregister_console(&hvc_console);
}
}
module_exit(hvc_exit);
4 changes: 2 additions & 2 deletions drivers/char/hvsi.c
Original file line number Diff line number Diff line change
Expand Up @@ -1255,7 +1255,7 @@ static int __init hvsi_console_setup(struct console *console, char *options)
return 0;
}

static struct console hvsi_con_driver = {
static struct console hvsi_console = {
.name = "hvsi",
.write = hvsi_console_print,
.device = hvsi_console_device,
Expand Down Expand Up @@ -1308,7 +1308,7 @@ static int __init hvsi_console_init(void)
}

if (hvsi_count)
register_console(&hvsi_con_driver);
register_console(&hvsi_console);
return 0;
}
console_initcall(hvsi_console_init);

0 comments on commit 4455b11

Please sign in to comment.