[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#677727: linux-image-3.4-trunk-686-pae: lirc / devinput remote control, only avery second keypress is recognised vs. sids 3.2 kernel where everything is ok



On Sat, 2012-06-16 at 16:26 +0200, Michael Schmitt wrote:
> Package: linux-2.6
> Version: 3.4.1-1~experimental.1
> Severity: normal
> 
> Dear Maintainers,
> 
> with the current kernel from experimental only every second keypress is
> recognized on my ir remote control. Reboot to kernel 3.2 from sid, all back to
> normal.
> I have no idea how the kernel could be responsible there... ok, a weird bug in
> the responsible kernel module for the remote, but somehow I doubt that.
> 
> If you need any lirc configs or logfiles give me a note, as right now I don't
> think my lirc config does help much here.

It looks like you're using the mceusb driver, which hasn't been changed
between these versions.  So this is presumably due to a change in the
core remote control code.

Could you test the attached patch, following the instructions at
<http://kernel-handbook.alioth.debian.org/ch-common-tasks.html#s-common-official>?

Ben.

-- 
Ben Hutchings
We get into the habit of living before acquiring the habit of thinking.
                                                              - Albert Camus
From 81c813bb7eba803f619e617f325c3820972d3d52 Mon Sep 17 00:00:00 2001
From: Ben Hutchings <ben@decadent.org.uk>
Date: Sat, 16 Jun 2012 17:32:08 +0100
Subject: [PATCH] Revert "[media] rc/ir-raw: use kfifo_rec_ptr_1 instead of
 kfifo"

This reverts commit df1caa587aa526f73d07abf85ebab88d0fd754ec.
---
 drivers/media/rc/rc-core-priv.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/rc/rc-core-priv.h b/drivers/media/rc/rc-core-priv.h
index 96f0a8b..b72f858 100644
--- a/drivers/media/rc/rc-core-priv.h
+++ b/drivers/media/rc/rc-core-priv.h
@@ -35,7 +35,7 @@ struct ir_raw_event_ctrl {
 	struct list_head		list;		/* to keep track of raw clients */
 	struct task_struct		*thread;
 	spinlock_t			lock;
-	struct kfifo_rec_ptr_1		kfifo;		/* fifo for the pulse/space durations */
+	struct kfifo			kfifo;		/* fifo for the pulse/space durations */
 	ktime_t				last_event;	/* when last event occurred */
 	enum raw_event_type		last_type;	/* last event type */
 	struct rc_dev			*dev;		/* pointer to the parent rc_dev */
-- 
1.7.10

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: