From: "YOSHIFUJI Hideaki / 吉藤英明" <yoshfuji@linux-ipv6.org>
To: Philip.Blundell@pobox.com, tim@cyberelk.net, campbell@torque.net,
andrea@e-mind.com
Cc: linux-parport@torque.net, linux-kernel@vger.kernel.org,
yoshfuji@linux-ipv6.org
Subject: Re: [PATCH] PARPORT: C99 Initializers
Date: Sun, 01 Feb 2004 22:56:19 +0900 (JST) [thread overview]
Message-ID: <20040201.225619.67854403.yoshfuji@linux-ipv6.org> (raw)
In-Reply-To: <20040201.224431.17604798.yoshfuji@linux-ipv6.org>
In article <20040201.224431.17604798.yoshfuji@linux-ipv6.org> (at Sun, 01 Feb 2004 22:44:31 +0900 (JST)), YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@linux-ipv6.org> says:
> Hello.
>
> D: convert drivers/parport/procfs.c to C99 initializers.
Oops, it contains several typos... Sorry for the mess.
Please use this instead.
===== drivers/parport/procfs.c 1.2 vs edited =====
--- 1.2/drivers/parport/procfs.c Tue Feb 5 16:37:25 2002
+++ edited/drivers/parport/procfs.c Sun Feb 1 22:54:02 2004
@@ -232,12 +232,29 @@
return copy_to_user(result, buffer, len) ? -EFAULT : 0;
}
-#define PARPORT_PORT_DIR(child) { 0, NULL, NULL, 0, 0555, child }
-#define PARPORT_PARPORT_DIR(child) { DEV_PARPORT, "parport", \
- NULL, 0, 0555, child }
-#define PARPORT_DEV_DIR(child) { CTL_DEV, "dev", NULL, 0, 0555, child }
-#define PARPORT_DEVICES_ROOT_DIR { DEV_PARPORT_DEVICES, "devices", \
- NULL, 0, 0555, NULL }
+#define PARPORT_PORT_DIR(child) { \
+ .mode = 0555, \
+ .child = child, \
+}
+
+#define PARPORT_PARPORT_DIR(child) { \
+ .ctl_name = DEV_PARPORT, \
+ .procname = "parport", \
+ .mode = 0555, \
+ .child = child, \
+}
+
+#define PARPORT_DEV_DIR(child) { \
+ .ctl_name = CTL_DEV, \
+ .procname = "dev", \
+ .mode = 0555, \
+ .child = child, \
+}
+
+#define PARPORT_DEVICES_ROOT_DIR { \
+ .ctl_name = DEV_PARPORT_DEVICES, \
+ .procname = "devices", \
+}
static const unsigned long parport_min_timeslice_value =
PARPORT_MIN_TIMESLICE_VALUE;
@@ -264,48 +281,105 @@
static const struct parport_sysctl_table parport_sysctl_template = {
NULL,
{
- { DEV_PARPORT_SPINTIME, "spintime",
- NULL, sizeof(int), 0644, NULL,
- &proc_dointvec_minmax, NULL, NULL,
- (void*) &parport_min_spintime_value,
- (void*) &parport_max_spintime_value },
- { DEV_PARPORT_BASE_ADDR, "base-addr",
- NULL, 0, 0444, NULL,
- &do_hardware_base_addr },
- { DEV_PARPORT_IRQ, "irq",
- NULL, 0, 0444, NULL,
- &do_hardware_irq },
- { DEV_PARPORT_DMA, "dma",
- NULL, 0, 0444, NULL,
- &do_hardware_dma },
- { DEV_PARPORT_MODES, "modes",
- NULL, 0, 0444, NULL,
- &do_hardware_modes },
+ {
+ .ctl_name = DEV_PARPORT_SPINTIME,
+ .procname = "spintime",
+ .maxlen = sizeof(int),
+ .mode = 0644,
+ .proc_handler = &proc_dointvec_minmax,
+ .extra1 = (void *) &parport_min_spintime_value,
+ .extra2 = (void *) &parport_max_spintime_value,
+ },
+ {
+ .ctl_name = DEV_PARPORT_BASE_ADDR,
+ .procname = "base-addr",
+ .mode = 0444,
+ .proc_handler = &do_hardware_base_addr,
+ },
+ {
+ .ctl_name = DEV_PARPORT_IRQ,
+ .procname = "irq",
+ .mode = 0444,
+ .proc_handler = &do_hardware_irq,
+ },
+ {
+ .ctl_name = DEV_PARPORT_DMA,
+ .procname = "dma",
+ .mode = 0444,
+ .proc_handler = &do_hardware_dma,
+ },
+ {
+ .ctl_name = DEV_PARPORT_MODES,
+ .procname = "modes",
+ .mode = 0444,
+ .proc_handler = &do_hardware_modes,
+ },
PARPORT_DEVICES_ROOT_DIR,
#ifdef CONFIG_PARPORT_1284
- { DEV_PARPORT_AUTOPROBE, "autoprobe",
- NULL, 0, 0444, NULL,
- &do_autoprobe },
- { DEV_PARPORT_AUTOPROBE + 1, "autoprobe0",
- NULL, 0, 0444, NULL,
- &do_autoprobe },
- { DEV_PARPORT_AUTOPROBE + 2, "autoprobe1",
- NULL, 0, 0444, NULL,
- &do_autoprobe },
- { DEV_PARPORT_AUTOPROBE + 3, "autoprobe2",
- NULL, 0, 0444, NULL,
- &do_autoprobe },
- { DEV_PARPORT_AUTOPROBE + 4, "autoprobe3",
- NULL, 0, 0444, NULL,
- &do_autoprobe },
+ {
+ .ctl_name = DEV_PARPORT_AUTOPROBE,
+ .procname = "autoprobe",
+ .mode = 0444,
+ .proc_handler = &do_autoprobe,
+ },
+ {
+ .ctl_name = DEV_PARPORT_AUTOPROBE + 1,
+ .procname = "autoprobe0",
+ .mode = 0444,
+ .proc_handler = &do_autoprobe,
+ },
+ {
+ .ctl_name = DEV_PARPORT_AUTOPROBE + 2,
+ .procname = "autoprobe1",
+ .mode = 0444,
+ .proc_handler = &do_autoprobe,
+ },
+ {
+ .ctl_name = DEV_PARPORT_AUTOPROBE + 3,
+ .procname = "autoprobe2",
+ .mode = 0444,
+ .proc_handler = &do_autoprobe,
+ },
+ {
+ .ctl_name = DEV_PARPORT_AUTOPROBE + 4,
+ .procname = "autoprobe3",
+ .mode = 0444,
+ .proc_handler = &do_autoprobe,
+ },
#endif /* IEEE 1284 support */
- {0}
+ {
+ .ctl_name = 0, /* sentinel */
+ }
},
- { {DEV_PARPORT_DEVICES_ACTIVE, "active", NULL, 0, 0444, NULL,
- &do_active_device }, {0}},
- { PARPORT_PORT_DIR(NULL), {0}},
- { PARPORT_PARPORT_DIR(NULL), {0}},
- { PARPORT_DEV_DIR(NULL), {0}}
+ {
+ {
+ .ctl_name = DEV_PARPORT_DEVICES_ACTIVE,
+ .procname = "active",
+ .mode = 0444,
+ .proc_handler = &do_active_device,
+ },
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_PORT_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_PARPORT_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_DEV_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ }
};
struct parport_device_sysctl_table
@@ -323,17 +397,51 @@
parport_device_sysctl_template = {
NULL,
{
- { DEV_PARPORT_DEVICE_TIMESLICE, "timeslice",
- NULL, sizeof(int), 0644, NULL,
- &proc_doulongvec_ms_jiffies_minmax, NULL, NULL,
- (void*) &parport_min_timeslice_value,
- (void*) &parport_max_timeslice_value },
- },
- { {0, NULL, NULL, 0, 0555, NULL}, {0}},
- { PARPORT_DEVICES_ROOT_DIR, {0}},
- { PARPORT_PORT_DIR(NULL), {0}},
- { PARPORT_PARPORT_DIR(NULL), {0}},
- { PARPORT_DEV_DIR(NULL), {0}}
+ {
+ .ctl_name = DEV_PARPORT_DEVICE_TIMESLICE,
+ .procname = "timeslice",
+ .maxlen = sizeof(int),
+ .mode = 0644,
+ .proc_handler = &proc_doulongvec_ms_jiffies_minmax,
+ .extra1 = (void *) &parport_min_timeslice_value,
+ .extra2 = (void *) &parport_max_timeslice_value
+ },
+ {
+ ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ {
+ .mode = 0555,
+ },
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_DEVICES_ROOT_DIR,
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_PORT_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_PARPORT_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_DEV_DIR(NULL),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ }
};
struct parport_default_sysctl_table
@@ -352,26 +460,53 @@
parport_default_sysctl_table = {
NULL,
{
- { DEV_PARPORT_DEFAULT_TIMESLICE, "timeslice",
- &parport_default_timeslice,
- sizeof(parport_default_timeslice), 0644, NULL,
- &proc_doulongvec_ms_jiffies_minmax, NULL, NULL,
- (void*) &parport_min_timeslice_value,
- (void*) &parport_max_timeslice_value },
- { DEV_PARPORT_DEFAULT_SPINTIME, "spintime",
- &parport_default_spintime,
- sizeof(parport_default_spintime), 0644, NULL,
- &proc_dointvec_minmax, NULL, NULL,
- (void*) &parport_min_spintime_value,
- (void*) &parport_max_spintime_value },
- {0}
- },
- { { DEV_PARPORT_DEFAULT, "default", NULL, 0, 0555,
- parport_default_sysctl_table.vars },{0}},
- {
- PARPORT_PARPORT_DIR(parport_default_sysctl_table.default_dir),
- {0}},
- { PARPORT_DEV_DIR(parport_default_sysctl_table.parport_dir), {0}}
+ {
+ .ctl_name = DEV_PARPORT_DEFAULT_TIMESLICE,
+ .procname = "timeslice",
+ .data - &parport_default_timeslice,
+ .maxlen = sizeof(parport_default_timeslice),
+ .mode = 0644,
+ .proc_handler = &proc_doulongvec_ms_jiffies_minmax,
+ .extra1 = (void *) &parport_min_timeslice_value,
+ .extra2 = (void *) &parport_max_timeslice_value,
+ },
+ {
+ .ctl_name = DEV_PARPORT_DEFAULT_SPINTIME,
+ .procname = "spintime",
+ .data = &parport_default_spintime,
+ .maxlen = sizeof(parport_default_spintime),
+ .mode = 0644,
+ .proc_handler = &proc_dointvec_minmax,
+ .extra1 = (void *) &parport_min_spintime_value,
+ .extra2 = (void *) &parport_max_spintime_value,
+ },
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ {
+ .ctl_name = DEV_PARPORT_DEFAULT,
+ .procname = "default",
+ .mode = 0555,
+ .proc_handler = parport_default_sysctl_table.vars,
+ },
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_PARPORT_DIR(parport_default_sysctl_table.default_dir),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ },
+ {
+ PARPORT_DEV_DIR(parport_default_sysctl_table.parport_dir),
+ {
+ .ctl_name = 0, /* sentinel */
+ }
+ }
};
--
Hideaki YOSHIFUJI @ USAGI Project <yoshfuji@linux-ipv6.org>
GPG FP: 9022 65EB 1ECF 3AD1 0BDF 80D8 4807 F894 E062 0EEA
next prev parent reply other threads:[~2004-02-01 13:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-02-01 13:44 YOSHIFUJI Hideaki / 吉藤英明
2004-02-01 13:56 ` YOSHIFUJI Hideaki / 吉藤英明 [this message]
2004-02-01 14:35 ` YOSHIFUJI Hideaki / 吉藤英明
2004-02-02 12:24 ` Wojciech 'Sas' Cieciwa
2004-02-01 17:48 ` Arthur Othieno
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=20040201.225619.67854403.yoshfuji@linux-ipv6.org \
--to=yoshfuji@linux-ipv6.org \
--cc=Philip.Blundell@pobox.com \
--cc=andrea@e-mind.com \
--cc=campbell@torque.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parport@torque.net \
--cc=tim@cyberelk.net \
/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
all inboxes | Powered by JetHome®