| Message ID | 20170324075321.12384-1-joerg.krause@embedded.rocks |
|---|---|
| State | Accepted |
| Headers | show |
On 24-03-17 08:53, Jörg Krause wrote: > mdev does not set "/dev/snd" and "/dev/input" group and permission > propery, because with commit c3cf1e30a3022453311a7e9fe11d94c7a381640e properly > (May 2013!) the behavior of mdev has changed. I.e., released in busybox 1.22, added to Buildroot in 4e8dadfe4b9c. Note that at that time we still had a version choice for busybox, so there was anyway a problem. > The device name is now taken directly from the uevent file and does no > longer match the old rule. > > Fix the rules for "/dev/snd" and "/dev/input" according to the example > given in [1]. > > [1] http://lists.busybox.net/pipermail/busybox/2015-February/082297.html > > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> I guess almost nobody uses sound, and nowadays most GUI stuff that uses /dev/input needs (e)udev anyway. Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Regards, Arnout > --- > package/busybox/mdev.conf | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/package/busybox/mdev.conf b/package/busybox/mdev.conf > index 241c1c27d..eaece1b73 100644 > --- a/package/busybox/mdev.conf > +++ b/package/busybox/mdev.conf > @@ -23,17 +23,17 @@ ttyS[0-9]* root:root 660 > ttyUSB[0-9]* root:root 660 > > # alsa sound devices > -pcm.* root:audio 660 =snd/ > -control.* root:audio 660 =snd/ > -midi.* root:audio 660 =snd/ > -seq root:audio 660 =snd/ > -timer root:audio 660 =snd/ > +snd/pcm.* root:audio 660 > +snd/control.* root:audio 660 > +snd/midi.* root:audio 660 > +snd/seq root:audio 660 > +snd/timer root:audio 660 > > # input stuff > -event[0-9]+ root:root 640 =input/ > -mice root:root 640 =input/ > -mouse[0-9] root:root 640 =input/ > -ts[0-9] root:root 600 =input/ > +input/event[0-9]+ root:root 640 > +input/mice root:root 640 > +input/mouse[0-9] root:root 640 > +input/ts[0-9] root:root 600 > > # load modules > $MODALIAS=.* root:root 660 @modprobe "$MODALIAS" >
Hello, On Fri, 24 Mar 2017 08:53:21 +0100, Jörg Krause wrote: > mdev does not set "/dev/snd" and "/dev/input" group and permission > propery, because with commit c3cf1e30a3022453311a7e9fe11d94c7a381640e > (May 2013!) the behavior of mdev has changed. > > The device name is now taken directly from the uevent file and does no > longer match the old rule. > > Fix the rules for "/dev/snd" and "/dev/input" according to the example > given in [1]. > > [1] http://lists.busybox.net/pipermail/busybox/2015-February/082297.html > > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> > --- > package/busybox/mdev.conf | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) Applied to master, thanks. Thomas
>>>>> "Jörg" == Jörg Krause <joerg.krause@embedded.rocks> writes: > mdev does not set "/dev/snd" and "/dev/input" group and permission > propery, because with commit c3cf1e30a3022453311a7e9fe11d94c7a381640e > (May 2013!) the behavior of mdev has changed. > The device name is now taken directly from the uevent file and does no > longer match the old rule. > Fix the rules for "/dev/snd" and "/dev/input" according to the example > given in [1]. > [1] http://lists.busybox.net/pipermail/busybox/2015-February/082297.html > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> Committed to 2017.02.x, thanks.
diff --git a/package/busybox/mdev.conf b/package/busybox/mdev.conf index 241c1c27d..eaece1b73 100644 --- a/package/busybox/mdev.conf +++ b/package/busybox/mdev.conf @@ -23,17 +23,17 @@ ttyS[0-9]* root:root 660 ttyUSB[0-9]* root:root 660 # alsa sound devices -pcm.* root:audio 660 =snd/ -control.* root:audio 660 =snd/ -midi.* root:audio 660 =snd/ -seq root:audio 660 =snd/ -timer root:audio 660 =snd/ +snd/pcm.* root:audio 660 +snd/control.* root:audio 660 +snd/midi.* root:audio 660 +snd/seq root:audio 660 +snd/timer root:audio 660 # input stuff -event[0-9]+ root:root 640 =input/ -mice root:root 640 =input/ -mouse[0-9] root:root 640 =input/ -ts[0-9] root:root 600 =input/ +input/event[0-9]+ root:root 640 +input/mice root:root 640 +input/mouse[0-9] root:root 640 +input/ts[0-9] root:root 600 # load modules $MODALIAS=.* root:root 660 @modprobe "$MODALIAS"
mdev does not set "/dev/snd" and "/dev/input" group and permission propery, because with commit c3cf1e30a3022453311a7e9fe11d94c7a381640e (May 2013!) the behavior of mdev has changed. The device name is now taken directly from the uevent file and does no longer match the old rule. Fix the rules for "/dev/snd" and "/dev/input" according to the example given in [1]. [1] http://lists.busybox.net/pipermail/busybox/2015-February/082297.html Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks> --- package/busybox/mdev.conf | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-)