From: Takashi Iwai <tiwai@suse.de>
To: Andrey Borzenkov <arvidjaar@mail.ru>
Cc: rusty@rustcorp.com.au, ambx1@neo.rr.com, linux-kernel@vger.kernel.org
Subject: Re: file2alias for pnp (Re: modules.pnpmap output support)
Date: Fri, 28 Nov 2003 13:11:18 +0100 [thread overview]
Message-ID: <s5hk75kbsm1.wl@alsa2.suse.de> (raw)
In-Reply-To: <200311272159.00184.arvidjaar@mail.ru>
At Thu, 27 Nov 2003 21:58:58 +0300,
Andrey Borzenkov wrote:
>
> On Thursday 27 November 2003 17:41, Takashi Iwai wrote:
> > Hi,
> >
> > the attached is the patch to add pnp entries to file2alias.c.
> > i moved the definitions of pnp_device_id and pnp_card_device_id into
> > mod_devicetable.h as other devices do. if you don't like it, i'll try
> > to revert them and put definitions inside file2alias.c to keep the
> > changes minimum.
> >
> > the format of pnp alias is:
> > pnp:dXXXYYYY
> > or
> > pnp:cXXXYYYYdXXXYYYY[dXXXYYYY...]
> > where XXXYYYY is the pnp id with 7 letters (e.g. CTL0031), c shows the
> > card id, and d means the device id. multiple device ids will be
> > listed depending on the driver.
> >
> > for example,
> >
> > alias pnp:dYMH0021* opl3sa2
> > alias pnp:cALS0001d@@@0001d@X@0001d@H@0001* snd_als100
> >
> > Andrey, would it be feasible for hotplug stuff?
> >
>
> Will then every d be passed as separate parameter to hotplug? It means agent
> has to deal with unknown number of parameters or are there always fixed
> number of devs? apparently not as max is 8 and in your example only 3 are
> defined.
yes. the number of probed devices is variable.
note that, as seen above, there are two cases: with a card id and
without a card id. in the latter case, the card id is not checked but
only the given device id is checked, while the former case will need
checks all ids.
>
> If number is variable I guess better would be
>
> alias pnp:cXXXXXXdYYYYYYY[:YYYYYY...]
>
> i.e. put all devs IDs in one field; actually may be even separator is
> redundant as IDs have strict format to my knowledge.
it looks fine to me.
> Then hotplug agent gets two parameters - PNPID and PNPDEVS - and it is quite
> easy to build alias.
>
> can you give example how entries in sysfs look like (I do not have any ISA
> card). Is it possible to list them in the same order as in pnp_card_device_id
> table? Otherwise coldplugging becomes quite complicated. coldplugging script
> has to build the same string as hotplug invocation gets.
% ls -RF /sys/bus/pnp/
/sys/bus/pnp/:
devices/ drivers/
/sys/bus/pnp/devices:
00:01.00@ 00:01.01@ 00:01.02@ 00:01.03@ 00:01.04@ 00:01.05@
...
% ls -RF /sys/devices/pnp0/
/sys/devices/pnp0/:
00:01/ detach_state power/
/sys/devices/pnp0/00:01:
00:01.00/ 00:01.02/ 00:01.04/ detach_state
00:01.01/ 00:01.03/ 00:01.05/ power/
/sys/devices/pnp0/00:01/00:01.00:
detach_state id options power/ resources
...
...hmm, i don't see the card id on sysfs.
i think it's not implemented yet!
Adam, any plan to give the card id to sysfs ?
> Oh, BTW, what are those `@' in your example? If they mean single char
> wildcard, you should build them using `?' because modprobe is using normal
> fnmatch to match module name against aliases.
@ is a valid latter as the ISA pnp id string. it's not a wildcard.
(that's one of the reasons i showed it :)
the id consists of three letters [A-Z@] and four hex numbers
[0-9a-f].
one exception is 'XXXX' used as numbers, which means that any numbers
match. for example, CTLXXXX matches CTL0305, CTLa12f, etc.
ciao,
Takashi
next prev parent reply other threads:[~2003-11-28 12:14 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-17 12:37 modules.pnpmap output support "Andrey Borzenkov"
2003-11-17 13:34 ` Takashi Iwai
2003-11-17 14:05 ` Takashi Iwai
2003-11-20 4:35 ` Rusty Russell
2003-11-20 9:40 ` Takashi Iwai
2003-11-17 15:07 ` "Andrey Borzenkov"
2003-11-17 15:37 ` Takashi Iwai
2003-11-20 21:23 ` Adam Belay
2003-11-21 11:44 ` Takashi Iwai
2003-11-23 22:07 ` Adam Belay
2003-11-25 10:29 ` Takashi Iwai
2003-11-27 14:41 ` file2alias for pnp (Re: modules.pnpmap output support) Takashi Iwai
2003-11-27 18:58 ` Andrey Borzenkov
2003-11-28 12:11 ` Takashi Iwai [this message]
2003-12-02 22:31 ` Adam Belay
2003-12-03 11:31 ` Takashi Iwai
2003-11-18 3:07 ` modules.pnpmap output support Rusty Russell
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=s5hk75kbsm1.wl@alsa2.suse.de \
--to=tiwai@suse.de \
--cc=ambx1@neo.rr.com \
--cc=arvidjaar@mail.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
/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).