[PATCH] Staging: comedi: Fix an other build problem

[PATCH] Staging: comedi: Fix an other build problem

[Alessio Igor Bogani]

[...]
 CC [M]  drivers/staging/comedi/comedi_compat32.o
 drivers/staging/comedi/comedi_compat32.c: In function ‘raw_ioctl’:
 drivers/staging/comedi/comedi_compat32.c:451: error: ‘comedi32_chaninfo’ undeclared (first use in this function)
 drivers/staging/comedi/comedi_compat32.c:451: error: (Each undeclared identifier is reported only once
 drivers/staging/comedi/comedi_compat32.c:451: error: for each function it appears in.)
 drivers/staging/comedi/comedi_compat32.c:454: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:454: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:457: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:457: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:460: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:460: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:463: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:463: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:466: error: expected expression before ‘)’ token
 drivers/staging/comedi/comedi_compat32.c:466: error: expected expression before ‘)’ token
 [...]

Signed-off-by: Alessio Igor Bogani <abogani@texware.it>
---
 drivers/staging/comedi/comedi_compat32.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/comedi/comedi_compat32.c b/drivers/staging/comedi/comedi_compat32.c
index 2fa771e..5708f46 100644
--- a/drivers/staging/comedi/comedi_compat32.c
+++ b/drivers/staging/comedi/comedi_compat32.c
@@ -37,16 +37,16 @@
 #include <linux/ioctl32.h>	/* for (un)register_ioctl32_conversion */
 #endif
 
-#define COMEDI32_CHANINFO _IOR(CIO, 3, comedi32_chaninfo)
-#define COMEDI32_RANGEINFO _IOR(CIO, 8, (struct comedi32_rangeinfo_struct))
+#define COMEDI32_CHANINFO _IOR(CIO, 3, struct comedi32_chaninfo_struct)
+#define COMEDI32_RANGEINFO _IOR(CIO, 8, struct comedi32_rangeinfo_struct)
 /* N.B. COMEDI32_CMD and COMEDI_CMD ought to use _IOWR, not _IOR.
  * It's too late to change it now, but it only affects the command number. */
-#define COMEDI32_CMD _IOR(CIO, 9, (struct comedi32_cmd_struct))
+#define COMEDI32_CMD _IOR(CIO, 9, struct comedi32_cmd_struct)
 /* N.B. COMEDI32_CMDTEST and COMEDI_CMDTEST ought to use _IOWR, not _IOR.
  * It's too late to change it now, but it only affects the command number. */
-#define COMEDI32_CMDTEST _IOR(CIO, 10, (struct comedi32_cmd_struct))
-#define COMEDI32_INSNLIST _IOR(CIO, 11, (struct comedi32_insnlist_struct))
-#define COMEDI32_INSN _IOR(CIO, 12, (struct comedi32_insn_struct))
+#define COMEDI32_CMDTEST _IOR(CIO, 10, struct comedi32_cmd_struct)
+#define COMEDI32_INSNLIST _IOR(CIO, 11, struct comedi32_insnlist_struct)
+#define COMEDI32_INSN _IOR(CIO, 12, struct comedi32_insn_struct)
 
 struct comedi32_chaninfo_struct {
 	unsigned int subdev;
-- 
1.6.0.4

Re: [PATCH] Staging: comedi: Fix an other build problem

[Greg KH]

On Fri, Mar 20, 2009 at 05:37:52PM +0100, Alessio Igor Bogani wrote:
> [...]
>  CC [M]  drivers/staging/comedi/comedi_compat32.o
>  drivers/staging/comedi/comedi_compat32.c: In function ‘raw_ioctl’:
>  drivers/staging/comedi/comedi_compat32.c:451: error: ‘comedi32_chaninfo’ undeclared (first use in this function)
>  drivers/staging/comedi/comedi_compat32.c:451: error: (Each undeclared identifier is reported only once
>  drivers/staging/comedi/comedi_compat32.c:451: error: for each function it appears in.)
>  drivers/staging/comedi/comedi_compat32.c:454: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:454: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:457: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:457: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:460: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:460: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:463: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:463: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:466: error: expected expression before ‘)’ token
>  drivers/staging/comedi/comedi_compat32.c:466: error: expected expression before ‘)’ token
>  [...]

Ah, sorry, I just fixed this before I saw your patch (based on Randy's
previous complaint.)

thanks,

greg k-h