[alsa-devel] snd_mask_min

Tim Blechmann tim at klingt.org
Tue Jul 26 09:16:31 CEST 2011


>> it uses an inline function snd_mask_min, which is declared in
>> sound/pcm_params.h, but this file is not included by sound/pcm.h, where it
>> is defined. including it manually, causes a huge number of errors. it would
>> be very helpful, if pcm.h would be self-contained.
> 
> Well, pcm_params.h is just an additional header that aren't used by
> most drivers, so it doesn't make much sense to include such
> unconditionally from the common pcm.h.

then wouldn't it make sense to move those parts from pcm.h that depend on 
pcm_params.h to that header? especially the params_* family? at the moment pcm.h 
is not self-contained, which is not really what i expect from a header ...

tim



More information about the Alsa-devel mailing list