Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 666a9ed8 authored by Jarod Wilson's avatar Jarod Wilson Committed by Mauro Carvalho Chehab
Browse files

V4L/DVB: IR/imon: minor change_protocol fixups 



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: default avatarJarod Wilson <jarod@redhat.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 96c1f996

drivers/media/IR/imon.c

+9 −11
Original line number Diff line number Diff line
@@ -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 */