Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN
@ 2013-10-22 16:37 Tom Rini
  2013-10-31 11:27 ` Tom Rini
  0 siblings, 1 reply; 5+ messages in thread
From: Tom Rini @ 2013-10-22 16:37 UTC (permalink / raw)
  To: openembedded-core

Since udev 174 udev has been running input_id as a built-in command and
setting this value in the environment for touchscreens.  Use this logic
to detect when to make a touchscreen0 symlink.

Signed-off-by: Tom Rini <trini@ti.com>
---
 meta/recipes-core/udev/udev/local.rules |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/udev/udev/local.rules b/meta/recipes-core/udev/udev/local.rules
index 72d92ef..d0b0216 100644
--- a/meta/recipes-core/udev/udev/local.rules
+++ b/meta/recipes-core/udev/udev/local.rules
@@ -17,5 +17,5 @@
 ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*", RUN+="/sbin/modprobe $env{MODALIAS}"
 
 # Create a symlink to any touchscreen input device
-SUBSYSTEM=="input", KERNEL=="event[0-9]*", ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*", SYMLINK+="input/touchscreen0"
+SUBSYSTEM=="input", KERNEL=="event[0-9]*", ENV{ID_INPUT_TOUCHSCREEN}=="1", SYMLINK+="input/touchscreen0"
 
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN
  2013-10-22 16:37 [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN Tom Rini
@ 2013-10-31 11:27 ` Tom Rini
  2014-04-23 23:41   ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Tom Rini @ 2013-10-31 11:27 UTC (permalink / raw)
  To: openembedded-core

On Tue, Oct 22, 2013 at 12:37:10PM -0400, Tom Rini wrote:

> Since udev 174 udev has been running input_id as a built-in command and
> setting this value in the environment for touchscreens.  Use this logic
> to detect when to make a touchscreen0 symlink.
> 
> Signed-off-by: Tom Rini <trini@ti.com>
> ---
>  meta/recipes-core/udev/udev/local.rules |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/udev/udev/local.rules b/meta/recipes-core/udev/udev/local.rules
> index 72d92ef..d0b0216 100644
> --- a/meta/recipes-core/udev/udev/local.rules
> +++ b/meta/recipes-core/udev/udev/local.rules
> @@ -17,5 +17,5 @@
>  ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*", RUN+="/sbin/modprobe $env{MODALIAS}"
>  
>  # Create a symlink to any touchscreen input device
> -SUBSYSTEM=="input", KERNEL=="event[0-9]*", ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*", SYMLINK+="input/touchscreen0"
> +SUBSYSTEM=="input", KERNEL=="event[0-9]*", ENV{ID_INPUT_TOUCHSCREEN}=="1", SYMLINK+="input/touchscreen0"

Ping

-- 
Tom


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN
  2013-10-31 11:27 ` Tom Rini
@ 2014-04-23 23:41   ` Denys Dmytriyenko
  2014-04-24  8:56     ` Paul Eggleton
  0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2014-04-23 23:41 UTC (permalink / raw)
  To: Tom Rini; +Cc: openembedded-core

On Thu, Oct 31, 2013 at 07:27:30AM -0400, Tom Rini wrote:
> On Tue, Oct 22, 2013 at 12:37:10PM -0400, Tom Rini wrote:
> 
> > Since udev 174 udev has been running input_id as a built-in command and
> > setting this value in the environment for touchscreens.  Use this logic
> > to detect when to make a touchscreen0 symlink.
> > 
> > Signed-off-by: Tom Rini <trini@ti.com>
> > ---
> >  meta/recipes-core/udev/udev/local.rules |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/recipes-core/udev/udev/local.rules b/meta/recipes-core/udev/udev/local.rules
> > index 72d92ef..d0b0216 100644
> > --- a/meta/recipes-core/udev/udev/local.rules
> > +++ b/meta/recipes-core/udev/udev/local.rules
> > @@ -17,5 +17,5 @@
> >  ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*", RUN+="/sbin/modprobe $env{MODALIAS}"
> >  
> >  # Create a symlink to any touchscreen input device
> > -SUBSYSTEM=="input", KERNEL=="event[0-9]*", ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*", SYMLINK+="input/touchscreen0"
> > +SUBSYSTEM=="input", KERNEL=="event[0-9]*", ENV{ID_INPUT_TOUCHSCREEN}=="1", SYMLINK+="input/touchscreen0"
> 
> Ping

Was it ever reviewed? Ping^2

-- 
Denys


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN
  2014-04-23 23:41   ` Denys Dmytriyenko
