Fedora kernel-2.6.17-1.2142_FC4 patched with stable patch-2.6.17.4-vs2.0.2-rc26.diff
[linux-2.6.git] / sound / pci / mixart / mixart_hwdep.c
index 0da472b..ca05075 100644 (file)
@@ -40,7 +40,9 @@
  * @param value value
  * @param timeout timeout in centisenconds
  */
-static int mixart_wait_nice_for_register_value(mixart_mgr_t *mgr, u32 offset, int is_egal, u32 value, unsigned long timeout)
+static int mixart_wait_nice_for_register_value(struct mixart_mgr *mgr,
+                                              u32 offset, int is_egal,
+                                              u32 value, unsigned long timeout)
 {
        unsigned long end_time = jiffies + (timeout * HZ / 100);
        u32 read;
@@ -66,8 +68,6 @@ static int mixart_wait_nice_for_register_value(mixart_mgr_t *mgr, u32 offset, in
 /*
   structures needed to upload elf code packets 
  */
-typedef struct snd_mixart_elf32_ehdr snd_mixart_elf32_ehdr_t;
-
 struct snd_mixart_elf32_ehdr {
        u8      e_ident[16];
        u16     e_type;
@@ -85,8 +85,6 @@ struct snd_mixart_elf32_ehdr {
        u16     e_shstrndx;
 };
 
-typedef struct snd_mixart_elf32_phdr snd_mixart_elf32_phdr_t;
-
 struct snd_mixart_elf32_phdr {
        u32     p_type;
        u32     p_offset;
@@ -98,19 +96,19 @@ struct snd_mixart_elf32_phdr {
        u32     p_align;
 };
 
-static int mixart_load_elf(mixart_mgr_t *mgr, const struct firmware *dsp )
+static int mixart_load_elf(struct mixart_mgr *mgr, const struct firmware *dsp )
 {
        char                    elf32_magic_number[4] = {0x7f,'E','L','F'};
-       snd_mixart_elf32_ehdr_t *elf_header;
+       struct snd_mixart_elf32_ehdr *elf_header;
        int                     i;
 
-       elf_header = (snd_mixart_elf32_ehdr_t *)dsp->data;
+       elf_header = (struct snd_mixart_elf32_ehdr *)dsp->data;
        for( i=0; i<4; i++ )
                if ( elf32_magic_number[i] != elf_header->e_ident[i] )
                        return -EINVAL;
 
        if( elf_header->e_phoff != 0 ) {
-               snd_mixart_elf32_phdr_t     elf_programheader;
+               struct snd_mixart_elf32_phdr     elf_programheader;
 
                for( i=0; i < be16_to_cpu(elf_header->e_phnum); i++ ) {
                        u32 pos = be32_to_cpu(elf_header->e_phoff) + (u32)(i * be16_to_cpu(elf_header->e_phentsize));
@@ -137,32 +135,41 @@ static int mixart_load_elf(mixart_mgr_t *mgr, const struct firmware *dsp )
 #define MIXART_FIRST_ANA_AUDIO_ID       0
 #define MIXART_FIRST_DIG_AUDIO_ID       8
 
-static int mixart_enum_connectors(mixart_mgr_t *mgr)
+static int mixart_enum_connectors(struct mixart_mgr *mgr)
 {
        u32 k;
        int err;
-       mixart_msg_t request;
-       mixart_enum_connector_resp_t connector;
-       mixart_audio_info_req_t  audio_info_req;
-       mixart_audio_info_resp_t audio_info;
+       struct mixart_msg request;
+       struct mixart_enum_connector_resp *connector;
+       struct mixart_audio_info_req  *audio_info_req;
+       struct mixart_audio_info_resp *audio_info;
+
+       connector = kmalloc(sizeof(*connector), GFP_KERNEL);
+       audio_info_req = kmalloc(sizeof(*audio_info_req), GFP_KERNEL);
+       audio_info = kmalloc(sizeof(*audio_info), GFP_KERNEL);
+       if (! connector || ! audio_info_req || ! audio_info) {
+               err = -ENOMEM;
+               goto __error;
+       }
 
-       audio_info_req.line_max_level = MIXART_FLOAT_P_22_0_TO_HEX;
-       audio_info_req.micro_max_level = MIXART_FLOAT_M_20_0_TO_HEX;
-       audio_info_req.cd_max_level = MIXART_FLOAT____0_0_TO_HEX;
+       audio_info_req->line_max_level = MIXART_FLOAT_P_22_0_TO_HEX;
+       audio_info_req->micro_max_level = MIXART_FLOAT_M_20_0_TO_HEX;
+       audio_info_req->cd_max_level = MIXART_FLOAT____0_0_TO_HEX;
 
        request.message_id = MSG_SYSTEM_ENUM_PLAY_CONNECTOR;
-       request.uid = (mixart_uid_t){0,0};  /* board num = 0 */
+       request.uid = (struct mixart_uid){0,0};  /* board num = 0 */
        request.data = NULL;
        request.size = 0;
 
-       err = snd_mixart_send_msg(mgr, &request, sizeof(connector), &connector);
-       if((err < 0) || (connector.error_code) || (connector.uid_count > MIXART_MAX_PHYS_CONNECTORS)) {
+       err = snd_mixart_send_msg(mgr, &request, sizeof(*connector), connector);
+       if((err < 0) || (connector->error_code) || (connector->uid_count > MIXART_MAX_PHYS_CONNECTORS)) {
                snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_PLAY_CONNECTOR\n");
-               return -EINVAL;
+               err = -EINVAL;
+               goto __error;
        }
 
-       for(k=0; k < connector.uid_count; k++) {
-               mixart_pipe_t* pipe;
+       for(k=0; k < connector->uid_count; k++) {
+               struct mixart_pipe *pipe;
 
                if(k < MIXART_FIRST_DIG_AUDIO_ID) {
                        pipe = &mgr->chip[k/2]->pipe_out_ana;
@@ -170,40 +177,41 @@ static int mixart_enum_connectors(mixart_mgr_t *mgr)
                        pipe = &mgr->chip[(k-MIXART_FIRST_DIG_AUDIO_ID)/2]->pipe_out_dig;
                }
                if(k & 1) {
-                       pipe->uid_right_connector = connector.uid[k];   /* odd */
+                       pipe->uid_right_connector = connector->uid[k];   /* odd */
                } else {
-                       pipe->uid_left_connector = connector.uid[k];    /* even */
+                       pipe->uid_left_connector = connector->uid[k];    /* even */
                }
 
-               /* snd_printk(KERN_DEBUG "playback connector[%d].object_id = %x\n", k, connector.uid[k].object_id); */
+               /* snd_printk(KERN_DEBUG "playback connector[%d].object_id = %x\n", k, connector->uid[k].object_id); */
 
                /* TODO: really need send_msg MSG_CONNECTOR_GET_AUDIO_INFO for each connector ? perhaps for analog level caps ? */
                request.message_id = MSG_CONNECTOR_GET_AUDIO_INFO;
-               request.uid = connector.uid[k];
-               request.data = &audio_info_req;
-               request.size = sizeof(audio_info_req);
+               request.uid = connector->uid[k];
+               request.data = audio_info_req;
+               request.size = sizeof(*audio_info_req);
 
-               err = snd_mixart_send_msg(mgr, &request, sizeof(audio_info), &audio_info);
+               err = snd_mixart_send_msg(mgr, &request, sizeof(*audio_info), audio_info);
                if( err < 0 ) {
                        snd_printk(KERN_ERR "error MSG_CONNECTOR_GET_AUDIO_INFO\n");
-                       return err;
+                       goto __error;
                }
-               /*snd_printk(KERN_DEBUG "play  analog_info.analog_level_present = %x\n", audio_info.info.analog_info.analog_level_present);*/
+               /*snd_printk(KERN_DEBUG "play  analog_info.analog_level_present = %x\n", audio_info->info.analog_info.analog_level_present);*/
        }
 
        request.message_id = MSG_SYSTEM_ENUM_RECORD_CONNECTOR;
-       request.uid = (mixart_uid_t){0,0};  /* board num = 0 */
+       request.uid = (struct mixart_uid){0,0};  /* board num = 0 */
        request.data = NULL;
        request.size = 0;
 
-       err = snd_mixart_send_msg(mgr, &request, sizeof(connector), &connector);
-       if((err < 0) || (connector.error_code) || (connector.uid_count > MIXART_MAX_PHYS_CONNECTORS)) {
+       err = snd_mixart_send_msg(mgr, &request, sizeof(*connector), connector);
+       if((err < 0) || (connector->error_code) || (connector->uid_count > MIXART_MAX_PHYS_CONNECTORS)) {
                snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_RECORD_CONNECTOR\n");
-               return -EINVAL;
+               err = -EINVAL;
+               goto __error;
        }
 
-       for(k=0; k < connector.uid_count; k++) {
-               mixart_pipe_t* pipe;
+       for(k=0; k < connector->uid_count; k++) {
+               struct mixart_pipe *pipe;
 
                if(k < MIXART_FIRST_DIG_AUDIO_ID) {
                        pipe = &mgr->chip[k/2]->pipe_in_ana;
@@ -211,38 +219,44 @@ static int mixart_enum_connectors(mixart_mgr_t *mgr)
                        pipe = &mgr->chip[(k-MIXART_FIRST_DIG_AUDIO_ID)/2]->pipe_in_dig;
                }
                if(k & 1) {
-                       pipe->uid_right_connector = connector.uid[k];   /* odd */
+                       pipe->uid_right_connector = connector->uid[k];   /* odd */
                } else {
-                       pipe->uid_left_connector = connector.uid[k];    /* even */
+                       pipe->uid_left_connector = connector->uid[k];    /* even */
                }
 
-               /* snd_printk(KERN_DEBUG "capture connector[%d].object_id = %x\n", k, connector.uid[k].object_id); */
+               /* snd_printk(KERN_DEBUG "capture connector[%d].object_id = %x\n", k, connector->uid[k].object_id); */
 
                /* TODO: really need send_msg MSG_CONNECTOR_GET_AUDIO_INFO for each connector ? perhaps for analog level caps ? */
                request.message_id = MSG_CONNECTOR_GET_AUDIO_INFO;
-               request.uid = connector.uid[k];
-               request.data = &audio_info_req;
-               request.size = sizeof(audio_info_req);
+               request.uid = connector->uid[k];
+               request.data = audio_info_req;
+               request.size = sizeof(*audio_info_req);
 
-               err = snd_mixart_send_msg(mgr, &request, sizeof(audio_info), &audio_info);
+               err = snd_mixart_send_msg(mgr, &request, sizeof(*audio_info), audio_info);
                if( err < 0 ) {
                        snd_printk(KERN_ERR "error MSG_CONNECTOR_GET_AUDIO_INFO\n");
-                       return err;
+                       goto __error;
                }
-               /*snd_printk(KERN_DEBUG "rec  analog_info.analog_level_present = %x\n", audio_info.info.analog_info.analog_level_present);*/
+               /*snd_printk(KERN_DEBUG "rec  analog_info.analog_level_present = %x\n", audio_info->info.analog_info.analog_level_present);*/
        }
+       err = 0;
 
-       return 0;
+ __error:
+       kfree(connector);
+       kfree(audio_info_req);
+       kfree(audio_info);
+
+       return err;
 }
 
-static int mixart_enum_physio(mixart_mgr_t *mgr)
+static int mixart_enum_physio(struct mixart_mgr *mgr)
 {
        u32 k;
        int err;
-       mixart_msg_t request;
-       mixart_uid_t get_console_mgr;
-       mixart_return_uid_t console_mgr;
-       mixart_uid_enumeration_t phys_io;
+       struct mixart_msg request;
+       struct mixart_uid get_console_mgr;
+       struct mixart_return_uid console_mgr;
+       struct mixart_uid_enumeration phys_io;
 
        /* get the uid for the console manager */
        get_console_mgr.object_id = 0;
@@ -264,7 +278,7 @@ static int mixart_enum_physio(mixart_mgr_t *mgr)
        mgr->uid_console_manager = console_mgr.uid;
 
        request.message_id = MSG_SYSTEM_ENUM_PHYSICAL_IO;
-       request.uid = (mixart_uid_t){0,0};
+       request.uid = (struct mixart_uid){0,0};
        request.data = &console_mgr.uid;
        request.size = sizeof(console_mgr.uid);
 
@@ -285,11 +299,11 @@ static int mixart_enum_physio(mixart_mgr_t *mgr)
 }
 
 
-static int mixart_first_init(mixart_mgr_t *mgr)
+static int mixart_first_init(struct mixart_mgr *mgr)
 {
        u32 k;
        int err;
-       mixart_msg_t request;
+       struct mixart_msg request;
 
        if((err = mixart_enum_connectors(mgr)) < 0) return err;
 
@@ -298,7 +312,7 @@ static int mixart_first_init(mixart_mgr_t *mgr)
        /* send a synchro command to card (necessary to do this before first MSG_STREAM_START_STREAM_GRP_PACKET) */
        /* though why not here */
        request.message_id = MSG_SYSTEM_SEND_SYNCHRO_CMD;
-       request.uid = (mixart_uid_t){0,0};
+       request.uid = (struct mixart_uid){0,0};
        request.data = NULL;
        request.size = 0;
        /* this command has no data. response is a 32 bit status */
@@ -315,7 +329,7 @@ static int mixart_first_init(mixart_mgr_t *mgr)
 /* firmware base addresses (when hard coded) */
 #define MIXART_MOTHERBOARD_XLX_BASE_ADDRESS   0x00600000
 
-static int mixart_dsp_load(mixart_mgr_t* mgr, int index, const struct firmware *dsp)
+static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmware *dsp)
 {
        int           err, card_index;
        u32           status_xilinx, status_elf, status_daught;
@@ -497,7 +511,7 @@ static int mixart_dsp_load(mixart_mgr_t* mgr, int index, const struct firmware *
 
                /* create devices and mixer in accordance with HW options*/
         for (card_index = 0; card_index < mgr->num_cards; card_index++) {
-               mixart_t *chip = mgr->chip[card_index];
+               struct snd_mixart *chip = mgr->chip[card_index];
 
                if ((err = snd_mixart_create_pcm(chip)) < 0)
                        return err;
@@ -525,7 +539,7 @@ static int mixart_dsp_load(mixart_mgr_t* mgr, int index, const struct firmware *
 
 #ifdef SND_MIXART_FW_LOADER
 
-int snd_mixart_setup_firmware(mixart_mgr_t *mgr)
+int snd_mixart_setup_firmware(struct mixart_mgr *mgr)
 {
        static char *fw_files[3] = {
                "miXart8.xlx", "miXart8.elf", "miXart8AES.xlx"
@@ -546,6 +560,7 @@ int snd_mixart_setup_firmware(mixart_mgr_t *mgr)
                release_firmware(fw_entry);
                if (err < 0)
                        return err;
+               mgr->dsp_loaded |= 1 << i;
        }
        return 0;
 }
@@ -556,33 +571,35 @@ int snd_mixart_setup_firmware(mixart_mgr_t *mgr)
 /* miXart hwdep interface id string */
 #define SND_MIXART_HWDEP_ID       "miXart Loader"
 
-static int mixart_hwdep_open(snd_hwdep_t *hw, struct file *file)
+static int mixart_hwdep_open(struct snd_hwdep *hw, struct file *file)
 {
        return 0;
 }
 
-static int mixart_hwdep_release(snd_hwdep_t *hw, struct file *file)
+static int mixart_hwdep_release(struct snd_hwdep *hw, struct file *file)
 {
        return 0;
 }
 
-static int mixart_hwdep_dsp_status(snd_hwdep_t *hw, snd_hwdep_dsp_status_t *info)
+static int mixart_hwdep_dsp_status(struct snd_hwdep *hw,
+                                  struct snd_hwdep_dsp_status *info)
 {
-       mixart_mgr_t *mgr = hw->private_data;
+       struct mixart_mgr *mgr = hw->private_data;
 
        strcpy(info->id, "miXart");
         info->num_dsps = MIXART_HARDW_FILES_MAX_INDEX;
 
-       if (mgr->hwdep->dsp_loaded & (1 <<  MIXART_MOTHERBOARD_ELF_INDEX))
+       if (mgr->dsp_loaded & (1 <<  MIXART_MOTHERBOARD_ELF_INDEX))
                info->chip_ready = 1;
 
        info->version = MIXART_DRIVER_VERSION;
        return 0;
 }
 
-static int mixart_hwdep_dsp_load(snd_hwdep_t *hw, snd_hwdep_dsp_image_t *dsp)
+static int mixart_hwdep_dsp_load(struct snd_hwdep *hw,
+                                struct snd_hwdep_dsp_image *dsp)
 {
-       mixart_mgr_t* mgr = hw->private_data;
+       struct mixart_mgr* mgr = hw->private_data;
        struct firmware fw;
        int err;
 
@@ -599,13 +616,16 @@ static int mixart_hwdep_dsp_load(snd_hwdep_t *hw, snd_hwdep_dsp_image_t *dsp)
        }
        err = mixart_dsp_load(mgr, dsp->index, &fw);
        vfree(fw.data);
+       if (err < 0)
+               return err;
+       mgr->dsp_loaded |= 1 << dsp->index;
        return err;
 }
 
-int snd_mixart_setup_firmware(mixart_mgr_t *mgr)
+int snd_mixart_setup_firmware(struct mixart_mgr *mgr)
 {
        int err;
-       snd_hwdep_t *hw;
+       struct snd_hwdep *hw;
 
        /* only create hwdep interface for first cardX (see "index" module parameter)*/
        if ((err = snd_hwdep_new(mgr->chip[0]->card, SND_MIXART_HWDEP_ID, 0, &hw)) < 0)
@@ -619,8 +639,7 @@ int snd_mixart_setup_firmware(mixart_mgr_t *mgr)
        hw->ops.dsp_load = mixart_hwdep_dsp_load;
        hw->exclusive = 1;
        sprintf(hw->name,  SND_MIXART_HWDEP_ID);
-       mgr->hwdep = hw;
-       mgr->hwdep->dsp_loaded = 0;
+       mgr->dsp_loaded = 0;
 
        return snd_card_register(mgr->chip[0]->card);
 }