From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Re: [PATCH] emu10k1: minor send routing cleanup Date: Mon, 22 Nov 2004 12:36:32 +0100 Message-ID: References: <200411210121.25291.James@superbug.co.uk> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <200411210121.25291.James@superbug.co.uk> 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: James@superbug.co.uk Cc: Lee Revell , alsa-devel , perex@suse.cz List-Id: alsa-devel@alsa-project.org At Sun, 21 Nov 2004 01:21:25 +0000, James@superbug.co.uk wrote: >=20 > [1 ] > Lee Revell wrote: > > On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote: > >=20 > >>>Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate. =A0Als= o add > >>>a few comments to explain send routing structure. > >>> > >>>Signed-Off-By: Lee Revell > >> > >>Thanks, applied to CVS. > >> > >=20 > >=20 > > Sorry, please revert this one. =A0It disables the right channel. =A0T= he > > macro must not be correct. > >=20 > > Lee > >=20 >=20 > Use the attached patch to fix this problem. Could you give the summary and signed-off-by lines, please? thanks, Takashi > [2 emu10k1.h.diff ] > --- emu10k1.h.org 2004-11-21 01:04:06.875103895 +0000 > +++ emu10k1.h 2004-11-21 01:04:10.252609912 +0000 > @@ -840,10 +840,10 @@ > ((route[0] | (route[1] << 4) | (route[2] << 8) | (route[3] << 12)) << = 16) > =20 > #define snd_emu10k1_compose_audigy_fxrt1(route) \ > -(((unsigned int)route[0] | ((unsigned int)route[1] << 8) | ((unsigned = int)route[2] << 16) | ((unsigned int)route[3] << 12)) << 24) > +((unsigned int)route[0] | ((unsigned int)route[1] << 8) | ((unsigned i= nt)route[2] << 16) | ((unsigned int)route[3] << 24)) > =20 > #define snd_emu10k1_compose_audigy_fxrt2(route) \ > -(((unsigned int)route[4] | ((unsigned int)route[5] << 8) | ((unsigned = int)route[6] << 16) | ((unsigned int)route[7] << 12)) << 24) > +((unsigned int)route[4] | ((unsigned int)route[5] << 8) | ((unsigned i= nt)route[6] << 16) | ((unsigned int)route[7] << 24)) > =20 > typedef struct snd_emu10k1_memblk { > snd_util_memblk_t mem; ------------------------------------------------------- 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/