Re: [PATCH] emu10k1: add per-voice PTR register dump to /proc entry

[Takashi Iwai <tiwai@suse.de>]

At Fri, 19 Nov 2004 19:10:24 -0500,
Lee Revell wrote:
> 
> On Thu, 2004-11-18 at 14:22 +0100, Takashi Iwai wrote:
> > > 
> > > Sure, will do.  One file per voice for the PTR registers (and one file
> > > for FN0) might actually be most useful.  This way you can diff them.
> > 
> > Yes, that makes sense.
> > 
> 
> OK regarding the previous message, I figured out how to create a
> directory under the card root (just set S_IFDIR).  However there is no
> way to create files under that directory because snd_card_proc_new makes
> the following assumption (from alsa-kernel/core/info.c):
> 
>     852  *
>     853  * The parent is assumed as card->proc_root.
>     854  *
> 
> So, I think the API has to be modified to allow a parent to be
> specified.

snd_card_proc_new() is just a helper function.
When creating a subdirectory and files under it, you have to manage
the instaces by yourself (i.e. requires a proper destructor).

An example is found in pci/ac97/ac97_proc.c.


Takashi


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/