From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] emu10k1: add per-voice PTR register dump to /proc entry Date: Mon, 22 Nov 2004 11:40:33 +0100 Message-ID: References: <1100394086.2458.18.camel@krustophenia.net> <1100709062.6930.5.camel@krustophenia.net> <1100909425.13712.50.camel@krustophenia.net> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <1100909425.13712.50.camel@krustophenia.net> Sender: alsa-devel-admin@lists.sourceforge.net Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Lee Revell Cc: alsa-devel , James Courtier-Dutton List-Id: alsa-devel@alsa-project.org 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/