Re: [PATCH] [media] rc5-decoder: BZ#85721: Fix RC5-SZ decoding

2014-10-30 Thread David Härdeman 

On 2014-10-30 10:54, Mauro Carvalho Chehab wrote:

changeset e87b540be2dd broke RC5-SZ decoding, as it forgot to add
the extra bit check for the enabled protocols at the beginning of
the logic.

Signed-off-by: Mauro Carvalho Chehab 

Acked-by: David Härdeman 



diff --git a/drivers/media/rc/ir-rc5-decoder.c
b/drivers/media/rc/ir-rc5-decoder.c
index 2ef763928ca4..84fa6e9b59a1 100644
--- a/drivers/media/rc/ir-rc5-decoder.c
+++ b/drivers/media/rc/ir-rc5-decoder.c
@@ -53,7 +53,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct
ir_raw_event ev)
u32 scancode;
enum rc_type protocol;

-   if (!(dev->enabled_protocols & (RC_BIT_RC5 | RC_BIT_RC5X)))
+	if (!(dev->enabled_protocols & (RC_BIT_RC5 | RC_BIT_RC5X | 
RC_BIT_RC5_SZ)))

return 0;

if (!is_timing_event(ev)) {

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] [media] rc5-decoder: BZ#85721: Fix RC5-SZ decoding

2014-10-30 Thread Mauro Carvalho Chehab 
changeset e87b540be2dd broke RC5-SZ decoding, as it forgot to add
the extra bit check for the enabled protocols at the beginning of
the logic.

Signed-off-by: Mauro Carvalho Chehab 

diff --git a/drivers/media/rc/ir-rc5-decoder.c 
b/drivers/media/rc/ir-rc5-decoder.c
index 2ef763928ca4..84fa6e9b59a1 100644
--- a/drivers/media/rc/ir-rc5-decoder.c
+++ b/drivers/media/rc/ir-rc5-decoder.c
@@ -53,7 +53,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct 
ir_raw_event ev)
u32 scancode;
enum rc_type protocol;
 
-   if (!(dev->enabled_protocols & (RC_BIT_RC5 | RC_BIT_RC5X)))
+   if (!(dev->enabled_protocols & (RC_BIT_RC5 | RC_BIT_RC5X | 
RC_BIT_RC5_SZ)))
return 0;
 
if (!is_timing_event(ev)) {
-- 
1.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html