[alsa-devel] [PATCH 4/6] compress: add API header and driver header files

[Vinod Koul]

On Tue, 2011-11-22 at 12:18 +0100, Clemens Ladisch wrote:
> Vinod Koul wrote:
> > This patch adds the header files for ioctl definations and header file for
> > driver APIs for lower level device drivers to use
> 
> > +++ b/include/sound/compress_offload.h
> > ...
> > +#define SNDRV_COMPRESS_GET_CAPS		_IOWR('C', 0x00, struct snd_compr_caps *)
> > +#define SNDRV_COMPRESS_GET_CODEC_CAPS	_IOWR('C', 0x01, struct snd_compr_codec_caps *)
> > +#define SNDRV_COMPRESS_SET_PARAMS	_IOW('C', 0x02, struct snd_compr_params *)
> > +#define SNDRV_COMPRESS_GET_PARAMS	_IOR('C', 0x03, struct snd_compr_params *)
> > +#define SNDRV_COMPRESS_TSTAMP		_IOR('C', 0x10, struct snd_compr_tstamp *)
> > +#define SNDRV_COMPRESS_AVAIL		_IOR('C', 0x11, struct snd_compr_avail *)
> 
> The type whose size is made part of the ioctl number is supposed to be
> the actual structure, not a pointer to it.
Right, I missed that.
Fixed now, thanks for pointing

-- 
~Vinod