| Message ID | 20180918004732.9875-1-dmitry.torokhov@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [01/20] Input: iforce - remove "being used" silliness | expand |
On 18.09.18 02:47, Dmitry Torokhov wrote: > The kernel is supposed to handle multiple devices, static flags > in packet handling code will never work. > > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> > --- > > This is a random assortment of iforce patches that I made a few weeks back. > > Tim, I do not have hardware, so I was bound to screw it up, but if you > have some time I'd appreciate if you try them out (and if I indeed broke > things if you could identify issues that would be even more awesome). > > Thanks! > Hello Dmitry, I tested those patches and I didn't find any obvious issues. The basic functions do work (i.e. buttons and axes, don't have a HAT so I can't test that), force feedback seems to work to the extent it was before (I only have fftest though, no games that support force feedback). I'll go through a few more applications and see if anything not obvious is broken. Unfortunately, I only have that one wheel and I can only test USB connections at the moment (unless I find a proper serial adaptor, but I'm not sure if that would even work). Are those patches planned to go into 4.19 or are they intended to be merged in the next development cycle?
On Wed, Sep 19, 2018 at 04:51:26PM +0200, Tim Schumacher wrote: > On 18.09.18 02:47, Dmitry Torokhov wrote: > > The kernel is supposed to handle multiple devices, static flags > > in packet handling code will never work. > > > > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> > > --- > > > > This is a random assortment of iforce patches that I made a few weeks back. > > > > Tim, I do not have hardware, so I was bound to screw it up, but if you > > have some time I'd appreciate if you try them out (and if I indeed broke > > things if you could identify issues that would be even more awesome). > > > > Thanks! > > > Hello Dmitry, > > I tested those patches and I didn't find any obvious issues. The basic functions > do work (i.e. buttons and axes, don't have a HAT so I can't test that), force > feedback seems to work to the extent it was before (I only have fftest though, > no games that support force feedback). I'll go through a few more applications > and see if anything not obvious is broken. > Thank you for taking a look. > Unfortunately, I only have that one wheel and I can only test USB connections > at the moment (unless I find a proper serial adaptor, but I'm not sure if that > would even work). > > Are those patches planned to go into 4.19 or are they intended to be merged in > the next development cycle? Definitely not 4.19. Could be 4.20 or 4.21... Thanks.
On 19.09.18 19:10, Dmitry Torokhov wrote: > On Wed, Sep 19, 2018 at 04:51:26PM +0200, Tim Schumacher wrote: >> On 18.09.18 02:47, Dmitry Torokhov wrote: >>> The kernel is supposed to handle multiple devices, static flags >>> in packet handling code will never work. >>> >>> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> >>> --- >>> >>> This is a random assortment of iforce patches that I made a few weeks back. >>> >>> Tim, I do not have hardware, so I was bound to screw it up, but if you >>> have some time I'd appreciate if you try them out (and if I indeed broke >>> things if you could identify issues that would be even more awesome). >>> >>> Thanks! >>> >> Hello Dmitry, >> >> I tested those patches and I didn't find any obvious issues. The basic functions >> do work (i.e. buttons and axes, don't have a HAT so I can't test that), force >> feedback seems to work to the extent it was before (I only have fftest though, >> no games that support force feedback). I'll go through a few more applications >> and see if anything not obvious is broken. >> > > Thank you for taking a look. > >> Unfortunately, I only have that one wheel and I can only test USB connections >> at the moment (unless I find a proper serial adaptor, but I'm not sure if that >> would even work). >> >> Are those patches planned to go into 4.19 or are they intended to be merged in >> the next development cycle? > > Definitely not 4.19. Could be 4.20 or 4.21... > > Thanks. > Hello Dmitry, as a followup to the last E-Mail I sent about this topic, the complete chain of patches has been in my personal tree since September of 2018 and there have been no issues so far as well. I tested a few more games and utilities, and I was able to test with multiple wheels simultaneously as well. In the end it's still your decision, but imo those patches should be fine to pull in. Tim
On Wed, Jun 12, 2019 at 04:44:00PM +0200, Tim Schumacher wrote: > On 19.09.18 19:10, Dmitry Torokhov wrote: > > On Wed, Sep 19, 2018 at 04:51:26PM +0200, Tim Schumacher wrote: > >> On 18.09.18 02:47, Dmitry Torokhov wrote: > >>> The kernel is supposed to handle multiple devices, static flags > >>> in packet handling code will never work. > >>> > >>> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> > >>> --- > >>> > >>> This is a random assortment of iforce patches that I made a few weeks back. > >>> > >>> Tim, I do not have hardware, so I was bound to screw it up, but if you > >>> have some time I'd appreciate if you try them out (and if I indeed broke > >>> things if you could identify issues that would be even more awesome). > >>> > >>> Thanks! > >>> > >> Hello Dmitry, > >> > >> I tested those patches and I didn't find any obvious issues. The basic functions > >> do work (i.e. buttons and axes, don't have a HAT so I can't test that), force > >> feedback seems to work to the extent it was before (I only have fftest though, > >> no games that support force feedback). I'll go through a few more applications > >> and see if anything not obvious is broken. > >> > > > > Thank you for taking a look. > > > >> Unfortunately, I only have that one wheel and I can only test USB connections > >> at the moment (unless I find a proper serial adaptor, but I'm not sure if that > >> would even work). > >> > >> Are those patches planned to go into 4.19 or are they intended to be merged in > >> the next development cycle? > > > > Definitely not 4.19. Could be 4.20 or 4.21... > > > > Thanks. > > > > Hello Dmitry, > > as a followup to the last E-Mail I sent about this topic, the complete chain > of patches has been in my personal tree since September of 2018 and there > have been no issues so far as well. I tested a few more games and utilities, > and I was able to test with multiple wheels simultaneously as well. > > In the end it's still your decision, but imo those patches should be fine Thank you Tim, I'll put your name down as Tested-by then.
diff --git a/drivers/input/joystick/iforce/iforce-packets.c b/drivers/input/joystick/iforce/iforce-packets.c index c10169f4554e..91893c751524 100644 --- a/drivers/input/joystick/iforce/iforce-packets.c +++ b/drivers/input/joystick/iforce/iforce-packets.c @@ -149,12 +149,6 @@ void iforce_process_packet(struct iforce *iforce, u16 cmd, unsigned char *data) { struct input_dev *dev = iforce->dev; int i; - static int being_used = 0; - - if (being_used) - dev_warn(&iforce->dev->dev, - "re-entrant call to iforce_process %d\n", being_used); - being_used++; #ifdef CONFIG_JOYSTICK_IFORCE_232 if (HI(iforce->expect_packet) == HI(cmd)) { @@ -165,10 +159,8 @@ void iforce_process_packet(struct iforce *iforce, u16 cmd, unsigned char *data) #endif wake_up(&iforce->wait); - if (!iforce->type) { - being_used--; + if (!iforce->type) return; - } switch (HI(cmd)) { @@ -233,7 +225,6 @@ void iforce_process_packet(struct iforce *iforce, u16 cmd, unsigned char *data) } break; } - being_used--; } int iforce_get_id_packet(struct iforce *iforce, char *packet)
The kernel is supposed to handle multiple devices, static flags in packet handling code will never work. Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> --- This is a random assortment of iforce patches that I made a few weeks back. Tim, I do not have hardware, so I was bound to screw it up, but if you have some time I'd appreciate if you try them out (and if I indeed broke things if you could identify issues that would be even more awesome). Thanks! drivers/input/joystick/iforce/iforce-packets.c | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-)