Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-09-08 Thread David Härdeman
On Wed, Sep 08, 2010 at 10:16:13AM -0400, Jarod Wilson wrote: > On Wed, Sep 08, 2010 at 07:04:03AM -0700, Brian Rogers wrote: > > ir_dev->raw is also null. If I check these pointers before using > > them, and bail out if both are null, then I get a working lircd, but > > of course the file /sys/dev

Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-09-08 Thread Jarod Wilson
On Wed, Sep 08, 2010 at 07:04:03AM -0700, Brian Rogers wrote: > On 06/13/2010 01:29 PM, David Härdeman wrote: > >diff --git a/drivers/media/IR/ir-sysfs.c b/drivers/media/IR/ir-sysfs.c > >index daf33c1..7ae5662 100644 > >--- a/drivers/media/IR/ir-sysfs.c > >+++ b/drivers/media/IR/ir-sysfs.c > >@@ -

Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-09-08 Thread Brian Rogers
On 06/13/2010 01:29 PM, David Härdeman wrote: diff --git a/drivers/media/IR/ir-sysfs.c b/drivers/media/IR/ir-sysfs.c index daf33c1..7ae5662 100644 --- a/drivers/media/IR/ir-sysfs.c +++ b/drivers/media/IR/ir-sysfs.c @@ -33,122 +33,178 @@ static struct class ir_input_class = { }; /** - * show

Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-06-28 Thread Mauro Carvalho Chehab
Em 13-06-2010 17:29, David Härdeman escreveu: > With the current logic, each raw decoder needs to add a copy of the exact > same sysfs code. This is both unnecessary and also means that (re)loading > an IR driver after raw decoder modules have been loaded won't work as > expected. > > This patch m

Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-06-16 Thread Jarod Wilson
On Wed, Jun 16, 2010 at 4:05 PM, Jarod Wilson wrote: > On Sun, Jun 13, 2010 at 4:29 PM, David Härdeman wrote: >> With the current logic, each raw decoder needs to add a copy of the exact >> same sysfs code. This is both unnecessary and also means that (re)loading >> an IR driver after raw decoder

Re: [PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-06-16 Thread Jarod Wilson
On Sun, Jun 13, 2010 at 4:29 PM, David Härdeman wrote: > With the current logic, each raw decoder needs to add a copy of the exact > same sysfs code. This is both unnecessary and also means that (re)loading > an IR driver after raw decoder modules have been loaded won't work as > expected. > > Thi

[PATCH 1/2] ir-core: centralize sysfs raw decoder enabling/disabling

2010-06-13 Thread David Härdeman
With the current logic, each raw decoder needs to add a copy of the exact same sysfs code. This is both unnecessary and also means that (re)loading an IR driver after raw decoder modules have been loaded won't work as expected. This patch moves that logic into ir-raw-event and adds a single sysfs