@ 2014-04-24  8:56     ` Paul Eggleton
  2014-04-24 18:01       ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Paul Eggleton @ 2014-04-24  8:56 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: openembedded-core

On Wednesday 23 April 2014 19:41:56 Denys Dmytriyenko wrote:
> On Thu, Oct 31, 2013 at 07:27:30AM -0400, Tom Rini wrote:
> > On Tue, Oct 22, 2013 at 12:37:10PM -0400, Tom Rini wrote:
> > > Since udev 174 udev has been running input_id as a built-in command and
> > > setting this value in the environment for touchscreens.  Use this logic
> > > to detect when to make a touchscreen0 symlink.
> > > 
> > > Signed-off-by: Tom Rini <trini@ti.com>
> > > ---
> > > 
> > >  meta/recipes-core/udev/udev/local.rules |    2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/meta/recipes-core/udev/udev/local.rules
> > > b/meta/recipes-core/udev/udev/local.rules index 72d92ef..d0b0216 100644
> > > --- a/meta/recipes-core/udev/udev/local.rules
> > > +++ b/meta/recipes-core/udev/udev/local.rules
> > > @@ -17,5 +17,5 @@
> > > 
> > >  ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*",
> > >  RUN+="/sbin/modprobe $env{MODALIAS}"
> > >  
> > >  # Create a symlink to any touchscreen input device
> > > 
> > > -SUBSYSTEM=="input", KERNEL=="event[0-9]*",
> > > ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*",
> > > SYMLINK+="input/touchscreen0" +SUBSYSTEM=="input",
> > > KERNEL=="event[0-9]*", ENV{ID_INPUT_TOUCHSCREEN}=="1",
> > > SYMLINK+="input/touchscreen0"> 
> > Ping
> 
> Was it ever reviewed? Ping^2

Sorry, what do you mean? It was reviewed and merged:

http://cgit.openembedded.org/openembedded-core/commit/?id=5abcfcd4380aacafc45d776f557738fb18089113

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN
  2014-04-24  8:56     ` Paul Eggleton
@ 2014-04-24 18:01       ` Denys Dmytriyenko
  0 siblings, 0 replies; 5+ messages in thread
From: Denys Dmytriyenko @ 2014-04-24 18:01 UTC (permalink / raw)
  To: Paul Eggleton; +Cc: openembedded-core

On Thu, Apr 24, 2014 at 09:56:20AM +0100, Paul Eggleton wrote:
> On Wednesday 23 April 2014 19:41:56 Denys Dmytriyenko wrote:
> > On Thu, Oct 31, 2013 at 07:27:30AM -0400, Tom Rini wrote:
> > > On Tue, Oct 22, 2013 at 12:37:10PM -0400, Tom Rini wrote:
> > > > Since udev 174 udev has been running input_id as a built-in command and
> > > > setting this value in the environment for touchscreens.  Use this logic
> > > > to detect when to make a touchscreen0 symlink.
> > > > 
> > > > Signed-off-by: Tom Rini <trini@ti.com>
> > > > ---
> > > > 
> > > >  meta/recipes-core/udev/udev/local.rules |    2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > 
> > > > diff --git a/meta/recipes-core/udev/udev/local.rules
> > > > b/meta/recipes-core/udev/udev/local.rules index 72d92ef..d0b0216 100644
> > > > --- a/meta/recipes-core/udev/udev/local.rules
> > > > +++ b/meta/recipes-core/udev/udev/local.rules
> > > > @@ -17,5 +17,5 @@
> > > > 
> > > >  ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*",
> > > >  RUN+="/sbin/modprobe $env{MODALIAS}"
> > > >  
> > > >  # Create a symlink to any touchscreen input device
> > > > 
> > > > -SUBSYSTEM=="input", KERNEL=="event[0-9]*",
> > > > ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*",
> > > > SYMLINK+="input/touchscreen0" +SUBSYSTEM=="input",
> > > > KERNEL=="event[0-9]*", ENV{ID_INPUT_TOUCHSCREEN}=="1",
> > > > SYMLINK+="input/touchscreen0"> 
> > > Ping
> > 
> > Was it ever reviewed? Ping^2
> 
> Sorry, what do you mean? It was reviewed and merged:
> 
> http://cgit.openembedded.org/openembedded-core/commit/?id=5abcfcd4380aacafc45d776f557738fb18089113

Oops, I guess I looked in Dora and it was merged after the release. Sorry for 
the noise.

-- 
Denys


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-04-24 18:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-22 16:37 [PATCH] udev: Update touchscreen rule for ID_INPUT_TOUCHSCREEN Tom Rini
2013-10-31 11:27 ` Tom Rini
2014-04-23 23:41   ` Denys Dmytriyenko
2014-04-24  8:56     ` Paul Eggleton
2014-04-24 18:01       ` Denys Dmytriyenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox