V4L/DVB (13680): ir: use unsigned long instead of enum
authorMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 14 Dec 2009 16:53:37 +0000 (13:53 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Fri, 26 Feb 2010 18:10:24 +0000 (15:10 -0300)
When preparing the linux-next patches, I got those errors:

include/media/ir-core.h:29: warning: left shift count >= width of type
In file included from include/media/ir-common.h:29,
                 from drivers/media/video/ir-kbd-i2c.c:50:
drivers/media/video/ir-kbd-i2c.c: In function ‘ir_probe’:
drivers/media/video/ir-kbd-i2c.c:324: warning: left shift count >= width of type

Unfortunately, enum is 32 bits on i386. As we define IR_TYPE_OTHER as 1<<63,
it won't work on non 64 bits arch.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/IR/ir-functions.c
drivers/media/IR/ir-sysfs.c
drivers/media/dvb/dm1105/dm1105.c
drivers/media/video/bt8xx/bttv-input.c
drivers/media/video/cx88/cx88-input.c
drivers/media/video/em28xx/em28xx-input.c
drivers/media/video/ir-kbd-i2c.c
drivers/media/video/saa7134/saa7134-input.c
include/media/ir-common.h
include/media/ir-core.h
include/media/ir-kbd-i2c.h

index b501ac9d401d7fe18a50d8506aced742cf7f43d0..ab06919ad5fc03c68329307ad4e97b9367fbb703 100644 (file)
@@ -52,7 +52,7 @@ static void ir_input_key_event(struct input_dev *dev, struct ir_input_state *ir)
 /* -------------------------------------------------------------------------- */
 
 int ir_input_init(struct input_dev *dev, struct ir_input_state *ir,
-                 const enum ir_type ir_type)
+                 const u64 ir_type)
 {
        ir->ir_type = ir_type;
 
index d73589ad55e6acf5b8f9103d5b6f8c63df71cb61..d67c11d9921f62dcaddc4a5dc31af93af65316aa 100644 (file)
@@ -39,9 +39,9 @@ static ssize_t show_protocol(struct device *d,
 {
        char *s;
        struct ir_input_dev *ir_dev = dev_get_drvdata(d);
-       enum ir_type ir_type = ir_dev->rc_tab.ir_type;
+       u64 ir_type = ir_dev->rc_tab.ir_type;
 
-       IR_dprintk(1, "Current protocol is %ld\n", ir_type);
+       IR_dprintk(1, "Current protocol is %lld\n", (long long)ir_type);
 
        /* FIXME: doesn't support multiple protocols at the same time */
        if (ir_type == IR_TYPE_UNKNOWN)
@@ -77,7 +77,7 @@ static ssize_t store_protocol(struct device *d,
                              size_t len)
 {
        struct ir_input_dev *ir_dev = dev_get_drvdata(d);
-       enum ir_type ir_type = IR_TYPE_UNKNOWN;
+       u64 ir_type = IR_TYPE_UNKNOWN;
        int rc = -EINVAL;
        unsigned long flags;
        char *buf;
@@ -92,7 +92,8 @@ static ssize_t store_protocol(struct device *d,
                ir_type = IR_TYPE_NEC;
 
        if (ir_type == IR_TYPE_UNKNOWN) {
-               IR_dprintk(1, "Error setting protocol to %ld\n", ir_type);
+               IR_dprintk(1, "Error setting protocol to %lld\n",
+                          (long long)ir_type);
                return -EINVAL;
        }
 
@@ -101,7 +102,8 @@ static ssize_t store_protocol(struct device *d,
                                                    ir_type);
 
        if (rc < 0) {
-               IR_dprintk(1, "Error setting protocol to %ld\n", ir_type);
+               IR_dprintk(1, "Error setting protocol to %lld\n",
+                          (long long)ir_type);
                return -EINVAL;
        }
 
@@ -109,7 +111,8 @@ static ssize_t store_protocol(struct device *d,
        ir_dev->rc_tab.ir_type = ir_type;
        spin_unlock_irqrestore(&ir_dev->rc_tab.lock, flags);
 
-       IR_dprintk(1, "Current protocol is %ld\n", ir_type);
+       IR_dprintk(1, "Current protocol is %lld\n",
+                  (long long)ir_type);
 
        return len;
 }
index 414d3b2444a2874a81d1650ce4c03a391be2e5dd..aadf803c261c523d82a7f7c8d9dcc784f5f267d2 100644 (file)
@@ -578,7 +578,7 @@ int __devinit dm1105_ir_init(struct dm1105dvb *dm1105)
 {
        struct input_dev *input_dev;
        struct ir_scancode_table *ir_codes = &ir_codes_dm1105_nec_table;
-       enum ir_type ir_type = IR_TYPE_OTHER;
+       u64 ir_type = IR_TYPE_OTHER;
        int err = -ENOMEM;
 
        input_dev = input_allocate_device();
index f8053fd88346d28f96a28b6e0c0083d2a24af34c..b320dbd635aab0844b69ad2c2a99ed0e5b8b52b7 100644 (file)
@@ -247,7 +247,7 @@ int bttv_input_init(struct bttv *btv)
        struct card_ir *ir;
        struct ir_scancode_table *ir_codes = NULL;
        struct input_dev *input_dev;
-       enum ir_type ir_type = IR_TYPE_OTHER;
+       u64 ir_type = IR_TYPE_OTHER;
        int err = -ENOMEM;
 
        if (!btv->has_remote)
index 49c07535e7548630bac5f0588dbdb0370f3c0cec..de180d4d5a2139c3a7c430e484724421df4e305d 100644 (file)
@@ -192,7 +192,7 @@ int cx88_ir_init(struct cx88_core *core, struct pci_dev *pci)
        struct cx88_IR *ir;
        struct input_dev *input_dev;
        struct ir_scancode_table *ir_codes = NULL;
-       enum ir_type ir_type = IR_TYPE_OTHER;
+       u64 ir_type = IR_TYPE_OTHER;
        int err = -ENOMEM;
 
        ir = kzalloc(sizeof(*ir), GFP_KERNEL);
index 674622372d4d9c7a053216c240304031c61cedbf..69dcf0cc1f1e4dc167bdf43f5a775b273b07ad6f 100644 (file)
@@ -340,7 +340,7 @@ static void em28xx_ir_stop(struct em28xx_IR *ir)
        cancel_delayed_work_sync(&ir->work);
 }
 
-int em28xx_ir_change_protocol(void *priv, enum ir_type ir_type)
+int em28xx_ir_change_protocol(void *priv, u64 ir_type)
 {
        int rc = 0;
        struct em28xx_IR *ir = priv;
index 4cd75a3c47d1a8b30adbe1788ebaeffefcbbb84b..094e21dbb14ffe2260429c4fb92d34bc6818f89f 100644 (file)
@@ -299,7 +299,7 @@ static int ir_probe(struct i2c_client *client, const struct i2c_device_id *id)
 {
        struct ir_scancode_table *ir_codes = NULL;
        const char *name = NULL;
-       enum ir_type ir_type = 0;
+       u64 ir_type = 0;
        struct IR_i2c *ir;
        struct input_dev *input_dev;
        struct i2c_adapter *adap = client->adapter;
index 71b4b01824aca2cb5f704eec7c45d8466fa9fe6b..a4eaf1b75d701f571b42cfadd1b66d7f2322cf07 100644 (file)
@@ -460,7 +460,7 @@ int saa7134_input_init1(struct saa7134_dev *dev)
        int polling      = 0;
        int rc5_gpio     = 0;
        int nec_gpio     = 0;
-       enum ir_type ir_type = IR_TYPE_OTHER;
+       u64 ir_type = IR_TYPE_OTHER;
        int err;
 
        if (dev->has_remote != SAA7134_REMOTE_GPIO)
index 1b43b772165a5631f13264be3ccd6b9324333336..015db75b42f6d5dda0416ef2dc82097b6b5fde21 100644 (file)
@@ -35,7 +35,7 @@
 
 struct ir_input_state {
        /* configuration */
-       enum ir_type       ir_type;
+       u64      ir_type;
 
        /* key info */
        u32                ir_key;      /* ir scancode */
@@ -84,7 +84,7 @@ struct card_ir {
 /* Routines from ir-functions.c */
 
 int ir_input_init(struct input_dev *dev, struct ir_input_state *ir,
-                  const enum ir_type ir_type);
+                  const u64 ir_type);
 void ir_input_nokey(struct input_dev *dev, struct ir_input_state *ir);
 void ir_input_keydown(struct input_dev *dev, struct ir_input_state *ir,
                      u32 ir_key);
index a6d07dede09f2b79b47962eb5c6c4c934d36776b..d96f25a08451d709ecb19030079ce7f5945a5a5f 100644 (file)
@@ -21,13 +21,11 @@ extern int ir_core_debug;
 #define IR_dprintk(level, fmt, arg...) if (ir_core_debug >= level) \
        printk(KERN_DEBUG "%s: " fmt , __func__, ## arg)
 
-enum ir_type {
-       IR_TYPE_UNKNOWN = 0,
-       IR_TYPE_RC5     = 1L << 0,      /* Philips RC5 protocol */
-       IR_TYPE_PD      = 1L << 1,      /* Pulse distance encoded IR */
-       IR_TYPE_NEC     = 1L << 2,
-       IR_TYPE_OTHER   = 1L << 63,
-};
+#define IR_TYPE_UNKNOWN        0
+#define IR_TYPE_RC5    (1  << 0)       /* Philips RC5 protocol */
+#define IR_TYPE_PD     (1  << 1)       /* Pulse distance encoded IR */
+#define IR_TYPE_NEC    (1  << 2)
+#define IR_TYPE_OTHER  (((u64)1) << 63l)
 
 struct ir_scancode {
        u16     scancode;
@@ -37,14 +35,14 @@ struct ir_scancode {
 struct ir_scancode_table {
        struct ir_scancode      *scan;
        int                     size;
-       enum                    ir_type ir_type;
+       u64             ir_type;
        spinlock_t              lock;
 };
 
 struct ir_dev_props {
        unsigned long allowed_protos;
        void            *priv;
-       int (*change_protocol)(void *priv, enum ir_type ir_type);
+       int (*change_protocol)(void *priv, u64 ir_type);
 };
 
 
index 45926e3559f350064a37cbce45f8dd07582f3afb..9142936603cc89fc29cf911e4f8b2bdb29a6cf3b 100644 (file)
@@ -36,7 +36,7 @@ enum ir_kbd_get_key_fn {
 struct IR_i2c_init_data {
        struct ir_scancode_table *ir_codes;
        const char             *name;
-       enum ir_type           type; /* IR_TYPE_RC5, IR_TYPE_PD, etc */
+       u64          type; /* IR_TYPE_RC5, IR_TYPE_PD, etc */
        /*
         * Specify either a function pointer or a value indicating one of
         * ir_kbd_i2c's internal get_key functions