On Mon, 11 Apr 2011, Timo Juhani Lindfors wrote: > maximilian attems <[email protected]> writes: > >> +try_modprobe() > > I'd prefer the function to be named modprobe_verbose() > > Ok. > > >> + try_modprobe atkbd > > why not using "-v" from modprobe, > > the only argument against that would count is busybox support. > > please check? > > I'm not very familiar with busybox internals but at least on squeeze I > don't even have the modprobe applet:
I checked busybox source, it does support the verbose -v switch. so changing your function in the patch, looks good. now the only thing that is holding me up to merge your patch is that /proc/bus/input/devices usage, there should be some sysfs file, no? cat /sys/class/input/input*/device/description shows a KBD -- maks -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: http://lists.debian.org/[email protected]
