Thread (82 messages) 82 messages, 14 authors, 2007-01-27

Re: [PATCH 45/59] sysctl: C99 convert ctl_tables in drivers/parport/procfs.c

From: Ingo Oeser <hidden>
Date: 2007-01-16 22:16:08
Also in: linux-mips, linux-scsi, linuxppc-dev, lkml

Hi Eric,

On Tuesday, 16. January 2007 17:39, Eric W. Biederman wrote:
quoted hunk ↗ jump to hunk
diff --git a/drivers/parport/procfs.c b/drivers/parport/procfs.c
index 2e744a2..5337789 100644
--- a/drivers/parport/procfs.c
+++ b/drivers/parport/procfs.c
@@ -263,50 +263,118 @@ struct parport_sysctl_table {
+		{
+			.ctl_name	= DEV_PARPORT_BASE_ADDR,
+			.procname	= "base-addr",
+			.data		= NULL,
+			.maxlen		= 0,
+			.mode		= 0444,
+			.proc_handler	= &do_hardware_base_addr
+		},
No need to initialize to zero or NULL. Just list any variable, which is NOT zero or NULL.
+		{
+			.ctl_name	= DEV_PARPORT_AUTOPROBE + 1,
+			.procname	= "autoprobe0",
+			.data		= NULL,
+			.maxlen		= 0,
+			.maxlen		= 0444,
+			.proc_handler	=  &do_autoprobe
+		},
Typo here? .mode = 0444 makes mor sense.

Regards

Ingo Oeser
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help