From: Jose Luis Domingo Lopez <linux-kernel@24x7linux.com>
To: linux-kernel@vger.kernel.org
Cc: Nick.Holloway@pyrites.org.uk
Subject: [PATCH 2.6] Add missing MODULE_PARAM to dummy.c (and MAINTAINERShip)
Date: Thu, 8 Apr 2004 19:48:23 +0200 [thread overview]
Message-ID: <20040408174823.GA13335@localhost> (raw)
Hello:
It seems the "dummy" network interface driver is missing some MODULE_*
macros, needed with kernel 2.6.x and module-init-tools to show
information about the module (parameters, author, description, etc).
A patch follows to (hopefully) correct this. Another patch includes an
entry in the MAINTAINERS file for the "dummy" module. However, I suppose
the module author (Nick Holloway) will come up and say if he should
still be considered as the module maintainer, so to add the correct
information to the MAINTAINERS file.
Hope the patchs are correct, or at least useful to note the missing bits
I was trying to "patch" and someone else does the "technical" work :-)
Greetings.
--
Jose Luis Domingo Lopez
Linux Registered User #189436 Debian Linux Sid (Linux 2.6.5)
diff -Nrup linux-2.6.5/drivers/net/dummy.c linux-2.6.5-new/drivers/net/dummy.c
--- linux-2.6.5/drivers/net/dummy.c 2004-04-04 17:45:54.000000000 +0200
+++ linux-2.6.5-new/drivers/net/dummy.c 2004-04-08 19:23:23.000000000 +0200
@@ -89,7 +89,8 @@ static struct net_device_stats *dummy_ge
static struct net_device **dummies;
/* Number of dummy devices to be set up by this module. */
-module_param(numdummies, int, 0);
+MODULE_PARM(numdummies, "i");
+MODULE_PARM_DESC(numdummies, "Maximum number of dummy devices (defaults to one)");
static int __init dummy_init_one(int index)
{
@@ -144,3 +145,5 @@ static void __exit dummy_cleanup_module(
module_init(dummy_init_module);
module_exit(dummy_cleanup_module);
MODULE_LICENSE("GPL");
+MODULE_DESCRIPTION("Dummy network interface driver");
+MODULE_AUTHOR("Nick Holloway <Nick.Holloway@pyrites.org.uk>");
diff -Nrup linux-2.6.5/MAINTAINERS linux-2.6.5-new/MAINTAINERS
--- linux-2.6.5/MAINTAINERS 2004-04-04 17:49:26.000000000 +0200
+++ linux-2.6.5-new/MAINTAINERS 2004-04-08 19:22:01.000000000 +0200
@@ -707,6 +707,12 @@ M: romieu@cogenit.fr
M: romieu@ensta.fr
S: Maintained
+DUMMY NETWORK INTERFACE DRIVER
+P: Nick Holloway
+M: Nick.Holloway@pyrites.org.uk
+L: linux-kernel@vger.kernel.org
+S: Supported
+
DVB SUBSYSTEM AND DRIVERS
P: LinuxTV.org Project
M: linux-dvb-maintainer@linuxtv.org
next reply other threads:[~2004-04-08 17:48 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-04-08 17:48 Jose Luis Domingo Lopez [this message]
2004-04-08 17:54 ` [PATCH 2.6] Add missing MODULE_PARAM to dummy.c (and MAINTAINERShip) Chris Wright
2004-04-08 18:59 ` Jose Luis Domingo Lopez
2004-04-08 19:29 ` Chris Wright
2004-04-08 19:10 ` Nick Holloway
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20040408174823.GA13335@localhost \
--to=linux-kernel@24x7linux.com \
--cc=Nick.Holloway@pyrites.org.uk \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox