linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Dmitry Torokhov" <dmitry.torokhov@gmail.com>
To: "johann deneux" <johann.deneux@gmail.com>
Cc: "Jiri Slaby" <jirislaby@gmail.com>,
	"STenyaK (Bruno González)" <stenyak@gmail.com>,
	"Anssi Hannula" <anssi.hannula@gmail.com>,
	"Linux kernel mailing list" <linux-kernel@vger.kernel.org>,
	linux-input@atrey.karlin.mff.cuni.cz
Subject: Re: FF layer restrictions [Was: [PATCH 1/1] Input: add sensable phantom driver]
Date: Wed, 21 Mar 2007 15:22:39 -0400	[thread overview]
Message-ID: <d120d5000703211222j64fd22fbn1e06267790f8a569@mail.gmail.com> (raw)
In-Reply-To: <38b3b7c0703211202p29f7865cr34960c7ea708640e@mail.gmail.com>

On 3/21/07, johann deneux <johann.deneux@gmail.com> wrote:
> On 3/21/07, Jiri Slaby <jirislaby@gmail.com> wrote:
> > STenyaK (Bruno González) napsal(a):
> > > On 3/14/07, Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> > >> > I have a question: if the force is to be 3D, why only 3 possible
> > >> values?
> > >> > What would they be, 3 torques or 3 forces? In the case of car sims (ff
> > >> > steering wheels), only one axis of torque is usually used (except
> > >> for 6 dof
> > >> > platforms, as mentioned).
> > >> >
> > >>
> > >> I wonder if we could somehow extend or augment FF envelope se we could
> > >> specify a plane for the effect.. Then a vector could be represented by
> > >> a sum 3 constant effects in 3 separate planes and we could also use
> > >> spring and other effects as well.
> > >
> > > Ideally, afaik we should use:
> > > -3 values for translation force (linear force): x,y,z components of
> > > the force vector.
> > > -4 values for rotation force (torque): x,y,z,w components of the
> > > quaternion. You can also use euler angles (and i think there are
> > > another one or two notations), which is just 3 values, but i'm not
> > > sure it will be a correct decision (due to the gimbal lock problem,
> > > which may or may not be present in ff devices, dunno).
> >
> > So, the resolution is? Since I want no longer have out-of-kernel driver, I
> > need to know how to implement the phantom driver -- merge it `as is', i.e.
> > control through mmio or rewrite ff layer somehow, and in that case how?
> >
> > There seem to be only few possibilities:
> >
> > - new 3D effect, which will be problematic in any other future use, that may
> > need more than 3 axis. no matter if torques or vector is passed -- depending
> > on device and programmer (as I need to compute torques from forces in FP).
> > Maybe struct with 2x 3 axis is OK
> >
> > - "raw" effect, which may contain more axis, but this is ugly in Anssi's eyes
> >
> > - something else? (did I forget something)
> >
>
> I would suggest adding a new effect type (3d effect) and extending the
> union in struct ff_effect.
> Let me know if I'm too vague, I already suggested that solution but
> got no answer. I wonder if my mail got lost, nobody understood what I
> said, or if it's just a plain bad idea.
>

My concern with a new 3D effect is that it will be a very "simple"
effect with only constant force apllied. That might be enough for
phantom but may not be sufficient for future devices. If we add
ability to specify a "plane" for an effect we will be able to add
envelopes on top of more complex effects and get desired combined
effcet. I would not worry aboput extra memory needed on I-force like
devices because they just would not support additional "planes". What
about phantom? Would it have enough memory?

On the other hand if you guys (Anssi, Johann, Jiri...) decide that a
simple new 3d effect is the most efficient solution for now and will
be enough for a few years (till we get FF v2 API) I will merge it.

-- 
Dmitry

  reply	other threads:[~2007-03-21 19:22 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-07 11:36 [PATCH 1/1] Input: add sensable phantom driver Jiri Slaby
2007-03-07 14:47 ` Dmitry Torokhov
2007-03-07 16:38   ` Jiri Slaby
2007-03-07 16:50     ` Greg KH
2007-03-07 16:56     ` Dmitry Torokhov
2007-03-13 16:19       ` Jiri Slaby
     [not found]         ` <38b3b7c0703131450v2646e63fj2be4b9dda7f928c0@mail.gmail.com>
2007-03-13 22:16           ` FF layer restrictions [Was: [PATCH 1/1] Input: add sensable phantom driver] Jiri Slaby
2007-03-14 15:02             ` Dmitry Torokhov
2007-03-14 16:43               ` Jiri Slaby
2007-03-14 16:45                 ` Jiri Slaby
2007-03-14 18:04                 ` Anssi Hannula
2007-03-14 18:15                   ` Jiri Slaby
     [not found]                   ` <8e4ff20a0703141147n4b690ab8g4cc8138d1ecc94e1@mail.gmail.com>
2007-03-14 19:12                     ` STenyaK (Bruno González)
2007-03-14 19:13                     ` Dmitry Torokhov
2007-03-14 19:18                       ` STenyaK (Bruno González)
2007-03-15 20:51                         ` johann deneux
2007-03-15 21:06                           ` STenyaK (Bruno González)
2007-03-21 13:31                         ` Jiri Slaby
2007-03-21 13:32                           ` Jiri Slaby
2007-03-21 19:02                           ` johann deneux
2007-03-21 19:22                             ` Dmitry Torokhov [this message]
2007-03-21 20:04                               ` Jiri Slaby
2007-03-21 22:03                                 ` johann deneux
2007-03-22 15:50                                   ` Dmitry Torokhov
2007-03-27 12:20                                     ` Jiri Slaby
2007-03-27 18:36                                       ` johann deneux
2007-03-27 20:11                                         ` Jiri Slaby
2007-03-27 20:43                                           ` johann deneux
2007-03-27 20:51                                             ` Jiri Slaby
2007-03-27 21:34                                               ` johann deneux
2007-03-28  3:08                                                 ` Dmitry Torokhov
2007-03-28  9:28                                                   ` Jiri Slaby
2007-03-28 22:16                                                   ` Jiri Slaby
2007-03-28 22:22                                                     ` Jiri Slaby
2007-03-30 16:46                                                       ` Dmitry Torokhov
2007-03-30 19:11                                                 ` Anssi Hannula
2007-03-15 20:43                       ` johann deneux
2007-03-16 16:28               ` Pavel Machek
2007-03-17  7:28                 ` johann deneux

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d120d5000703211222j64fd22fbn1e06267790f8a569@mail.gmail.com \
    --to=dmitry.torokhov@gmail.com \
    --cc=anssi.hannula@gmail.com \
    --cc=jirislaby@gmail.com \
    --cc=johann.deneux@gmail.com \
    --cc=linux-input@atrey.karlin.mff.cuni.cz \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stenyak@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).