Re: Re: [PATCH] emu10k1: minor send routing cleanup

Re: Re: [PATCH] emu10k1: minor send routing cleanup

[Lee Revell]

On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote:
> > 
> > Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate.  Also add
> > a few comments to explain send routing structure.
> > 
> > Signed-Off-By: Lee Revell <rlrevell@joe-job.com>
> 
> Thanks, applied to CVS.
> 

Sorry, please revert this one.  It disables the right channel.  The
macro must not be correct.

Lee



-------------------------------------------------------
This SF.Net email is sponsored by: InterSystems CACHE
FREE OODBMS DOWNLOAD - A multidimensional database that combines
robust object and relational technologies, making it a perfect match
for Java, C++,COM, XML, ODBC and JDBC. www.intersystems.com/match8

Re: Re: [PATCH] emu10k1: minor send routing cleanup

[James]

[-- Attachment #1: Type: text/plain, Size: 475 bytes --]

Lee Revell wrote:
> On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote:
> 
>>>Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate.  Also add
>>>a few comments to explain send routing structure.
>>>
>>>Signed-Off-By: Lee Revell <rlrevell@joe-job.com>
>>
>>Thanks, applied to CVS.
>>
> 
> 
> Sorry, please revert this one.  It disables the right channel.  The
> macro must not be correct.
> 
> Lee
> 

Use the attached patch to fix this problem.

[-- Attachment #2: emu10k1.h.diff --]
[-- Type: text/x-diff, Size: 883 bytes --]

--- 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)
 
 #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 int)route[2] << 16) | ((unsigned int)route[3] << 24))
 
 #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 int)route[6] << 16) | ((unsigned int)route[7] << 24))
 
 typedef struct snd_emu10k1_memblk {
 	snd_util_memblk_t mem;

Re: Re: [PATCH] emu10k1: minor send routing cleanup

[Takashi Iwai]

At Sun, 21 Nov 2004 01:21:25 +0000,
James@superbug.co.uk wrote:
> 
> [1  <text/plain; iso-8859-1 (quoted-printable)>]
> Lee Revell wrote:
> > On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote:
> > 
> >>>Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate.  Also add
> >>>a few comments to explain send routing structure.
> >>>
> >>>Signed-Off-By: Lee Revell <rlrevell@joe-job.com>
> >>
> >>Thanks, applied to CVS.
> >>
> > 
> > 
> > Sorry, please revert this one.  It disables the right channel.  The
> > macro must not be correct.
> > 
> > Lee
> > 
> 
> 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 <text/x-diff; iso-8859-1 (7bit)>]
> --- 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)
>  
>  #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 int)route[2] << 16) | ((unsigned int)route[3] << 24))
>  
>  #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 int)route[6] << 16) | ((unsigned int)route[7] << 24))
>  
>  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/

Re: Re: [PATCH] emu10k1: minor send routing cleanup

[James Courtier-Dutton]

Takashi Iwai wrote:
> At Sun, 21 Nov 2004 01:21:25 +0000,
> James@superbug.co.uk wrote:
> 
>>[1  <text/plain; iso-8859-1 (quoted-printable)>]
>>Lee Revell wrote:
>>
>>>On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote:
>>>
>>>
>>>>>Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate.  Also add
>>>>>a few comments to explain send routing structure.
>>>>>
>>>>>Signed-Off-By: Lee Revell <rlrevell@joe-job.com>
>>>>
>>>>Thanks, applied to CVS.
>>>>
>>>
>>>
>>>Sorry, please revert this one.  It disables the right channel.  The
>>>macro must not be correct.
>>>
>>>Lee
>>>
>>
>>Use the attached patch to fix this problem.
> 
> 
> Could you give the summary and signed-off-by lines, please?

Summary: Fixes the "It disables the right channel" bug.
Signed-off-by: James Courtier-Dutton

> 
> thanks,
> 
> 
> Takashi
> 
> 
>>[2 emu10k1.h.diff <text/x-diff; iso-8859-1 (7bit)>]
>>--- 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)
>> 
>> #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 int)route[2] << 16) | ((unsigned int)route[3] << 24))
>> 
>> #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 int)route[6] << 16) | ((unsigned int)route[7] << 24))
>> 
>> 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/

Re: Re: [PATCH] emu10k1: minor send routing cleanup

[Takashi Iwai]

At Mon, 22 Nov 2004 18:30:55 +0000,
James Courtier-Dutton wrote:
> 
> Takashi Iwai wrote:
> > At Sun, 21 Nov 2004 01:21:25 +0000,
> > James@superbug.co.uk wrote:
> > 
> >>[1  <text/plain; iso-8859-1 (quoted-printable)>]
> >>Lee Revell wrote:
> >>
> >>>On Wed, 2004-11-17 at 14:38 +0100, Takashi Iwai wrote:
> >>>
> >>>
> >>>>>Use snd_emu10k1_compose_audigy_fxrt* macro where appropriate.  Also add
> >>>>>a few comments to explain send routing structure.
> >>>>>
> >>>>>Signed-Off-By: Lee Revell <rlrevell@joe-job.com>
> >>>>
> >>>>Thanks, applied to CVS.
> >>>>
> >>>
> >>>
> >>>Sorry, please revert this one.  It disables the right channel.  The
> >>>macro must not be correct.
> >>>
> >>>Lee
> >>>
> >>
> >>Use the attached patch to fix this problem.
> > 
> > 
> > Could you give the summary and signed-off-by lines, please?
> 
> Summary: Fixes the "It disables the right channel" bug.
> Signed-off-by: James Courtier-Dutton

Ok, applied now.
Thanks.


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/