[PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Khem Raj]

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33 ++++++++++++++++++++++++++
 recipes/busybox/busybox_1.18.2.bb            |    2 +-
 2 files changed, 34 insertions(+), 1 deletions(-)
 create mode 100644 recipes/busybox/busybox-1.18.2/slugos/usb.sh

diff --git a/recipes/busybox/busybox-1.18.2/slugos/usb.sh b/recipes/busybox/busybox-1.18.2/slugos/usb.sh
new file mode 100644
index 0000000..172fdc6
--- /dev/null
+++ b/recipes/busybox/busybox-1.18.2/slugos/usb.sh
@@ -0,0 +1,33 @@
+#!/bin/sh
+
+case "$ACTION" in
+  add|"")
+    for uevent in /sys/class/usb_endpoint/usbdev?.*/*/uevent; do
+      . $uevent
+      if [ ! -e /dev/bus/usb/$BUSNUM/$DEVNUM ]; then
+        mkdir -p /dev/bus/usb/$BUSNUM
+        mknod /dev/bus/usb/$BUSNUM/$DEVNUM c 189 $MINOR
+      fi
+    done
+    ;;
+  remove)
+    for device in /dev/bus/usb/*/*; do
+      REMOVED=1
+      dev=`basename $device`
+      bus=`basename $(dirname $device)`
+      for uevent in /sys/class/usb_endpoint/usbdev?.*/*/uevent; do
+        . $uevent
+        if [ $dev -eq $DEVNUM ] && [ $bus -eq $BUSNUM ]; then
+          REMOVED=0
+          break;
+        fi
+      done
+      if [ $REMOVED -eq 1 ]; then
+        rm /dev/bus/usb/$bus/$dev
+        if [ -z $(ls /dev/bus/usb/$bus/) ]; then
+          rmdir /dev/bus/usb/$bus/
+        fi
+      fi
+    done
+    ;;
+esac
diff --git a/recipes/busybox/busybox_1.18.2.bb b/recipes/busybox/busybox_1.18.2.bb
index e6d17d3..7c47ab4 100644
--- a/recipes/busybox/busybox_1.18.2.bb
+++ b/recipes/busybox/busybox_1.18.2.bb
@@ -1,5 +1,5 @@
 require busybox_1.1x.inc
-PR = "${INC_PR}.2"
+PR = "${INC_PR}.3"
 
 SRC_URI += " \
 	http://busybox.net/downloads/fixes-1.18.2/busybox-1.18.2-modprobe-small.patch;name=patch01 \
-- 
1.7.2.3

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Eric Bénard]

Hi Khem,

On 21/01/2011 09:29, Khem Raj wrote:
> Signed-off-by: Khem Raj<raj.khem@gmail.com>
> ---
>   recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33 ++++++++++++++++++++++++++

Can't it become the generic script ? What makes it slugos specific ? (I didn't 
make a diff vs the other one so the question may be stupid ;-)

Thanks,
Eric

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Khem Raj]

On Fri, Jan 21, 2011 at 12:58 AM, Eric Bénard <eric@eukrea.com> wrote:
> Hi Khem,
>
> On 21/01/2011 09:29, Khem Raj wrote:
>>
>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>> ---
>>  recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33
>> ++++++++++++++++++++++++++
>
> Can't it become the generic script ? What makes it slugos specific ? (I
> didn't make a diff vs the other one so the question may be stupid ;-)
>

you should have diffed them. uevents are stored in different directory on slugos

> Thanks,
> Eric
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Eric Bénard]

On 21/01/2011 17:33, Khem Raj wrote:
> On Fri, Jan 21, 2011 at 12:58 AM, Eric Bénard<eric@eukrea.com>  wrote:
>> Hi Khem,
>>
>> On 21/01/2011 09:29, Khem Raj wrote:
>>>
>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>> ---
>>>   recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33
>>> ++++++++++++++++++++++++++
>>
>> Can't it become the generic script ? What makes it slugos specific ? (I
>> didn't make a diff vs the other one so the question may be stupid ;-)
>>
>
> you should have diffed them. uevents are stored in different directory on slugos
>
you could also put this in the commit log to prevent this diff to discover 
what is the change :-)


Eric

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Khem Raj]

On Fri, Jan 21, 2011 at 8:44 AM, Eric Bénard <eric@eukrea.com> wrote:
> On 21/01/2011 17:33, Khem Raj wrote:
>>
>> On Fri, Jan 21, 2011 at 12:58 AM, Eric Bénard<eric@eukrea.com>  wrote:
>>>
>>> Hi Khem,
>>>
>>> On 21/01/2011 09:29, Khem Raj wrote:
>>>>
>>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>>> ---
>>>>  recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33
>>>> ++++++++++++++++++++++++++
>>>
>>> Can't it become the generic script ? What makes it slugos specific ? (I
>>> didn't make a diff vs the other one so the question may be stupid ;-)
>>>
>>
>> you should have diffed them. uevents are stored in different directory on
>> slugos
>>
> you could also put this in the commit log to prevent this diff to discover
> what is the change :-)
>

will do

>
> Eric
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Mike Westerhof]

On 1/21/2011 10:33 AM, Khem Raj wrote:
> On Fri, Jan 21, 2011 at 12:58 AM, Eric Bénard <eric@eukrea.com> wrote:
>> Hi Khem,
>>
>> On 21/01/2011 09:29, Khem Raj wrote:
>>>
>>> Signed-off-by: Khem Raj<raj.khem@gmail.com>
>>> ---
>>>  recipes/busybox/busybox-1.18.2/slugos/usb.sh |   33
>>> ++++++++++++++++++++++++++
>>
>> Can't it become the generic script ? What makes it slugos specific ? (I
>> didn't make a diff vs the other one so the question may be stupid ;-)
>>
> 
> you should have diffed them. uevents are stored in different directory on slugos

Isn't this kernel-specific, though?  I don't have any other OE-based
devices handy just now to check, but what's difference, specifically,
and should we change something to bring SlugOS into line with the rest
of the world?

-Mike (mwester)

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Khem Raj]

On Fri, Jan 21, 2011 at 5:25 PM, Mike Westerhof <mike@mwester.net> wrote:
> Isn't this kernel-specific, though?  I don't have any other OE-based
> devices handy just now to check, but what's difference, specifically,
> and should we change something to bring SlugOS into line with the rest
> of the world?
>
> -Mike (mwester)
>

yes seems like slugos kernel enables.
CONFIG_SYSFS_DEPRECATED=y
CONFIG_SYSFS_DEPRECATED_V2=y

which enabled these. I havent tested the kernel by disabling them
as I havent figured out how to boot a kernel from network on slug
but if you can try that and see what /sys/class looks like that
would help or if I find some time to make slug suck the kernel
from a tftp then I can do it too.

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Tom Rini]

On 01/23/2011 11:27 AM, Khem Raj wrote:
> On Fri, Jan 21, 2011 at 5:25 PM, Mike Westerhof<mike@mwester.net>  wrote:
>> Isn't this kernel-specific, though?  I don't have any other OE-based
>> devices handy just now to check, but what's difference, specifically,
>> and should we change something to bring SlugOS into line with the rest
>> of the world?
>>
>> -Mike (mwester)
>>
>
> yes seems like slugos kernel enables.
> CONFIG_SYSFS_DEPRECATED=y
> CONFIG_SYSFS_DEPRECATED_V2=y
>
> which enabled these. I havent tested the kernel by disabling them
> as I havent figured out how to boot a kernel from network on slug
> but if you can try that and see what /sys/class looks like that
> would help or if I find some time to make slug suck the kernel
> from a tftp then I can do it too.

Note that these kernel options also place an upper bound on what udev 
can be used and it's very much worth checking if they can be turned off.

-- 
Tom Rini
Mentor Graphics Corporation

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Koen Kooi]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 23-01-11 19:27, Khem Raj wrote:
> On Fri, Jan 21, 2011 at 5:25 PM, Mike Westerhof <mike@mwester.net> wrote:
>> Isn't this kernel-specific, though?  I don't have any other OE-based
>> devices handy just now to check, but what's difference, specifically,
>> and should we change something to bring SlugOS into line with the rest
>> of the world?
>>
>> -Mike (mwester)
>>
> 
> yes seems like slugos kernel enables.
> CONFIG_SYSFS_DEPRECATED=y
> CONFIG_SYSFS_DEPRECATED_V2=y
> 
> which enabled these.

Good luck using udev >125 with those enabled.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFNPUBlMkyGM64RGpERAtQ9AJ9uXJt3LkVLmFcdGtlzyXrkcS9GogCgsZd2
so8m83VyDJYV0ZRGbpxNrJA=
=DOKc
-----END PGP SIGNATURE-----

Re: [PATCH 3/3] busybox_1.18.2.bb: slugos needs a modified version of usb.sh script

[Mike Westerhof]

On 1/23/2011 4:57 PM, Tom Rini wrote:
> On 01/23/2011 11:27 AM, Khem Raj wrote:
>> On Fri, Jan 21, 2011 at 5:25 PM, Mike Westerhof<mike@mwester.net>  wrote:
>>> Isn't this kernel-specific, though?  I don't have any other OE-based
>>> devices handy just now to check, but what's difference, specifically,
>>> and should we change something to bring SlugOS into line with the rest
>>> of the world?
>>>
>>> -Mike (mwester)
>>>
>>
>> yes seems like slugos kernel enables.
>> CONFIG_SYSFS_DEPRECATED=y
>> CONFIG_SYSFS_DEPRECATED_V2=y
>>
>> which enabled these. I havent tested the kernel by disabling them
>> as I havent figured out how to boot a kernel from network on slug
>> but if you can try that and see what /sys/class looks like that
>> would help or if I find some time to make slug suck the kernel
>> from a tftp then I can do it too.
> 
> Note that these kernel options also place an upper bound on what udev
> can be used and it's very much worth checking if they can be turned off.

SlugOS has bumped up to a 2.6.29 kernel with the above CONFIG items
disabled.  (This is still a relatively old kernel, but I know that it
basically works with SlugOS, so it's a good place to test mdev and, if
anyone wants to, newer udevs.)

-Mike (mwester)