From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: snd-rme9652_mem and the H-DSP Date: Mon, 13 May 2002 12:18:48 +0200 Sender: alsa-devel-admin@lists.sourceforge.net Message-ID: References: <200205091257.g49CvTO16266@op.net> Mime-Version: 1.0 (generated by SEMI 1.14.3 - "Ushinoya") Content-Type: text/plain; charset=US-ASCII Return-path: Received: from Cantor.suse.de (ns.suse.de [213.95.15.193]) by alsa.alsa-project.org (8.9.3/8.9.3/SuSE Linux 8.9.3-0.1) with ESMTP id MAA04169 for ; Mon, 13 May 2002 12:18:54 +0200 In-Reply-To: Errors-To: alsa-devel-admin@lists.sourceforge.net List-Help: List-Post: List-Subscribe: , List-Unsubscribe: , List-Archive: To: Jaroslav Kysela Cc: Paul Davis , "alsa-devel@alsa-project.org" List-Id: alsa-devel@alsa-project.org At Thu, 9 May 2002 16:43:32 +0200 (CEST), Jaroslav wrote: > > On Thu, 9 May 2002, Paul Davis wrote: > > > both the hammerfall and hammerfall-dsp require exactly the same kind > > of memory allocation. the snd-rme9652_mem module, which allocates > > a contiguous chunk of about 1.2MB of physical RAM, uses the "driver > > index" to identify which snd-card is using a given buffer. this means > > that if we try to use both a hammerfall and a hammerfall-dsp, things > > don't work: both drivers pass "driver index" = 0 to the memory > > module, and the second try (whatever it is) concludes that it already > > has buffers allocated. > > > > i propose using the card->number value instead, so that the numbers are > > unique across all driver modules. > > > > the only other option i can see is to duplicate the code in > > snd-rme9652_mem, which seems really dumb. > > > > i also propose renaming the module to snd-hammerfall_mem to clearly > > indicate its generic status, and moving it into its own directory. it > > will be built if either CONFIG_SND_CARD_RME9652 and/or > > CONFIG_SND_CARD_HDSP are defined. > > > > let me know if this sounds like the right solution, or if you have a > > better one. > > Perhaps, we can use a more universal name like 'snd-pci-bigmem' or > something like that. Other hardware might require this (if I remember > correctly S3 SonicVibes cards also requires a big memory chunk for DLS > samples). We can move this module to /alsa-kernel/pci. yes. ice1712 needs relatively big chunk, too. how about to add module options to specify the pci id and the required size for non-default cards? Takashi _______________________________________________________________ Have big pipes? SourceForge.net is looking for download mirrors. We supply the hardware. You get the recognition. Email Us: bandwidth@sourceforge.net