From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules Date: Wed, 08 Jun 2005 14:19:38 +0200 Message-ID: References: <200506071917.42953.alien999999999@users.sourceforge.net> <42A5DB84.2030407@mail.ru> <200506072002.49600.alien999999999@users.sourceforge.net> <42A6D936.8040101@netvigator.com> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <42A6D936.8040101@netvigator.com> 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: Raymond Cc: alsa-devel@lists.sourceforge.net, openvortex-dev@nongnu.org List-Id: alsa-devel@alsa-project.org At Wed, 08 Jun 2005 19:40:38 +0800, Raymond wrote: > > > typedef struct { > unsigned int *this00; /*CAsp4HwIO */ > long this04; /* How many filters for each side (default = 10) */ > long this08; /* inited to cero. Stereo flag? */ > } eqhw_t; > > static void vortex_EqHw_SetLevels(vortex_t * vortex, u16 peaks[]) > { > eqhw_t *eqhw = &(vortex->eq.this04); > int i; > > /* set left peaks */ > for (i = 0; i < eqhw->this04; i++) { > hwwrite(vortex->mmio, 0x2b024 + i * VORTEX_BAND_COEFF_SIZE, peaks[i]); > } > > hwwrite(vortex->mmio, 0x2b3cc, peaks[eqhw->this04]); > hwwrite(vortex->mmio, 0x2b3d8, peaks[eqhw->this04 + 1]); > > /* set right peaks */ > for (i = 0; i < eqhw->this04; i++) { > hwwrite(vortex->mmio, 0x2b204 + i * VORTEX_BAND_COEFF_SIZE, > peaks[i + (eqhw->this04 + 2)]); > } > > hwwrite(vortex->mmio, 0x2b3e4, peaks[2 + (eqhw->this04 * 2)]); > hwwrite(vortex->mmio, 0x2b3f0, peaks[3 + (eqhw->this04 * 2)]); > } > > > vortex_EqHw_SetLevels(vortex, (u16 *) eq_levels); > > > > > What is the reason to increase the size of the array eq_levels from 32 > to 64 ? ( eqhw->this04 seem to be 10 ) I guess he wants to keep the total array size just to be sure. But it seems not necessary. (Oh, and the cast above is evil. Let's remove it.) Above all, any chance to rename the mysterious this* fields to more understandable ones? Takashi ------------------------------------------------------- This SF.Net email is sponsored by: NEC IT Guy Games. How far can you shotput a projector? How fast can you ride your desk chair down the office luge track? If you want to score the big prize, get to know the little guy. Play to win an NEC 61" plasma display: http://www.necitguy.com/?r=20