Re: [PATCH][RFCA] Sound: adding /proc/driver/{vendor}/{dev_pci}/ac97 entry

[salvador <salvador@inti.gov.ar>]

Alan Cox wrote:

> > +               if (proc_mkdir (proc_str, 0)) {
> > +                       sprintf(proc_str, "driver/%s/%s", name,
> > card->pci_dev->slot_name);
> > +                       if (proc_mkdir (proc_str, 0)) {
> > +                               sprintf(proc_str, "driver/%s/%s/ac97", name,
> > card->pci_dev->slot_name);
> > +                               create_proc_read_entry (proc_str, 0, 0,
> > ac97_read_proc, codec);
> > +                       }
> > +               }
>
> Where do you remove it.

I don't remove it, sorry I'll add it in the next iteration of the patch
BTW: I think the entry goes away when the module is removed. Is the kernel doing
a cleanup?

> Also a card can have multiple ac97 codecs

You are right, will also take care about it. Do you think
/proc/driver/{vendor}/{dev_pci}/{num_ac97}/ac97 will be ok?

SET

--
Salvador Eduardo Tropea (SET). (Electronics Engineer)
Visit my home page: http://welcome.to/SetSoft or
http://www.geocities.com/SiliconValley/Vista/6552/
Alternative e-mail: set@computer.org set@ieee.org
Address: Curapaligue 2124, Caseros, 3 de Febrero
Buenos Aires, (1678), ARGENTINA Phone: +(5411) 4759 0013