V4L/DVB: IR/imon: minor change_protocol fixups
authorJarod Wilson <jarod@redhat.com>
Wed, 28 Apr 2010 17:37:29 +0000 (14:37 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 19 May 2010 15:58:26 +0000 (12:58 -0300)
This is a follow-up to my prior patch implementing ir-core's
change_protocol functionality in the imon driver, which eliminates
a false warning when change_protocol is called without a specific
protocol selected yet (i.e., still IR_TYPE_UNKNOWN). It also removes
some extraneous blank lines getting spewn into dmesg.

Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/IR/imon.c

index b65c31ab4a4f4f80887dcc9b19b52eb645a78ed0..f5fb08fba63748d043425135748a379fcc6f7341 100644 (file)
@@ -999,7 +999,7 @@ int imon_ir_change_protocol(void *priv, u64 ir_type)
        unsigned char ir_proto_packet[] = {
                0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x86 };
 
-       if (!(ir_type & ictx->props->allowed_protos))
+       if (ir_type && !(ir_type & ictx->props->allowed_protos))
                dev_warn(dev, "Looks like you're trying to use an IR protocol "
                         "this device does not support\n");
 
@@ -1014,12 +1014,11 @@ int imon_ir_change_protocol(void *priv, u64 ir_type)
                break;
        case IR_TYPE_UNKNOWN:
        case IR_TYPE_OTHER:
-               dev_dbg(dev, "Configuring IR receiver for iMON protocol");
-               if (pad_stabilize) {
-                       printk(KERN_CONT "\n");
+               dev_dbg(dev, "Configuring IR receiver for iMON protocol\n");
+               if (pad_stabilize)
                        pad_mouse = true;
-               else {
-                       printk(KERN_CONT " (without PAD stabilization)\n");
+               else {
+                       dev_dbg(dev, "PAD stabilize functionality disabled\n");
                        pad_mouse = false;
                }
                /* ir_proto_packet[0] = 0x00; // already the default */
@@ -1027,12 +1026,11 @@ int imon_ir_change_protocol(void *priv, u64 ir_type)
                break;
        default:
                dev_warn(dev, "Unsupported IR protocol specified, overriding "
-                        "to iMON IR protocol");
-               if (pad_stabilize) {
-                       printk(KERN_CONT "\n");
+                        "to iMON IR protocol\n");
+               if (pad_stabilize)
                        pad_mouse = true;
-               else {
-                       printk(KERN_CONT " (without PAD stabilization)\n");
+               else {
+                       dev_dbg(dev, "PAD stabilize functionality disabled\n");
                        pad_mouse = false;
                }
                /* ir_proto_packet[0] = 0x00; // already the default */