linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Matthias Schwarzott <zzam@gentoo.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: udev problem DVB-S/T Cards
Date: Thu, 03 May 2007 12:13:41 +0000	[thread overview]
Message-ID: <200705031413.41703.zzam@gentoo.org> (raw)
In-Reply-To: <000301c78982$db8615e0$929241a0$@com>

[-- Attachment #1: Type: text/plain, Size: 2061 bytes --]

On Donnerstag, 3. Mai 2007, Razza wrote:
> Bryan Kadzban wrote on 03 May 2007 12:13:
> > Razza wrote:
> > > I have located path_id which is in /sbin on Fedora core 5
> >
> > Hmm; it should be in /lib/udev AFAIK.  But maybe this doesn't matter.
>
> Wouldn't be the first time something is in a completely different place in
> Fedora!
>
> > > there are some pretty long case statements in there (one I counted had
>
> 47 lines!),
>
> > Yeah, it's complicated.  But I don't think we can do much about that...
> >
> :o) Agree - The user (me) needs to get more intelligent!
>
> ----------8< snip! >8----------
>
> > That should be the right case, but that doesn't look like the current
> > version of the script.  The script from udev-110 looks like this:
> >
> > case "$TYPE" in
> >     block)
> >         handle_device
> >         echo "ID_PATH=$d"
> >         ;;
> >     scsi_tape)
> >         handle_device
> >         echo "ID_PATH=$d"
> >         ;;
> >     input)
> >         handle_device
> >         echo "ID_PATH=$d"
> >         ;;
> >     *)
> >         RESULT=1
> >         ;;
> > esac

Yes, this is the case-statement I changed last time.
As I did not have my changes (deleted long ago) I needed to redo it. Result is 
attached. The only change is duplicating one of those statementes.

By the way: why isnt there just
case "$TYPE" in
  block|scsi_tape|input)
    ...
    ;;
  *)
esac
as the code is the same in every place?

The reason for no longer doing this, is that I talked to linuxtv guys, and 
they told me they do not support such stuff :(

And well, to make it general applicable you need a good scheme to identify the 
cards.
A. by path: like above, fails with usb devices
B. by pci ids: fails with multiple identical cards, or with switching 
card-slots
Need to be improved for cards with multiple dvb-adapters (twin-tuner and 
similar)

C. by serial: some devices have serial numbers like usb, but not all


I just switched back to blacklisting all dvb-modules, and load them in well 
known order.

Matthias

-- 
Matthias Schwarzott (zzam)

[-- Attachment #2: path_id-dvb.diff --]
[-- Type: text/x-diff, Size: 259 bytes --]

--- /lib/udev/path_id.old	2007-05-03 14:03:39.000000000 +0200
+++ /lib/udev/path_id	2007-05-03 14:04:08.000000000 +0200
@@ -549,6 +549,10 @@
 		handle_device
 		echo "ID_PATH=$d"
 		;;
+	dvb)
+		handle_device
+		echo "ID_PATH=$d"
+		;;
 	*)
 		RESULT=1
 		;;

[-- Attachment #3: Type: text/plain, Size: 286 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

[-- Attachment #4: Type: text/plain, Size: 226 bytes --]

_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

  parent reply	other threads:[~2007-05-03 12:13 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-28 10:49 udev problem DVB-S/T Cards Razza
2007-05-02  8:47 ` Razza
2007-05-02 11:08 ` Bryan Kadzban
2007-05-02 12:10 ` Razza
2007-05-02 12:36 ` Matthias Schwarzott
2007-05-02 13:31 ` Matthias Schwarzott
2007-05-02 21:38 ` Razza
2007-05-02 22:20 ` Bryan Kadzban
2007-05-02 23:48 ` Razza
2007-05-03  1:56 ` Bryan Kadzban
2007-05-03  9:10 ` Razza
2007-05-03 11:12 ` Bryan Kadzban
2007-05-03 11:54 ` Razza
2007-05-03 12:13 ` Matthias Schwarzott [this message]
2007-05-07 17:18 ` Razza
2007-05-09 17:49 ` Razza
2007-05-30 10:35 ` Razza
2007-07-12  9:05 ` Razza

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=200705031413.41703.zzam@gentoo.org \
    --to=zzam@gentoo.org \
    --cc=linux-hotplug@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;
as well as URLs for NNTP newsgroup(s).