Takashi Iwai wrote: > At Wed, 10 May 2006 15:28:09 +0200, > Hans-Christian Egtvedt wrote: >> Takashi Iwai wrote: >>> At Wed, 10 May 2006 14:47:01 +0200, >>> Hans-Christian Egtvedt wrote: >> >> >>>> I have attached a patch which changes all enums into ints, and the >>>> library now works for me. I have no idea if I have broken something >>>> else. I have another issue, but I'll post that in a separate email. >>> The change to include/sound/*.h looks OK. Actually, the kernel side >>> was already fixed to use int. I was too lazy to update the alsa-lib >>> side. >>> >>> But, are other changes really necessary? They are local in alsa-lib, >>> so the size shouldn't matter. >> I don't have the complete insight of which structs are being used >> towards the kernel and which are just internal, so I did the "easy" way >> of search and replace the entire library. If you, or somebody else with >> good knowledge of the alsa library, redo my patch to those only used for >> communication towards the kernel I can give it a go and see if it's >> still good. > > The files in include/sound/* are kernel API. > > Some files in include/*.h are public (you can find in > include/Makefile.am), so you should be careful to change them. > Other files are internal in alsa-lib. > > So, just take a hunk to change include/sound/*.h from the original > patch, and drop the rest. I've stripped the patch to do what you say now, and alsamixer and amixer works fine. I have an issue with aplay and arecord, will post in another email. New patch attached. -- With kind regards, Med vennlig hilsen, Hans-Christian Egtvedt Applications Engineer - AVR Applications Lab Atmel Norway