[alsa-devel] ALSA 1.0.17rc2 release

[Alan Horstmann]

On Tuesday 17 June 2008 11:29, you wrote:
> At Tue, 17 Jun 2008 11:16:27 +0100,
>
> Alan Horstmann wrote:
> > On Tuesday 17 June 2008 10:18, I wrote:
> > > On Monday 16 June 2008 10:24, Jaroslav Kysela wrote:
> > > > 	ALSA 1.0.17rc2 packages (except tools, oss, python - no changes)
> > > > were released and are available for download on ALSA server.
> > > >
> > > > 	The alsa-driver compilation with 2.4 kernels should be fixed in
> > > > this release.
> > >
> > > The posted ak4531_codec issues are fixed.  I didn't get to post details
> > > of the other error, which still occurs with this 2.4.21-99 system:
> > >
> > > In file included from memory.c:3:
> > > ../../alsa-kernel/pci/emu10k1/memory.c: In function
> > > `synth_alloc_pages': ../../alsa-kernel/pci/emu10k1/memory.c:466: error:
> > > `GFP_DMA32' undeclared (first use in this function)
> > > ../../alsa-kernel/pci/emu10k1/memory.c:466: error: (Each undeclared
> > > identifier is reported only once
> > > ../../alsa-kernel/pci/emu10k1/memory.c:466: error: for each function it
> > > appears in.)
> > > ../../alsa-kernel/pci/emu10k1/memory.c:467: warning: implicit
> > > declaration of function `page_to_pfn'
> > > make[2]: *** [memory.o] Error 1
> > > make[2]: Leaving directory
> > > `/usr/share/alsa1.0.17rc2/alsa-driver-1.0.17rc2/ pci/emu10k1'
> >
> > I have just confirmed that pasting
> >
> > 	#define GFP_DMA32	((__force gfp_t)0x04u)
> >
> > into /alsa-kernel/pci/emu10k1/memory.c (not a correct fix -just taken
> > from 2.6.24 headers) enables build to complete.  So there should be no
> > other hidden issues.
>
> Could you check whether the tarball below works?
>
>  
> ftp://ftp.kernel.org/pub/linux/kernel/people/tiwai/snapshot/alsa-driver-sna
>pshot.tar.bz2

Yes that does build successfully.

However, I had noticed that GFP_DMA32 was also used by /pci/asihpi/
hpios_linux_kernel.c apparently without trouble.  But it would appear that 
asihpi is not being built, and checking shows neither is CS5535audio or hda.  
Make just reports 'nothing to be done..' in these 3 cases.  Is that 
intentional?

NB patch from Jaroslav also successful.

Alan