Revert to Fedora kernel-2.6.17-1.2187_FC5 patched with vs2.0.2.1; there are too many...
[linux-2.6.git] / drivers / media / video / pwc / pwc-ctrl.c
index 0bd1155..4ba549b 100644 (file)
@@ -2,7 +2,7 @@
    Functions that send various control messages to the webcam, including
    video modes.
    (C) 1999-2003 Nemosoft Unv.
-   (C) 2004-2006 Luc Saillard (luc@saillard.org)
+   (C) 2004      Luc Saillard (luc@saillard.org)
 
    NOTE: this version of pwc is an unofficial (modified) release of pwc & pcwx
    driver and thus may have bugs that are not present in the original version.
 #include <asm/uaccess.h>
 #endif
 #include <asm/errno.h>
-#include <linux/version.h>
 
 #include "pwc.h"
+#include "pwc-ioctl.h"
 #include "pwc-uncompress.h"
 #include "pwc-kiara.h"
 #include "pwc-timon.h"
-#include "pwc-dec1.h"
-#include "pwc-dec23.h"
 
 /* Request types: video */
 #define SET_LUM_CTL                    0x01
 #define GET_STATUS_CTL                 0x06
 #define SET_EP_STREAM_CTL              0x07
 #define GET_EP_STREAM_CTL              0x08
-#define GET_XX_CTL                     0x09
-#define SET_XX_CTL                     0x0A
-#define GET_XY_CTL                     0x0B
-#define SET_XY_CTL                     0x0C
 #define SET_MPT_CTL                    0x0D
 #define GET_MPT_CTL                    0x0E
 
 #define READ_SHUTTER_FORMATTER                 0x0600
 #define READ_RED_GAIN_FORMATTER                        0x0700
 #define READ_BLUE_GAIN_FORMATTER               0x0800
-#define GET_STATUS_B00                         0x0B00
 #define SENSOR_TYPE_FORMATTER1                 0x0C00
-#define GET_STATUS_3000                                0x3000
 #define READ_RAW_Y_MEAN_FORMATTER              0x3100
 #define SET_POWER_SAVE_MODE_FORMATTER          0x3200
 #define MIRROR_IMAGE_FORMATTER                 0x3300
 #define LED_FORMATTER                          0x3400
-#define LOWLIGHT                               0x3500
-#define GET_STATUS_3600                                0x3600
 #define SENSOR_TYPE_FORMATTER2                 0x3700
-#define GET_STATUS_3800                                0x3800
-#define GET_STATUS_4000                                0x4000
-#define GET_STATUS_4100                                0x4100  /* Get */
-#define CTL_STATUS_4200                                0x4200  /* [GS] 1 */
 
 /* Formatters for the Video Endpoint controls [GS]ET_EP_STREAM_CTL */
 #define VIDEO_OUTPUT_CONTROL_FORMATTER         0x0100
@@ -152,7 +138,6 @@ static struct Nala_table_entry Nala_table[PSZ_MAX][8] =
 #include "pwc-nala.h"
 };
 
-static void pwc_set_image_buffer_size(struct pwc_device *pdev);
 
 /****************************************************************************/
 
@@ -174,7 +159,31 @@ static void pwc_set_image_buffer_size(struct pwc_device *pdev);
                &buf, buflen, 500)
 
 
