![](https://secure.gravatar.com/avatar/bf81b6d3ff849b0a5646ca26862e1e26.jpg?s=120&d=mm&r=g)
7 Jul
2009
7 Jul
'09
8:03 a.m.
+/* Firmware Version info */ +struct snd_sst_fw_version {
- __u8 build; /* build number*/
- __u8 minor; /* minor number*/
- __u8 major; /* major number*/
- __u8 type; /* build type*/
+};
+/* Port info structure */ +struct snd_sst_port_info {
- __u16 port_type;
Just wondering -- is there big-endian support?
We have not tested this on big-endian, Could you tell us that should we make it endian safe?
+struct snd_sst_vol {
- unsigned int stream_id;
- int volume;
- unsigned long ramp_duration;
Are you sure to use long? Long can be different between 32 and 64bit architectures.
We will change it to u32.
+struct snd_sst_buff_entry {
- union {
void *user;
unsigned int offset;
- } buffer;
Is it OK? The pointer and int can be different sizes.
void* user - is the pointer to the buffer unsigned int offset - is the offset inside the buffer area. Not sure if I understand your comment. Please let us know if there is any issue you see with this structure.
Thanks, Harsha