media: rc: ir-sharp-decoder: add support for Denon variant of the protocol

PATCH New Heiner Kallweit 1 participant 1 message Oct 29, 2015

Subsystems (2)

Media (V4L/DVB) RC-core / lirc framework

lore

Share & tools

lore

Heiner Kallweit

author

2015-10-29 19:37 UTC #0 lore raw

PATCH 1/1 media: rc: ir-sharp-decoder: add support for Denon variant of the protocol

Denon also uses the Sharp protocol, however with different check bits.

It would have been also possible to add this as a separate protocol but this may not be worth the effort.

Successfully tested with a Denon RC-1002 remote control.

Signed-off-by Heiner Kallweit <[email protected]>

---
 drivers/media/rc/Kconfig            | 3 ++-
 drivers/media/rc/ir-sharp-decoder.c | 4 +++-
 2 files changed, 5 insertions(+), 2 deletions(-)

drivers/media/rc/Kconfig +2 -1

index b6e1311..bd4d685 100644
@@ -101,7 +101,8 @@ config IR_SHARP_DECODER
 
 	---help---
 	   Enable this option if you have an infrared remote control which
-	   uses the Sharp protocol, and you need software decoding support.
+	   uses the Sharp protocol (Sharp, Denon), and you need software
+	   decoding support.
 
 config IR_MCE_KBD_DECODER
 	tristate "Enable IR raw decoder for the MCE keyboard/mouse protocol"

drivers/media/rc/ir-sharp-decoder.c +3 -1

index b7acdba..1f33164 100644
@@ -118,7 +118,9 @@ static int ir_sharp_decode(struct rc_dev *dev, struct ir_raw_event ev)
 
 		if (data->count == SHARP_NBITS) {
 			/* exp,chk bits should be 1,0 */
-			if ((data->bits & 0x3) != 0x2)
+			if ((data->bits & 0x3) != 0x2 &&
+			/* DENON variant, both chk bits 0 */
+			    (data->bits & 0x3) != 0x0)
 				break;
 			data->state = STATE_ECHO_SPACE;
 		} else {

Signature · 1 line

2.6.2