-static int send_video_command(struct usb_device *udev, int index, void *buf, int buflen)
+#if PWC_DEBUG
+void pwc_hexdump(void *p, int len)
+{
+       int i;
+       unsigned char *s;
+       char buf[100], *d;
+
+       s = (unsigned char *)p;
+       d = buf;
+       *d = '\0';
+       Debug("Doing hexdump @ %p, %d bytes.\n", p, len);
+       for (i = 0; i < len; i++) {
+               d += sprintf(d, "%02X ", *s++);
+               if ((i & 0xF) == 0xF) {
+                       Debug("%s\n", buf);
+                       d = buf;
+                       *d = '\0';
+               }
+       }
+       if ((i & 0xF) != 0)
+               Debug("%s\n", buf);
+}
+#endif
+
+static inline int send_video_command(struct usb_device *udev, int index, void *buf, int buflen)
 {
        return usb_control_msg(udev,
                usb_sndctrlpipe(udev, 0),
@@ -187,7 +196,7 @@ static int send_video_command(struct usb_device *udev, int index, void *buf, int
 
 
 
-static int set_video_mode_Nala(struct pwc_device *pdev, int size, int frames)
+static inline int set_video_mode_Nala(struct pwc_device *pdev, int size, int frames)
 {
        unsigned char buf[3];
        int ret, fps;
@@ -220,14 +229,34 @@ static int set_video_mode_Nala(struct pwc_device *pdev, int size, int frames)
        if (pEntry->alternate == 0)
                return -EINVAL;
 
+       if (pEntry->compressed)
+               return -ENOENT; /* Not supported. */
+
        memcpy(buf, pEntry->mode, 3);
        ret = send_video_command(pdev->udev, pdev->vendpoint, buf, 3);
        if (ret < 0) {
-               PWC_DEBUG_MODULE("Failed to send video command... %d\n", ret);
+               Debug("Failed to send video command... %d\n", ret);
                return ret;
        }
        if (pEntry->compressed && pdev->vpalette != VIDEO_PALETTE_RAW)
-               pwc_dec1_init(pdev->type, pdev->release, buf, pdev->decompress_data);
+        {
+          switch(pdev->type) {
+            case 645:
+            case 646:
+/*            pwc_dec1_init(pdev->type, pdev->release, buf, pdev->decompress_data); */
+              break;
+
+            case 675:
+            case 680:
+            case 690:
+            case 720:
+            case 730:
+            case 740:
+            case 750:
+/*            pwc_dec23_init(pdev->type, pdev->release, buf, pdev->decompress_data); */
+              break;
+          }
+       }
 
        pdev->cmd_len = 3;
        memcpy(pdev->cmd_buf, buf, 3);
@@ -254,7 +283,7 @@ static int set_video_mode_Nala(struct pwc_device *pdev, int size, int frames)
 }
 
 
-static int set_video_mode_Timon(struct pwc_device *pdev, int size, int frames, int compression, int snapshot)
+static inline int set_video_mode_Timon(struct pwc_device *pdev, int size, int frames, int compression, int snapshot)
 {
        unsigned char buf[13];
        const struct Timon_table_entry *pChoose;
@@ -286,8 +315,8 @@ static int set_video_mode_Timon(struct pwc_device *pdev, int size, int frames, i
        if (ret < 0)
                return ret;
 
-       if (pChoose->bandlength > 0 && pdev->vpalette != VIDEO_PALETTE_RAW)
-               pwc_dec23_init(pdev, pdev->type, buf);
+/*     if (pChoose->bandlength > 0 && pdev->vpalette != VIDEO_PALETTE_RAW)
+          pwc_dec23_init(pdev->type, pdev->release, buf, pdev->decompress_data); */
 
        pdev->cmd_len = 13;
        memcpy(pdev->cmd_buf, buf, 13);
@@ -307,7 +336,7 @@ static int set_video_mode_Timon(struct pwc_device *pdev, int size, int frames, i
 }
 
 
-static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, int compression, int snapshot)
+static inline int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, int compression, int snapshot)
 {
        const struct Kiara_table_entry *pChoose = NULL;
        int fps, ret;
@@ -321,14 +350,21 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, i
        fps = (frames / 5) - 1;
 
        /* special case: VGA @ 5 fps and snapshot is raw bayer mode */
-       if (size == PSZ_VGA && frames == 5 && snapshot && pdev->vpalette == VIDEO_PALETTE_RAW)
+       if (size == PSZ_VGA && frames == 5 && snapshot)
        {
                /* Only available in case the raw palette is selected or
                   we have the decompressor available. This mode is
                   only available in compressed form
                */
-               PWC_DEBUG_SIZE("Choosing VGA/5 BAYER mode.\n");
-               pChoose = &RawEntry;
+               if (pdev->vpalette == VIDEO_PALETTE_RAW)
+               {
+                       Info("Choosing VGA/5 BAYER mode (%d).\n", pdev->vpalette);
+                       pChoose = &RawEntry;
+               }
+               else
+               {
+                       Info("VGA/5 BAYER mode _must_ have a decompressor available, or use RAW palette.\n");
+               }
        }
        else
        {
@@ -336,7 +372,6 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, i
                   if the preferred ratio is not available.
                   Skip this step when using RAW modes.
                */
-               snapshot = 0;
                while (compression <= 3) {
                        pChoose = &Kiara_table[size][fps][compression];
                        if (pChoose->alternate != 0)
@@ -347,7 +382,7 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, i
        if (pChoose == NULL || pChoose->alternate == 0)
                return -ENOENT; /* Not supported. */
 
-       PWC_TRACE("Using alternate setting %d.\n", pChoose->alternate);
+       Debug("Using alternate setting %d.\n", pChoose->alternate);
 
        /* usb_control_msg won't take staticly allocated arrays as argument?? */
        memcpy(buf, pChoose->mode, 12);
@@ -359,8 +394,8 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, i
        if (ret < 0)
                return ret;
 
-       if (pChoose->bandlength > 0 && pdev->vpalette != VIDEO_PALETTE_RAW)
-               pwc_dec23_init(pdev, pdev->type, buf);
+/*     if (pChoose->bandlength > 0 && pdev->vpalette != VIDEO_PALETTE_RAW)
+         pwc_dec23_init(pdev->type, pdev->release, buf, pdev->decompress_data); */
 
        pdev->cmd_len = 12;
        memcpy(pdev->cmd_buf, buf, 12);
@@ -375,13 +410,49 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int frames, i
                pdev->frame_size = (pdev->vbandlength * pdev->image.y) / 4;
        else
                pdev->frame_size = (pdev->image.x * pdev->image.y * 12) / 8;
-       PWC_TRACE("frame_size=%d, vframes=%d, vsize=%d, vsnapshot=%d, vbandlength=%d\n",
-           pdev->frame_size,pdev->vframes,pdev->vsize,pdev->vsnapshot,pdev->vbandlength);
        return 0;
 }
 
 
 
+static void pwc_set_image_buffer_size(struct pwc_device *pdev)
+{
+       int i, factor = 0, filler = 0;
+
+       /* for PALETTE_YUV420P */
+       switch(pdev->vpalette)
+       {
+       case VIDEO_PALETTE_YUV420P:
+               factor = 6;
+               filler = 128;
+               break;
+       case VIDEO_PALETTE_RAW:
+               factor = 6; /* can be uncompressed YUV420P */
+               filler = 0;
+               break;
+       }
+
+       /* Set sizes in bytes */
+       pdev->image.size = pdev->image.x * pdev->image.y * factor / 4;
+       pdev->view.size  = pdev->view.x  * pdev->view.y  * factor / 4;
+
+       /* Align offset, or you'll get some very weird results in
+          YUV420 mode... x must be multiple of 4 (to get the Y's in
+          place), and y even (or you'll mixup U & V). This is less of a
+          problem for YUV420P.
+        */
+       pdev->offset.x = ((pdev->view.x - pdev->image.x) / 2) & 0xFFFC;
+       pdev->offset.y = ((pdev->view.y - pdev->image.y) / 2) & 0xFFFE;
+
+       /* Fill buffers with gray or black */
+       for (i = 0; i < MAX_IMAGES; i++) {
+               if (pdev->image_ptr[i] != NULL)
+                       memset(pdev->image_ptr[i], filler, pdev->view.size);
+       }
+}
+
+
+
 /**
    @pdev: device structure
    @width: viewport width
@@ -394,78 +465,50 @@ int pwc_set_video_mode(struct pwc_device *pdev, int width, int height, int frame
 {
        int ret, size;
 
-       PWC_DEBUG_FLOW("set_video_mode(%dx%d @ %d, palette %d).\n", width, height, frames, pdev->vpalette);
+       Trace(TRACE_FLOW, "set_video_mode(%dx%d @ %d, palette %d).\n", width, height, frames, pdev->vpalette);
        size = pwc_decode_size(pdev, width, height);
        if (size < 0) {
-               PWC_DEBUG_MODULE("Could not find suitable size.\n");
+               Debug("Could not find suitable size.\n");
                return -ERANGE;
        }
-       PWC_TRACE("decode_size = %d.\n", size);
+       Debug("decode_size = %d.\n", size);
 
-       if (DEVICE_USE_CODEC1(pdev->type)) {
+       ret = -EINVAL;
+       switch(pdev->type) {
+       case 645:
+       case 646:
                ret = set_video_mode_Nala(pdev, size, frames);
+               break;
 
-       } else if (DEVICE_USE_CODEC3(pdev->type)) {
-               ret = set_video_mode_Kiara(pdev, size, frames, compression, snapshot);
-
-       } else {
+       case 675:
+       case 680:
+       case 690:
                ret = set_video_mode_Timon(pdev, size, frames, compression, snapshot);
+               break;
+
+       case 720:
+       case 730:
+       case 740:
+       case 750:
+               ret = set_video_mode_Kiara(pdev, size, frames, compression, snapshot);
+               break;
        }
        if (ret < 0) {
-               PWC_ERROR("Failed to set video mode %s@%d fps; return code = %d\n", size2name[size], frames, ret);
+               if (ret == -ENOENT)
+                       Info("Video mode %s@%d fps is only supported with the decompressor module (pwcx).\n", size2name[size], frames);
+               else {
+                       Err("Failed to set video mode %s@%d fps; return code = %d\n", size2name[size], frames, ret);
+               }
                return ret;
        }
        pdev->view.x = width;
        pdev->view.y = height;
        pdev->frame_total_size = pdev->frame_size + pdev->frame_header_size + pdev->frame_trailer_size;
        pwc_set_image_buffer_size(pdev);
-       PWC_DEBUG_SIZE("Set viewport to %dx%d, image size is %dx%d.\n", width, height, pwc_image_sizes[size].x, pwc_image_sizes[size].y);
+       Trace(TRACE_SIZE, "Set viewport to %dx%d, image size is %dx%d.\n", width, height, pwc_image_sizes[size].x, pwc_image_sizes[size].y);
        return 0;
 }
 
-#define BLACK_Y 0
-#define BLACK_U 128
-#define BLACK_V 128
-
-static void pwc_set_image_buffer_size(struct pwc_device *pdev)
-{
-       int i, factor = 0;
-
-       /* for PALETTE_YUV420P */
-       switch(pdev->vpalette)
-       {
-       case VIDEO_PALETTE_YUV420P:
-               factor = 6;
-               break;
-       case VIDEO_PALETTE_RAW:
-               factor = 6; /* can be uncompressed YUV420P */
-               break;
-       }
-
-       /* Set sizes in bytes */
-       pdev->image.size = pdev->image.x * pdev->image.y * factor / 4;
-       pdev->view.size  = pdev->view.x  * pdev->view.y  * factor / 4;
-
-       /* Align offset, or you'll get some very weird results in
-          YUV420 mode... x must be multiple of 4 (to get the Y's in
-          place), and y even (or you'll mixup U & V). This is less of a
-          problem for YUV420P.
-        */
-       pdev->offset.x = ((pdev->view.x - pdev->image.x) / 2) & 0xFFFC;
-       pdev->offset.y = ((pdev->view.y - pdev->image.y) / 2) & 0xFFFE;
-
-       /* Fill buffers with black colors */
-       for (i = 0; i < pwc_mbufs; i++) {
-               unsigned char *p = pdev->image_data + pdev->images[i].offset;
-               memset(p, BLACK_Y, pdev->view.x * pdev->view.y);
-               p += pdev->view.x * pdev->view.y;
-               memset(p, BLACK_U, pdev->view.x * pdev->view.y/4);
-               p += pdev->view.x * pdev->view.y/4;
-               memset(p, BLACK_V, pdev->view.x * pdev->view.y/4);
-       }
-}
-
-
 
 /* BRIGHTNESS */
 
@@ -477,7 +520,7 @@ int pwc_get_brightness(struct pwc_device *pdev)
        ret = RecvControlMsg(GET_LUM_CTL, BRIGHTNESS_FORMATTER, 1);
        if (ret < 0)
                return ret;
-       return buf;
+       return buf << 9;
 }
 
 int pwc_set_brightness(struct pwc_device *pdev, int value)
@@ -502,7 +545,7 @@ int pwc_get_contrast(struct pwc_device *pdev)
        ret = RecvControlMsg(GET_LUM_CTL, CONTRAST_FORMATTER, 1);
        if (ret < 0)
                return ret;
-       return buf;
+       return buf << 10;
 }
 
 int pwc_set_contrast(struct pwc_device *pdev, int value)
@@ -527,7 +570,7 @@ int pwc_get_gamma(struct pwc_device *pdev)
        ret = RecvControlMsg(GET_LUM_CTL, GAMMA_FORMATTER, 1);
        if (ret < 0)
                return ret;
-       return buf;
+       return buf << 11;
 }
 
 int pwc_set_gamma(struct pwc_device *pdev, int value)
@@ -545,47 +588,37 @@ int pwc_set_gamma(struct pwc_device *pdev, int value)
 
 /* SATURATION */
 
-/* return a value between [-100 , 100] */
-int pwc_get_saturation(struct pwc_device *pdev, int *value)
+int pwc_get_saturation(struct pwc_device *pdev)
 {
        char buf;
-       int ret, saturation_register;
+       int ret;
 
        if (pdev->type < 675)
-               return -EINVAL;
-       if (pdev->type < 730)
-               saturation_register = SATURATION_MODE_FORMATTER2;
-       else
-               saturation_register = SATURATION_MODE_FORMATTER1;
-       ret = RecvControlMsg(GET_CHROM_CTL, saturation_register, 1);
+               return -1;
+       ret = RecvControlMsg(GET_CHROM_CTL, pdev->type < 730 ? SATURATION_MODE_FORMATTER2 : SATURATION_MODE_FORMATTER1, 1);
        if (ret < 0)
                return ret;
-       *value = (signed)buf;
-       return 0;
+       return 32768 + buf * 327;
 }
 
-/* @param value saturation color between [-100 , 100] */
 int pwc_set_saturation(struct pwc_device *pdev, int value)
 {
        char buf;
-       int saturation_register;
 
        if (pdev->type < 675)
                return -EINVAL;
-       if (value < -100)
-               value = -100;
-       if (value > 100)
-               value = 100;
-       if (pdev->type < 730)
-               saturation_register = SATURATION_MODE_FORMATTER2;
-       else
-               saturation_register = SATURATION_MODE_FORMATTER1;
-       return SendControlMsg(SET_CHROM_CTL, saturation_register, 1);
+       if (value < 0)
+               value = 0;
+       if (value > 0xffff)
+               value = 0xffff;
+       /* saturation ranges from -100 to +100 */
+       buf = (value - 32768) / 327;
+       return SendControlMsg(SET_CHROM_CTL, pdev->type < 730 ? SATURATION_MODE_FORMATTER2 : SATURATION_MODE_FORMATTER1, 1);
 }
 
 /* AGC */
 
-int pwc_set_agc(struct pwc_device *pdev, int mode, int value)
+static inline int pwc_set_agc(struct pwc_device *pdev, int mode, int value)
 {
        char buf;
        int ret;
@@ -610,7 +643,7 @@ int pwc_set_agc(struct pwc_device *pdev, int mode, int value)
        return 0;
 }
 
-int pwc_get_agc(struct pwc_device *pdev, int *value)
+static inline int pwc_get_agc(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -640,7 +673,7 @@ int pwc_get_agc(struct pwc_device *pdev, int *value)
        return 0;
 }
 
-int pwc_set_shutter_speed(struct pwc_device *pdev, int mode, int value)
+static inline int pwc_set_shutter_speed(struct pwc_device *pdev, int mode, int value)
 {
        char buf[2];
        int speed, ret;
@@ -658,16 +691,23 @@ int pwc_set_shutter_speed(struct pwc_device *pdev, int mode, int value)
                        value = 0;
                if (value > 0xffff)
                        value = 0xffff;
-
-               if (DEVICE_USE_CODEC2(pdev->type)) {
+               switch(pdev->type) {
+               case 675:
+               case 680:
+               case 690:
                        /* speed ranges from 0x0 to 0x290 (656) */
                        speed = (value / 100);
                        buf[1] = speed >> 8;
                        buf[0] = speed & 0xff;
-               } else if (DEVICE_USE_CODEC3(pdev->type)) {
+                       break;
+               case 720:
+               case 730:
+               case 740:
+               case 750:
                        /* speed seems to range from 0x0 to 0xff */
                        buf[1] = 0;
                        buf[0] = value >> 8;
+                       break;
                }
 
                ret = SendControlMsg(SET_LUM_CTL, PRESET_SHUTTER_FORMATTER, 2);
@@ -675,25 +715,6 @@ int pwc_set_shutter_speed(struct pwc_device *pdev, int mode, int value)
        return ret;
 }
 
-/* This function is not exported to v4l1, so output values between 0 -> 256 */
-int pwc_get_shutter_speed(struct pwc_device *pdev, int *value)
-{
-       unsigned char buf[2];
-       int ret;
-
-       ret = RecvControlMsg(GET_STATUS_CTL, READ_SHUTTER_FORMATTER, 2);
-       if (ret < 0)
-               return ret;
-       *value = buf[0] + (buf[1] << 8);
-       if (DEVICE_USE_CODEC2(pdev->type)) {
-               /* speed ranges from 0x0 to 0x290 (656) */
-               *value *= 256/656;
-       } else if (DEVICE_USE_CODEC3(pdev->type)) {
-               /* speed seems to range from 0x0 to 0xff */
-       }
-       return 0;
-}
-
 
 /* POWER */
 
@@ -715,19 +736,19 @@ int pwc_camera_power(struct pwc_device *pdev, int power)
 
 /* private calls */
 
-int pwc_restore_user(struct pwc_device *pdev)
+static inline int pwc_restore_user(struct pwc_device *pdev)
 {
        char buf; /* dummy */
        return SendControlMsg(SET_STATUS_CTL, RESTORE_USER_DEFAULTS_FORMATTER, 0);
 }
 
-int pwc_save_user(struct pwc_device *pdev)
+static inline int pwc_save_user(struct pwc_device *pdev)
 {
        char buf; /* dummy */
        return SendControlMsg(SET_STATUS_CTL, SAVE_USER_DEFAULTS_FORMATTER, 0);
 }
 
-int pwc_restore_factory(struct pwc_device *pdev)
+static inline int pwc_restore_factory(struct pwc_device *pdev)
 {
        char buf; /* dummy */
        return SendControlMsg(SET_STATUS_CTL, RESTORE_FACTORY_DEFAULTS_FORMATTER, 0);
@@ -745,7 +766,7 @@ int pwc_restore_factory(struct pwc_device *pdev)
   * 03: manual
   * 04: auto
   */
-int pwc_set_awb(struct pwc_device *pdev, int mode)
+static inline int pwc_set_awb(struct pwc_device *pdev, int mode)
 {
        char buf;
        int ret;
@@ -765,7 +786,7 @@ int pwc_set_awb(struct pwc_device *pdev, int mode)
        return 0;
 }
 
-int pwc_get_awb(struct pwc_device *pdev)
+static inline int pwc_get_awb(struct pwc_device *pdev)
 {
        unsigned char buf;
        int ret;
@@ -777,7 +798,7 @@ int pwc_get_awb(struct pwc_device *pdev)
        return buf;
 }
 
-int pwc_set_red_gain(struct pwc_device *pdev, int value)
+static inline int pwc_set_red_gain(struct pwc_device *pdev, int value)
 {
        unsigned char buf;
 
@@ -790,7 +811,7 @@ int pwc_set_red_gain(struct pwc_device *pdev, int value)
        return SendControlMsg(SET_CHROM_CTL, PRESET_MANUAL_RED_GAIN_FORMATTER, 1);
 }
 
-int pwc_get_red_gain(struct pwc_device *pdev, int *value)
+static inline int pwc_get_red_gain(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -803,7 +824,7 @@ int pwc_get_red_gain(struct pwc_device *pdev, int *value)
 }
 
 
-int pwc_set_blue_gain(struct pwc_device *pdev, int value)
+static inline int pwc_set_blue_gain(struct pwc_device *pdev, int value)
 {
        unsigned char buf;
 
@@ -816,7 +837,7 @@ int pwc_set_blue_gain(struct pwc_device *pdev, int value)
        return SendControlMsg(SET_CHROM_CTL, PRESET_MANUAL_BLUE_GAIN_FORMATTER, 1);
 }
 
-int pwc_get_blue_gain(struct pwc_device *pdev, int *value)
+static inline int pwc_get_blue_gain(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -833,7 +854,7 @@ int pwc_get_blue_gain(struct pwc_device *pdev, int *value)
    internal red/blue gains, which may be different from the manual
    gains set or read above.
  */
-static int pwc_read_red_gain(struct pwc_device *pdev, int *value)
+static inline int pwc_read_red_gain(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -845,7 +866,7 @@ static int pwc_read_red_gain(struct pwc_device *pdev, int *value)
        return 0;
 }
 
-static int pwc_read_blue_gain(struct pwc_device *pdev, int *value)
+static inline int pwc_read_blue_gain(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -858,7 +879,7 @@ static int pwc_read_blue_gain(struct pwc_device *pdev, int *value)
 }
 
 
-static int pwc_set_wb_speed(struct pwc_device *pdev, int speed)
+static inline int pwc_set_wb_speed(struct pwc_device *pdev, int speed)
 {
        unsigned char buf;
 
@@ -867,7 +888,7 @@ static int pwc_set_wb_speed(struct pwc_device *pdev, int speed)
        return SendControlMsg(SET_CHROM_CTL, AWB_CONTROL_SPEED_FORMATTER, 1);
 }
 
-static int pwc_get_wb_speed(struct pwc_device *pdev, int *value)
+static inline int pwc_get_wb_speed(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -880,7 +901,7 @@ static int pwc_get_wb_speed(struct pwc_device *pdev, int *value)
 }
 
 
-static int pwc_set_wb_delay(struct pwc_device *pdev, int delay)
+static inline int pwc_set_wb_delay(struct pwc_device *pdev, int delay)
 {
        unsigned char buf;
 
@@ -889,7 +910,7 @@ static int pwc_set_wb_delay(struct pwc_device *pdev, int delay)
        return SendControlMsg(SET_CHROM_CTL, AWB_CONTROL_DELAY_FORMATTER, 1);
 }
 
-static int pwc_get_wb_delay(struct pwc_device *pdev, int *value)
+static inline int pwc_get_wb_delay(struct pwc_device *pdev, int *value)
 {
        unsigned char buf;
        int ret;
@@ -944,7 +965,7 @@ static int pwc_get_leds(struct pwc_device *pdev, int *on_value, int *off_value)
        return 0;
 }
 
-int pwc_set_contour(struct pwc_device *pdev, int contour)
+static inline int pwc_set_contour(struct pwc_device *pdev, int contour)
 {
        unsigned char buf;
        int ret;
@@ -969,7 +990,7 @@ int pwc_set_contour(struct pwc_device *pdev, int contour)
        return 0;
 }
 
-int pwc_get_contour(struct pwc_device *pdev, int *contour)
+static inline int pwc_get_contour(struct pwc_device *pdev, int *contour)
 {
        unsigned char buf;
        int ret;
@@ -991,7 +1012,7 @@ int pwc_get_contour(struct pwc_device *pdev, int *contour)
 }
 
 
-int pwc_set_backlight(struct pwc_device *pdev, int backlight)
+static inline int pwc_set_backlight(struct pwc_device *pdev, int backlight)
 {
        unsigned char buf;
 
@@ -1002,7 +1023,7 @@ int pwc_set_backlight(struct pwc_device *pdev, int backlight)
        return SendControlMsg(SET_LUM_CTL, BACK_LIGHT_COMPENSATION_FORMATTER, 1);
 }
 
-int pwc_get_backlight(struct pwc_device *pdev, int *backlight)
+static inline int pwc_get_backlight(struct pwc_device *pdev, int *backlight)
 {
        int ret;
        unsigned char buf;
@@ -1010,35 +1031,12 @@ int pwc_get_backlight(struct pwc_device *pdev, int *backlight)
        ret = RecvControlMsg(GET_LUM_CTL, BACK_LIGHT_COMPENSATION_FORMATTER, 1);
        if (ret < 0)
                return ret;
-       *backlight = !!buf;
+       *backlight = buf;
        return 0;
 }
 
-int pwc_set_colour_mode(struct pwc_device *pdev, int colour)
-{
-       unsigned char buf;
 
-       if (colour)
-               buf = 0xff;
-       else
-               buf = 0x0;
-       return SendControlMsg(SET_CHROM_CTL, COLOUR_MODE_FORMATTER, 1);
-}
-
-int pwc_get_colour_mode(struct pwc_device *pdev, int *colour)
-{
-       int ret;
-       unsigned char buf;
-
-       ret = RecvControlMsg(GET_CHROM_CTL, COLOUR_MODE_FORMATTER, 1);
-       if (ret < 0)
-               return ret;
-       *colour = !!buf;
-       return 0;
-}
-
-
-int pwc_set_flicker(struct pwc_device *pdev, int flicker)
+static inline int pwc_set_flicker(struct pwc_device *pdev, int flicker)
 {
        unsigned char buf;
 
@@ -1049,7 +1047,7 @@ int pwc_set_flicker(struct pwc_device *pdev, int flicker)
        return SendControlMsg(SET_LUM_CTL, FLICKERLESS_MODE_FORMATTER, 1);
 }
 
-int pwc_get_flicker(struct pwc_device *pdev, int *flicker)
+static inline int pwc_get_flicker(struct pwc_device *pdev, int *flicker)
 {
        int ret;
        unsigned char buf;
@@ -1057,11 +1055,12 @@ int pwc_get_flicker(struct pwc_device *pdev, int *flicker)
        ret = RecvControlMsg(GET_LUM_CTL, FLICKERLESS_MODE_FORMATTER, 1);
        if (ret < 0)
                return ret;
-       *flicker = !!buf;
+       *flicker = buf;
        return 0;
 }
 
-int pwc_set_dynamic_noise(struct pwc_device *pdev, int noise)
+
+static inline int pwc_set_dynamic_noise(struct pwc_device *pdev, int noise)
 {
        unsigned char buf;
 
@@ -1073,7 +1072,7 @@ int pwc_set_dynamic_noise(struct pwc_device *pdev, int noise)
        return SendControlMsg(SET_LUM_CTL, DYNAMIC_NOISE_CONTROL_FORMATTER, 1);
 }
 
-int pwc_get_dynamic_noise(struct pwc_device *pdev, int *noise)
+static inline int pwc_get_dynamic_noise(struct pwc_device *pdev, int *noise)
 {
        int ret;
        unsigned char buf;
@@ -1085,7 +1084,7 @@ int pwc_get_dynamic_noise(struct pwc_device *pdev, int *noise)
        return 0;
 }
 
-static int _pwc_mpt_reset(struct pwc_device *pdev, int flags)
+static int pwc_mpt_reset(struct pwc_device *pdev, int flags)
 {
        unsigned char buf;
 
@@ -1093,18 +1092,7 @@ static int _pwc_mpt_reset(struct pwc_device *pdev, int flags)
        return SendControlMsg(SET_MPT_CTL, PT_RESET_CONTROL_FORMATTER, 1);
 }
 
-int pwc_mpt_reset(struct pwc_device *pdev, int flags)
-{
-       int ret;
-       ret = _pwc_mpt_reset(pdev, flags);
-       if (ret >= 0) {
-               pdev->pan_angle = 0;
-               pdev->tilt_angle = 0;
-       }
-       return ret;
-}
-
-static int _pwc_mpt_set_angle(struct pwc_device *pdev, int pan, int tilt)
+static inline int pwc_mpt_set_angle(struct pwc_device *pdev, int pan, int tilt)
 {
        unsigned char buf[4];
 
@@ -1122,35 +1110,7 @@ static int _pwc_mpt_set_angle(struct pwc_device *pdev, int pan, int tilt)
        return SendControlMsg(SET_MPT_CTL, PT_RELATIVE_CONTROL_FORMATTER, 4);
 }
 
-int pwc_mpt_set_angle(struct pwc_device *pdev, int pan, int tilt)
-{
-       int ret;
-
-       /* check absolute ranges */
-       if (pan  < pdev->angle_range.pan_min  ||
-           pan  > pdev->angle_range.pan_max  ||
-           tilt < pdev->angle_range.tilt_min ||
-           tilt > pdev->angle_range.tilt_max)
-               return -ERANGE;
-
-       /* go to relative range, check again */
-       pan  -= pdev->pan_angle;
-       tilt -= pdev->tilt_angle;
-       /* angles are specified in degrees * 100, thus the limit = 36000 */
-       if (pan < -36000 || pan > 36000 || tilt < -36000 || tilt > 36000)
-               return -ERANGE;
-
-       ret = _pwc_mpt_set_angle(pdev, pan, tilt);
-       if (ret >= 0) {
-               pdev->pan_angle  += pan;
-               pdev->tilt_angle += tilt;
-       }
-       if (ret == -EPIPE) /* stall -> out of range */
-               ret = -ERANGE;
-       return ret;
-}
-
-static int pwc_mpt_get_status(struct pwc_device *pdev, struct pwc_mpt_status *status)
+static inline int pwc_mpt_get_status(struct pwc_device *pdev, struct pwc_mpt_status *status)
 {
        int ret;
        unsigned char buf[5];
@@ -1191,26 +1151,6 @@ int pwc_get_cmos_sensor(struct pwc_device *pdev, int *sensor)
  /* End of Add-Ons                                    */
  /* ************************************************* */
 
-/* Linux 2.5.something and 2.6 pass direct pointers to arguments of
-   ioctl() calls. With 2.4, you have to do tedious copy_from_user()
-   and copy_to_user() calls. With these macros we circumvent this,
-   and let me maintain only one source file. The functionality is
-   exactly the same otherwise.
- */
-
-
-/* define local variable for arg */
-#define ARG_DEF(ARG_type, ARG_name)\
-       ARG_type *ARG_name = arg;
-/* copy arg to local variable */
-#define ARG_IN(ARG_name) /* nothing */
-/* argument itself (referenced) */
-#define ARGR(ARG_name) (*ARG_name)
-/* argument address */
-#define ARGA(ARG_name) ARG_name
-/* copy local variable to arg */
-#define ARG_OUT(ARG_name) /* nothing */
-
 
 int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 {
@@ -1240,243 +1180,206 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 
        case VIDIOCPWCSCQUAL:
        {
-               ARG_DEF(int, qual)
+               int *qual = arg;
 
-               ARG_IN(qual)
-               if (ARGR(qual) < 0 || ARGR(qual) > 3)
+               if (*qual < 0 || *qual > 3)
                        ret = -EINVAL;
                else
-                       ret = pwc_try_video_mode(pdev, pdev->view.x, pdev->view.y, pdev->vframes, ARGR(qual), pdev->vsnapshot);
+                       ret = pwc_try_video_mode(pdev, pdev->view.x, pdev->view.y, pdev->vframes, *qual, pdev->vsnapshot);
                if (ret >= 0)
-                       pdev->vcompression = ARGR(qual);
+                       pdev->vcompression = *qual;
                break;
        }
 
        case VIDIOCPWCGCQUAL:
        {
-               ARG_DEF(int, qual)
-
-               ARGR(qual) = pdev->vcompression;
-               ARG_OUT(qual)
+               int *qual = arg;
+               *qual = pdev->vcompression;
                break;
        }
 
        case VIDIOCPWCPROBE:
        {
-               ARG_DEF(struct pwc_probe, probe)
-
-               strcpy(ARGR(probe).name, pdev->vdev->name);
-               ARGR(probe).type = pdev->type;
-               ARG_OUT(probe)
+               struct pwc_probe *probe = arg;
+               strcpy(probe->name, pdev->vdev->name);
+               probe->type = pdev->type;
                break;
        }
 
        case VIDIOCPWCGSERIAL:
        {
-               ARG_DEF(struct pwc_serial, serial)
-
-               strcpy(ARGR(serial).serial, pdev->serial);
-               ARG_OUT(serial)
+               struct pwc_serial *serial = arg;
+               strcpy(serial->serial, pdev->serial);
                break;
        }
 
        case VIDIOCPWCSAGC:
        {
-               ARG_DEF(int, agc)
-
-               ARG_IN(agc)
-               if (pwc_set_agc(pdev, ARGR(agc) < 0 ? 1 : 0, ARGR(agc)))
+               int *agc = arg;
+               if (pwc_set_agc(pdev, *agc < 0 ? 1 : 0, *agc))
                        ret = -EINVAL;
                break;
        }
 
        case VIDIOCPWCGAGC:
        {
-               ARG_DEF(int, agc)
+               int *agc = arg;
 
-               if (pwc_get_agc(pdev, ARGA(agc)))
+               if (pwc_get_agc(pdev, agc))
                        ret = -EINVAL;
-               ARG_OUT(agc)
                break;
        }
 
        case VIDIOCPWCSSHUTTER:
        {
-               ARG_DEF(int, shutter_speed)
-
-               ARG_IN(shutter_speed)
-               ret = pwc_set_shutter_speed(pdev, ARGR(shutter_speed) < 0 ? 1 : 0, ARGR(shutter_speed));
+               int *shutter_speed = arg;
+               ret = pwc_set_shutter_speed(pdev, *shutter_speed < 0 ? 1 : 0, *shutter_speed);
                break;
        }
 
        case VIDIOCPWCSAWB:
        {
-               ARG_DEF(struct pwc_whitebalance, wb)
+               struct pwc_whitebalance *wb = arg;
 
-               ARG_IN(wb)
-               ret = pwc_set_awb(pdev, ARGR(wb).mode);
-               if (ret >= 0 && ARGR(wb).mode == PWC_WB_MANUAL) {
-                       pwc_set_red_gain(pdev, ARGR(wb).manual_red);
-                       pwc_set_blue_gain(pdev, ARGR(wb).manual_blue);
+               ret = pwc_set_awb(pdev, wb->mode);
+               if (ret >= 0 && wb->mode == PWC_WB_MANUAL) {
+                       pwc_set_red_gain(pdev, wb->manual_red);
+                       pwc_set_blue_gain(pdev, wb->manual_blue);
                }
                break;
        }
 
        case VIDIOCPWCGAWB:
        {
-               ARG_DEF(struct pwc_whitebalance, wb)
+               struct pwc_whitebalance *wb = arg;
 
-               memset(ARGA(wb), 0, sizeof(struct pwc_whitebalance));
-               ARGR(wb).mode = pwc_get_awb(pdev);
-               if (ARGR(wb).mode < 0)
+               memset(wb, 0, sizeof(struct pwc_whitebalance));
+               wb->mode = pwc_get_awb(pdev);
+               if (wb->mode < 0)
                        ret = -EINVAL;
                else {
-                       if (ARGR(wb).mode == PWC_WB_MANUAL) {
-                               ret = pwc_get_red_gain(pdev, &ARGR(wb).manual_red);
+                       if (wb->mode == PWC_WB_MANUAL) {
+                               ret = pwc_get_red_gain(pdev, &wb->manual_red);
                                if (ret < 0)
                                        break;
-                               ret = pwc_get_blue_gain(pdev, &ARGR(wb).manual_blue);
+                               ret = pwc_get_blue_gain(pdev, &wb->manual_blue);
                                if (ret < 0)
                                        break;
                        }
-                       if (ARGR(wb).mode == PWC_WB_AUTO) {
-                               ret = pwc_read_red_gain(pdev, &ARGR(wb).read_red);
+                       if (wb->mode == PWC_WB_AUTO) {
+                               ret = pwc_read_red_gain(pdev, &wb->read_red);
                                if (ret < 0)
                                        break;
-                               ret =pwc_read_blue_gain(pdev, &ARGR(wb).read_blue);
+                               ret = pwc_read_blue_gain(pdev, &wb->read_blue);
                                if (ret < 0)
                                        break;
                        }
                }
-               ARG_OUT(wb)
                break;
        }
 
        case VIDIOCPWCSAWBSPEED:
        {
-               ARG_DEF(struct pwc_wb_speed, wbs)
+               struct pwc_wb_speed *wbs = arg;
 
-               if (ARGR(wbs).control_speed > 0) {
-                       ret = pwc_set_wb_speed(pdev, ARGR(wbs).control_speed);
+               if (wbs->control_speed > 0) {
+                       ret = pwc_set_wb_speed(pdev, wbs->control_speed);
                }
-               if (ARGR(wbs).control_delay > 0) {
-                       ret = pwc_set_wb_delay(pdev, ARGR(wbs).control_delay);
+               if (wbs->control_delay > 0) {
+                       ret = pwc_set_wb_delay(pdev, wbs->control_delay);
                }
                break;
        }
 
        case VIDIOCPWCGAWBSPEED:
        {
-               ARG_DEF(struct pwc_wb_speed, wbs)
+               struct pwc_wb_speed *wbs = arg;
 
-               ret = pwc_get_wb_speed(pdev, &ARGR(wbs).control_speed);
+               ret = pwc_get_wb_speed(pdev, &wbs->control_speed);
                if (ret < 0)
                        break;
-               ret = pwc_get_wb_delay(pdev, &ARGR(wbs).control_delay);
+               ret = pwc_get_wb_delay(pdev, &wbs->control_delay);
                if (ret < 0)
                        break;
-               ARG_OUT(wbs)
                break;
        }
 
        case VIDIOCPWCSLED:
        {
-               ARG_DEF(struct pwc_leds, leds)
-
-               ARG_IN(leds)
-               ret = pwc_set_leds(pdev, ARGR(leds).led_on, ARGR(leds).led_off);
-               break;
+               struct pwc_leds *leds = arg;
+               ret = pwc_set_leds(pdev, leds->led_on, leds->led_off);
+               break;
        }
 
 
        case VIDIOCPWCGLED:
        {
-               ARG_DEF(struct pwc_leds, leds)
-
-               ret = pwc_get_leds(pdev, &ARGR(leds).led_on, &ARGR(leds).led_off);
-               ARG_OUT(leds)
+               struct pwc_leds *leds = arg;
+               ret = pwc_get_leds(pdev, &leds->led_on, &leds->led_off);
                break;
        }
 
        case VIDIOCPWCSCONTOUR:
        {
-               ARG_DEF(int, contour)
-
-               ARG_IN(contour)
-               ret = pwc_set_contour(pdev, ARGR(contour));
+               int *contour = arg;
+               ret = pwc_set_contour(pdev, *contour);
                break;
        }
 
        case VIDIOCPWCGCONTOUR:
        {
-               ARG_DEF(int, contour)
-
-               ret = pwc_get_contour(pdev, ARGA(contour));
-               ARG_OUT(contour)
+               int *contour = arg;
+               ret = pwc_get_contour(pdev, contour);
                break;
        }
 
        case VIDIOCPWCSBACKLIGHT:
        {
-               ARG_DEF(int, backlight)
-
-               ARG_IN(backlight)
-               ret = pwc_set_backlight(pdev, ARGR(backlight));
+               int *backlight = arg;
+               ret = pwc_set_backlight(pdev, *backlight);
                break;
        }
 
        case VIDIOCPWCGBACKLIGHT:
        {
-               ARG_DEF(int, backlight)
-
-               ret = pwc_get_backlight(pdev, ARGA(backlight));
-               ARG_OUT(backlight)
+               int *backlight = arg;
+               ret = pwc_get_backlight(pdev, backlight);
                break;
        }
 
        case VIDIOCPWCSFLICKER:
        {
-               ARG_DEF(int, flicker)
-
-               ARG_IN(flicker)
-               ret = pwc_set_flicker(pdev, ARGR(flicker));
+               int *flicker = arg;
+               ret = pwc_set_flicker(pdev, *flicker);
                break;
        }
 
        case VIDIOCPWCGFLICKER:
        {
-               ARG_DEF(int, flicker)
-
-               ret = pwc_get_flicker(pdev, ARGA(flicker));
-               ARG_OUT(flicker)
+               int *flicker = arg;
+               ret = pwc_get_flicker(pdev, flicker);
                break;
        }
 
        case VIDIOCPWCSDYNNOISE:
        {
-               ARG_DEF(int, dynnoise)
-
-               ARG_IN(dynnoise)
-               ret = pwc_set_dynamic_noise(pdev, ARGR(dynnoise));
+               int *dynnoise = arg;
+               ret = pwc_set_dynamic_noise(pdev, *dynnoise);
                break;
        }
 
        case VIDIOCPWCGDYNNOISE:
        {
-               ARG_DEF(int, dynnoise)
-
-               ret = pwc_get_dynamic_noise(pdev, ARGA(dynnoise));
-               ARG_OUT(dynnoise);
+               int *dynnoise = arg;
+               ret = pwc_get_dynamic_noise(pdev, dynnoise);
                break;
        }
 
        case VIDIOCPWCGREALSIZE:
        {
-               ARG_DEF(struct pwc_imagesize, size)
-
-               ARGR(size).width = pdev->image.x;
-               ARGR(size).height = pdev->image.y;
-               ARG_OUT(size)
+               struct pwc_imagesize *size = arg;
+               size->width = pdev->image.x;
+               size->height = pdev->image.y;
                break;
        }
 
@@ -1484,10 +1387,14 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
        {
                if (pdev->features & FEATURE_MOTOR_PANTILT)
                {
-                       ARG_DEF(int, flags)
+                       int *flags = arg;
 
-                       ARG_IN(flags)
-                       ret = pwc_mpt_reset(pdev, ARGR(flags));
+                       ret = pwc_mpt_reset(pdev, *flags);
+                       if (ret >= 0)
+                       {
+                               pdev->pan_angle = 0;
+                               pdev->tilt_angle = 0;
+                       }
                }
                else
                {
@@ -1500,10 +1407,8 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
        {
                if (pdev->features & FEATURE_MOTOR_PANTILT)
                {
-                       ARG_DEF(struct pwc_mpt_range, range)
-
-                       ARGR(range) = pdev->angle_range;
-                       ARG_OUT(range)
+                       struct pwc_mpt_range *range = arg;
+                       *range = pdev->angle_range;
                }
                else
                {
@@ -1518,23 +1423,48 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 
                if (pdev->features & FEATURE_MOTOR_PANTILT)
                {
-                       ARG_DEF(struct pwc_mpt_angles, angles)
-
-                       ARG_IN(angles)
+                       struct pwc_mpt_angles *angles = arg;
                        /* The camera can only set relative angles, so
                           do some calculations when getting an absolute angle .
                         */
-                       if (ARGR(angles).absolute)
+                       if (angles->absolute)
                        {
-                               new_pan  = ARGR(angles).pan;
-                               new_tilt = ARGR(angles).tilt;
+                               new_pan  = angles->pan;
+                               new_tilt = angles->tilt;
                        }
                        else
                        {
-                               new_pan  = pdev->pan_angle  + ARGR(angles).pan;
-                               new_tilt = pdev->tilt_angle + ARGR(angles).tilt;
+                               new_pan  = pdev->pan_angle  + angles->pan;
+                               new_tilt = pdev->tilt_angle + angles->tilt;
+                       }
+                       /* check absolute ranges */
+                       if (new_pan  < pdev->angle_range.pan_min  ||
+                           new_pan  > pdev->angle_range.pan_max  ||
+                           new_tilt < pdev->angle_range.tilt_min ||
+                           new_tilt > pdev->angle_range.tilt_max)
+                       {
+                               ret = -ERANGE;
+                       }
+                       else
+                       {
+                               /* go to relative range, check again */
+                               new_pan  -= pdev->pan_angle;
+                               new_tilt -= pdev->tilt_angle;
+                               /* angles are specified in degrees * 100, thus the limit = 36000 */
+                               if (new_pan < -36000 || new_pan > 36000 || new_tilt < -36000 || new_tilt > 36000)
+                                       ret = -ERANGE;
+                       }
+                       if (ret == 0) /* no errors so far */
+                       {
+                               ret = pwc_mpt_set_angle(pdev, new_pan, new_tilt);
+                               if (ret >= 0)
+                               {
+                                       pdev->pan_angle  += new_pan;
+                                       pdev->tilt_angle += new_tilt;
+                               }
+                               if (ret == -EPIPE) /* stall -> out of range */
+                                       ret = -ERANGE;
                        }
-                       ret = pwc_mpt_set_angle(pdev, new_pan, new_tilt);
                }
                else
                {
@@ -1548,12 +1478,11 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 
                if (pdev->features & FEATURE_MOTOR_PANTILT)
                {
-                       ARG_DEF(struct pwc_mpt_angles, angles)
+                       struct pwc_mpt_angles *angles = arg;
 
-                       ARGR(angles).absolute = 1;
-                       ARGR(angles).pan  = pdev->pan_angle;
-                       ARGR(angles).tilt = pdev->tilt_angle;
-                       ARG_OUT(angles)
+                       angles->absolute = 1;
+                       angles->pan  = pdev->pan_angle;
+                       angles->tilt = pdev->tilt_angle;
                }
                else
                {
@@ -1566,10 +1495,8 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
        {
                if (pdev->features & FEATURE_MOTOR_PANTILT)
                {
-                       ARG_DEF(struct pwc_mpt_status, status)
-
-                       ret = pwc_mpt_get_status(pdev, ARGA(status));
-                       ARG_OUT(status)
+                       struct pwc_mpt_status *status = arg;
+                       ret = pwc_mpt_get_status(pdev, status);
                }
                else
                {
@@ -1580,24 +1507,22 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 
        case VIDIOCPWCGVIDCMD:
        {
-               ARG_DEF(struct pwc_video_command, cmd);
-
-               ARGR(cmd).type = pdev->type;
-               ARGR(cmd).release = pdev->release;
-               ARGR(cmd).command_len = pdev->cmd_len;
-               memcpy(&ARGR(cmd).command_buf, pdev->cmd_buf, pdev->cmd_len);
-               ARGR(cmd).bandlength = pdev->vbandlength;
-               ARGR(cmd).frame_size = pdev->frame_size;
-               ARG_OUT(cmd)
+               struct pwc_video_command *cmd = arg;
+
+               cmd->type = pdev->type;
+               cmd->release = pdev->release;
+               cmd->command_len = pdev->cmd_len;
+               memcpy(&cmd->command_buf, pdev->cmd_buf, pdev->cmd_len);
+               cmd->bandlength = pdev->vbandlength;
+               cmd->frame_size = pdev->frame_size;
                break;
        }
        /*
        case VIDIOCPWCGVIDTABLE:
        {
-               ARG_DEF(struct pwc_table_init_buffer, table);
-               ARGR(table).len = pdev->cmd_len;
-               memcpy(&ARGR(table).buffer, pdev->decompress_data, pdev->decompressor->table_size);
-               ARG_OUT(table)
+               struct pwc_table_init_buffer *table = arg;
+               table->len = pdev->cmd_len;
+               memcpy(&table->buffer, pdev->decompress_data, pdev->decompressor->table_size);
                break;
        }
        */
@@ -1613,4 +1538,4 @@ int pwc_ioctl(struct pwc_device *pdev, unsigned int cmd, void *arg)
 }
 
 
-/* vim: set cinoptions= formatoptions=croql cindent shiftwidth=8 tabstop=8: */
+