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