From: Zhang Rui <rui.zhang@intel.com>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Len Brown <lenb@kernel.org>,
Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>
Subject: Re: a question about lid input device
Date: Mon, 05 Jul 2010 09:09:00 +0800 [thread overview]
Message-ID: <1278292140.4537.6088.camel@rzhang1-desktop> (raw)
In-Reply-To: <20100704083553.GE32427@core.coreip.homeip.net>
On Sun, 2010-07-04 at 16:35 +0800, Dmitry Torokhov wrote:
> On Mon, Jun 21, 2010 at 09:56:37PM -0400, Len Brown wrote:
> > Dmitry,
> > Is there a reason not to simply add a "state" attribute
> > to a LID input device?
> >
>
> Len,
>
> There is no LED input device, there is an input device that happens to
> report EV_SW/SW_LID state. If we did that we'd need to add attributes
> for all types of switches, keys, current values of REL and ABS axis,
> min/max/fuzz/flat settings of ABS axis and so forth that input device
> might report. I do not think it is a good idea.
>
I agree.
How about adding a field in uevent file that reports the switch status?
thanks,
rui
> >
> > thanks,
> > Len Brown, Intel Open Source Technology Center
> >
> > On Sun, 13 Jun 2010, Henrique de Moraes Holschuh wrote:
> >
> > > On Sun, 13 Jun 2010, Zhang Rui wrote:
> > > > Then we update the lid switch status when a Lid notification comes.
> > > > right?
> > >
> > > Yes.
> > >
> > > > Then, IMO, userspace can get the lid status
> > > > via /sys/class/input/inputX/uevent, right?
> > >
> > > No, only through an IOCTL.
> > >
> > > > /sys/class/input/input1/uevent:EV==21
> > >
> > > That's an bitmap of al EV_EV it supports.
> > >
> > > > /sys/class/input/input1/uevent:SW==1
> > >
> > > That's an bitmap of al EV_SW it supports.
> > >
> > > > Lid is opened but SW is set, I tried to close/open the lid and found
> > > > that this bit never changes. is there something I misunderstand? can we
> > > > get the lid status in userspace?
> > >
> > > IOCTL(), only.
> > >
> > > Since nobody got a input-utils standard package (or added something to
> > > util-linux) yet to do that (AFAIK anyway), it is a MAJOR annoyance for
> > > shell scripts that want to query EV_SW state...
> > >
> >
>
next prev parent reply other threads:[~2010-07-05 1:07 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-13 6:53 a question about lid input device Zhang Rui
2010-06-14 2:01 ` Henrique de Moraes Holschuh
2010-06-22 1:56 ` Len Brown
2010-07-04 8:35 ` Dmitry Torokhov
2010-07-05 1:09 ` Zhang Rui [this message]
2010-07-05 1:28 ` Dmitry Torokhov
2010-07-05 1:47 ` Zhang Rui
2010-07-05 19:49 ` Henrique de Moraes Holschuh
2010-07-05 20:04 ` Paul Fox
2010-07-05 20:27 ` Dmitry Torokhov
2010-07-07 3:08 ` Zhang Rui
2010-07-07 5:36 ` Dmitry Torokhov
2010-07-07 15:14 ` Paul Fox
2010-07-07 16:25 ` Dmitry Torokhov
2010-07-07 16:51 ` Paul Fox
2010-07-07 17:01 ` Dmitry Torokhov
2010-07-07 17:08 ` Paul Fox
2010-07-07 17:19 ` Dmitry Torokhov
2010-09-29 1:12 ` Len Brown
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=1278292140.4537.6088.camel@rzhang1-desktop \
--to=rui.zhang@intel.com \
--cc=dmitry.torokhov@gmail.com \
--cc=hmh@hmh.eng.br \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox