Testers of M-Audio Delta/Audiophile cards wanted

Testers of M-Audio Delta/Audiophile cards wanted

[Takashi Iwai]

Hi,

can M-Audio Delta or Audiophile users test the latest ice1712 driver?
i'd like to confirm that the new one works fine before removing the
old code.

the ice1712 codes are split to several files for better readability,
and now located under alsa-kernel/pci/ice1712 directory on cvs.
there is no new functionality added due to this change.

since the old ice1712.c is still compiled as default, you'll have to
run "make" on alsa-driver/pci/ice1712.  the new files are not compiled
otherwise.
then copy snd-ice1712.o on that directory to
/lib/modules/XXX/kernel/sound/pci directory manually, run "depmod -a",
and reload the module. 


TIA,

Takashi


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Tim Goetze]

Takashi Iwai wrote:

>since the old ice1712.c is still compiled as default, you'll have to
>run "make" on alsa-driver/pci/ice1712.  the new files are not compiled
>otherwise.

[~/audio/alsa-cvs/alsa-driver] make -C pci/ice1712
make: Entering directory
`/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'
gcc -DALSA_BUILD -D__KERNEL__ -DMODULE=1
-I/home/tim/audio/alsa-cvs/alsa-driver/include
-I/lib/modules/2.4.19-ll/build/include -O2
-mpreferred-stack-boundary=2 -march=k6 -DLINUX -Wall
-Wstrict-prototypes -fomit-frame-pointer -pipe  -DKBUILD_BASENAME=ews
-c -o ews.o ews.c
In file included from ews.c:1:
../../alsa-kernel/pci/ice1712/ews.c: In function
`snd_ice1712_ews_init':
../../alsa-kernel/pci/ice1712/ews.c:403: parse error before `*'
../../alsa-kernel/pci/ice1712/ews.c:406: `ak' undeclared (first use in
this function)
../../alsa-kernel/pci/ice1712/ews.c:406: (Each undeclared identifier
is reported only once
../../alsa-kernel/pci/ice1712/ews.c:406: for each function it appears
in.)
make: *** [ews.o] Error 1
make: Leaving directory
`/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'

i don't have a clue what gives. was my make invocation wrong? 

tim



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Takashi Iwai]

At Thu, 19 Sep 2002 16:35:19 +0200 (CEST),
Tim Goetze wrote:
> 
> Takashi Iwai wrote:
> 
> >since the old ice1712.c is still compiled as default, you'll have to
> >run "make" on alsa-driver/pci/ice1712.  the new files are not compiled
> >otherwise.
> 
> [~/audio/alsa-cvs/alsa-driver] make -C pci/ice1712
> make: Entering directory
> `/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'
> gcc -DALSA_BUILD -D__KERNEL__ -DMODULE=1
> -I/home/tim/audio/alsa-cvs/alsa-driver/include
> -I/lib/modules/2.4.19-ll/build/include -O2
> -mpreferred-stack-boundary=2 -march=k6 -DLINUX -Wall
> -Wstrict-prototypes -fomit-frame-pointer -pipe  -DKBUILD_BASENAME=ews
> -c -o ews.o ews.c
> In file included from ews.c:1:
> ../../alsa-kernel/pci/ice1712/ews.c: In function
> `snd_ice1712_ews_init':
> ../../alsa-kernel/pci/ice1712/ews.c:403: parse error before `*'
> ../../alsa-kernel/pci/ice1712/ews.c:406: `ak' undeclared (first use in
> this function)
> ../../alsa-kernel/pci/ice1712/ews.c:406: (Each undeclared identifier
> is reported only once
> ../../alsa-kernel/pci/ice1712/ews.c:406: for each function it appears
> in.)
> make: *** [ews.o] Error 1
> make: Leaving directory
> `/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'

could you check whether ice1712.h exists on alsa-kernel/pci/ice1712?
also, please check whether the file with the same name exists
somewhere in alsa-driver/include?

it seems that cvs server is dead now and i cannot check the cvs
respositry...


Takashi


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Tim Goetze]

Takashi Iwai wrote:

>At Thu, 19 Sep 2002 16:35:19 +0200 (CEST),
>Tim Goetze wrote:

[ak4524_t undefined ...]

>> make: *** [ews.o] Error 1
>> make: Leaving directory
>> `/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'
>
>could you check whether ice1712.h exists on alsa-kernel/pci/ice1712?
>also, please check whether the file with the same name exists
>somewhere in alsa-driver/include?

[~/audio/alsa-cvs/alsa-driver] find .. -name "ice1712.h"
../alsa-kernel/pci/ice1712/ice1712.h
[~/audio/alsa-cvs/alsa-driver] file alsa-kernel
alsa-kernel: symbolic link to ../alsa-kernel

tim



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Takashi Iwai]

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

At Thu, 19 Sep 2002 16:50:13 +0200 (CEST),
Tim Goetze wrote:
> 
> Takashi Iwai wrote:
> 
> >At Thu, 19 Sep 2002 16:35:19 +0200 (CEST),
> >Tim Goetze wrote:
> 
> [ak4524_t undefined ...]
> 
> >> make: *** [ews.o] Error 1
> >> make: Leaving directory
> >> `/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'
> >
> >could you check whether ice1712.h exists on alsa-kernel/pci/ice1712?
> >also, please check whether the file with the same name exists
> >somewhere in alsa-driver/include?
> 
> [~/audio/alsa-cvs/alsa-driver] find .. -name "ice1712.h"
> ../alsa-kernel/pci/ice1712/ice1712.h
> [~/audio/alsa-cvs/alsa-driver] file alsa-kernel
> alsa-kernel: symbolic link to ../alsa-kernel

possibly my tree was not checked in properly to cvs.
unfortuantely the cvs server is not accessible now, so i attach the
whole files under pci/ice1712.  please try this version.


Takashi

[-- Attachment #2: ice1712.tar.bz2 --]
[-- Type: application/octet-stream, Size: 23965 bytes --]

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Tim Goetze]

Takashi Iwai wrote:

>At Thu, 19 Sep 2002 16:50:13 +0200 (CEST),
>Tim Goetze wrote:
>> 
>> Takashi Iwai wrote:
>> 
>> >At Thu, 19 Sep 2002 16:35:19 +0200 (CEST),
>> >Tim Goetze wrote:
>> 
>> [ak4524_t undefined ...]
>> 
>> >> make: *** [ews.o] Error 1
>> >> make: Leaving directory
>> >> `/home/tim/audio/alsa-cvs/alsa-driver/pci/ice1712'
>> >
>> >could you check whether ice1712.h exists on alsa-kernel/pci/ice1712?
>> >also, please check whether the file with the same name exists
>> >somewhere in alsa-driver/include?
>> 
>> [~/audio/alsa-cvs/alsa-driver] find .. -name "ice1712.h"
>> ../alsa-kernel/pci/ice1712/ice1712.h
>> [~/audio/alsa-cvs/alsa-driver] file alsa-kernel
>> alsa-kernel: symbolic link to ../alsa-kernel
>
>possibly my tree was not checked in properly to cvs.
>unfortuantely the cvs server is not accessible now, so i attach the
>whole files under pci/ice1712.  please try this version.

much simpler. you did a c++ style auto variable allocation, declare
'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.

tim 



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Takashi Iwai]

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

At Thu, 19 Sep 2002 17:06:08 +0200 (CEST),
Tim Goetze wrote:
> 
> much simpler. you did a c++ style auto variable allocation, declare
> 'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.

oops, never noticed that.  i relied too much upon the compiler.
well, this means that gcc-3.2 accepts this style as normal C...?

will be fixed once when cvs server is repaired.

thanks,


Takashi

[-- Attachment #2: ews-fix.dif --]
[-- Type: application/octet-stream, Size: 718 bytes --]

Index: alsa-kernel/pci/ice1712/ews.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/ice1712/ews.c,v
retrieving revision 1.3
diff -u -r1.3 ews.c
--- alsa-kernel/pci/ice1712/ews.c	16 Sep 2002 10:12:31 -0000	1.3
+++ alsa-kernel/pci/ice1712/ews.c	19 Sep 2002 15:09:01 -0000
@@ -325,6 +325,7 @@
 static int __devinit snd_ice1712_ews_init(ice1712_t *ice)
 {
 	int err;
+	ak4524_t *ak;
 
 	/* set the analog DACs */
 	switch (ice->eeprom.subvendor) {
@@ -400,7 +401,7 @@
 	}
 
 	/* analog section */
-	ak4524_t *ak = &ice->ak4524;
+	ak = &ice->ak4524;
 	switch (ice->eeprom.subvendor) {
 	case ICE1712_SUBDEVICE_EWS88MT:
 		ak->num_adcs = ak->num_dacs = 8;

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Jaroslav Kysela]

On Thu, 19 Sep 2002, Takashi Iwai wrote:

> At Thu, 19 Sep 2002 17:06:08 +0200 (CEST),
> Tim Goetze wrote:
> > 
> > much simpler. you did a c++ style auto variable allocation, declare
> > 'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.
> 
> oops, never noticed that.  i relied too much upon the compiler.
> well, this means that gcc-3.2 accepts this style as normal C...?
> 
> will be fixed once when cvs server is repaired.

CVS works now.

						Jaroslav

-----
Jaroslav Kysela <perex@suse.cz>
Linux Kernel Sound Maintainer
ALSA Project  http://www.alsa-project.org
SuSE Linux    http://www.suse.com



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Tim Goetze]

Takashi Iwai wrote:

>At Thu, 19 Sep 2002 17:06:08 +0200 (CEST),
>Tim Goetze wrote:
>> 
>> much simpler. you did a c++ style auto variable allocation, declare
>> 'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.
>
>oops, never noticed that.  i relied too much upon the compiler.
>well, this means that gcc-3.2 accepts this style as normal C...?

strange that 3.2 isn't ANSI C compliant here.

anyway, driver load/unload, midi and pcm work with the audiophile
after sorting this out.

tim



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Clemens Ladisch]

Tim Goetze wrote:
> Takashi Iwai wrote:
> 
> >At Thu, 19 Sep 2002 17:06:08 +0200 (CEST),
> >Tim Goetze wrote:
> >>
> >> much simpler. you did a c++ style auto variable allocation, declare
> >> 'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.
> >
> >oops, never noticed that.  i relied too much upon the compiler.
> >well, this means that gcc-3.2 accepts this style as normal C...?
> 
> strange that 3.2 isn't ANSI C compliant here.

3.2 _is_ ANSI C compliant here. ("ANSI C" being C99 in this case :-)

I wanted to use mixed declarations and code in my drivers, too, but
there are too many users out there using 2.95.x.


HTH
Clemens


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

[ot] Re: Testers of M-Audio Delta/Audiophile cards wanted

[Tim Goetze]

Clemens Ladisch wrote:

>Tim Goetze wrote:
>> Takashi Iwai wrote:
>> 
>> >At Thu, 19 Sep 2002 17:06:08 +0200 (CEST),
>> >Tim Goetze wrote:
>> >>
>> >> much simpler. you did a c++ style auto variable allocation, declare
>> >> 'ak' at the start of snd_ice1712_ews_init and it compiles cleanly.
>> >
>> >oops, never noticed that.  i relied too much upon the compiler.
>> >well, this means that gcc-3.2 accepts this style as normal C...?
>> 
>> strange that 3.2 isn't ANSI C compliant here.
>
>3.2 _is_ ANSI C compliant here. ("ANSI C" being C99 in this case :-)

makes me look forward to writing

  try {
    pcm = new ALSA::PCM ("hw:0,0");
    ...

when C05 is here. 

;) tim




-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[James Tappin]

On Thu, 19 Sep 2002 14:49:19 +0200
Takashi Iwai <tiwai@suse.de> wrote:

> Hi,
> 
> can M-Audio Delta or Audiophile users test the latest ice1712 driver?
> i'd like to confirm that the new one works fine before removing the
> old code.
> 
> the ice1712 codes are split to several files for better readability,
> and now located under alsa-kernel/pci/ice1712 directory on cvs.
> there is no new functionality added due to this change.
> 
> since the old ice1712.c is still compiled as default, you'll have to
> run "make" on alsa-driver/pci/ice1712.  the new files are not compiled
> otherwise.
> then copy snd-ice1712.o on that directory to
> /lib/modules/XXX/kernel/sound/pci directory manually, run "depmod -a",
> and reload the module. 

With the Midiman Audiophile: aplay/arecord work correctly as does
recording from gramofile and playback via gwc (both use the OSS api). With
snd playback works but recording gives a no device error -- but that may
be because it's the version from Debian/Woody and it may be build against
an incompatible version of something (since I haven't tried to record with
snd since I switched from SuSE to Debian I'm not sure if the new driver
has any responsibility for this). 

Since I don't have any digital hardware I've not tested the AC97/SPDIF
stuff.

Best regards,
	James

-- 
James Tappin,               O__      "I forget the punishment for using
james@tappin.me.uk         --  \/`    Microsoft --- Something lingering
http://www.xena.uklinux.net/          with data loss in it I fancy"  


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

Re: Testers of M-Audio Delta/Audiophile cards wanted

[Takashi Iwai]

Hi,

thanks for your response.

it looks the new driver working, so i removed the old one and changed
Makefile to build the new driver on cvs.

if you have any problems, please report to alsa-devel.


ciao,

Takashi


At Thu, 19 Sep 2002 14:49:19 +0200,
I wrote:
> 
> Hi,
> 
> can M-Audio Delta or Audiophile users test the latest ice1712 driver?
> i'd like to confirm that the new one works fine before removing the
> old code.
> 
> the ice1712 codes are split to several files for better readability,
> and now located under alsa-kernel/pci/ice1712 directory on cvs.
> there is no new functionality added due to this change.
> 
> since the old ice1712.c is still compiled as default, you'll have to
> run "make" on alsa-driver/pci/ice1712.  the new files are not compiled
> otherwise.
> then copy snd-ice1712.o on that directory to
> /lib/modules/XXX/kernel/sound/pci directory manually, run "depmod -a",
> and reload the module. 
> 
> 
> TIA,
> 
> Takashi
> 
> 
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/alsa-devel
> 


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf