All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: "Holmberg, Hans" <hans.holmberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 0/2] GPIO joystick driver
Date: Sat, 7 Feb 2015 12:46:58 -0800	[thread overview]
Message-ID: <20150207204658.GA29015@dtor-ws> (raw)
In-Reply-To: <4B6D6D87A8FB62428BE0A3E38461A1AB18D6AF04-pww93C2UFcwu0RiL9chJVbfspsVTdybXVpNB7YpNyf8@public.gmane.org>

Hi Hans,

On Sat, Feb 07, 2015 at 07:35:46AM +0000, Holmberg, Hans wrote:
> Hi Dmitry,
> 
> > Why do we need this new driver when we have gpio-keys driver that can be
> > configured to emit ABS_* events?
> > 
> 
> As far as I can tell, there is no way to specify values for ABS-"keys" in the
> device tree binding. 

It may mot be present in device tree binding, but the driver does
support it, so I would rather extend the binding than have a brand new
driver.

> 
> While both the device tree binding and the gpio-keys driver could possibly be adapted to 
> support this, it makes sense to me to describe joysticks in a separate binding and keep
> the driver implementations apart.

Why? From the input core perspective there is no difference. It is just
a device that emits set of events.

> 
> In addition to this,  auto-repeat and the sys fs disable interface for individual keys 
> does not make sense for joysticks.

You do not need to use it when you describe the device (BTW autorepeat
for trigger might be a good thing).

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2015-02-07 20:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-06  9:32 [PATCH 0/2] GPIO joystick driver Hans Holmberg
     [not found] ` <1423215122-19947-1-git-send-email-hans.holmberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-02-06  9:32   ` [PATCH 1/2] Documentation: Device Tree binding information for gpio-joy driver Hans Holmberg
     [not found]     ` <1423215122-19947-2-git-send-email-hans.holmberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-02-06 10:53       ` Mark Rutland
2015-02-07  7:50         ` Holmberg, Hans
2015-02-06 15:31   ` [PATCH 0/2] GPIO joystick driver Dmitry Torokhov
     [not found]     ` <1AA6C46C-36D9-4243-9DC5-254A7202D339-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-02-07  7:35       ` Holmberg, Hans
     [not found]         ` <4B6D6D87A8FB62428BE0A3E38461A1AB18D6AF04-pww93C2UFcwu0RiL9chJVbfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2015-02-07 20:46           ` Dmitry Torokhov [this message]
2015-02-09 10:19             ` Holmberg, Hans
2015-02-11  8:49               ` Holmberg, Hans
2015-02-11 18:06                 ` 'Dmitry Torokhov'
2015-02-18  8:42                   ` Holmberg, Hans
2015-02-06  9:32 ` [PATCH 2/2] input: gpio-joy - " Hans Holmberg
     [not found]   ` <1423215122-19947-3-git-send-email-hans.holmberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-02-06 11:02     ` Mark Rutland
2015-02-07  7:53       ` Holmberg, Hans

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=20150207204658.GA29015@dtor-ws \
    --to=dmitry.torokhov-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=hans.holmberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.