[alsa-devel] [PATCH 15/24] sound/pci: coding style fixes: mixart

- <asm/io.h> -> <linux/io.h> - remove trailing whitespaces - convert comments

Only compile tested.

Signed-off-by: Alexander Beregalov a.beregalov@gmail.com ---

sound/pci/mixart/mixart.c | 148 +++++++++++++++++++------------------- sound/pci/mixart/mixart_core.c | 100 +++++++++++++------------- sound/pci/mixart/mixart_hwdep.c | 128 +++++++++++++++++----------------- sound/pci/mixart/mixart_hwdep.h | 28 ++++---- sound/pci/mixart/mixart_mixer.c | 56 +++++++------- 5 files changed, 230 insertions(+), 230 deletions(-)

diff --git a/sound/pci/mixart/mixart.c b/sound/pci/mixart/mixart.c index 2d0dce6..9db4225 100644 --- a/sound/pci/mixart/mixart.c +++ b/sound/pci/mixart/mixart.c @@ -80,10 +80,10 @@ static int mixart_set_pipe_state(struct mixart_mgr *mgr, switch(pipe->status) { case PIPE_RUNNING: case PIPE_CLOCK_SET: - if(start) return 0; /* already started */ + if (start) return 0; /* already started */ break; case PIPE_STOPPED: - if(!start) return 0; /* already stopped */ + if (!start) return 0; /* already stopped */ break; default: snd_printk(KERN_ERR "error mixart_set_pipe_state called with wrong pipe->status!\n"); @@ -100,7 +100,7 @@ static int mixart_set_pipe_state(struct mixart_mgr *mgr, request.size = sizeof(system_msg_uid);

err = snd_mixart_send_msg_wait_notif(mgr, &request, system_msg_uid); - if(err) { + if (err) { snd_printk(KERN_ERR "error : MSG_SYSTEM_WAIT_SYNCHRO_CMD was not notified !\n"); return err; } @@ -111,7 +111,7 @@ static int mixart_set_pipe_state(struct mixart_mgr *mgr, group_state.pipe_count = 1; group_state.pipe_uid[0] = pipe->group_uid;

- if(start) + if (start) request.message_id = MSG_STREAM_START_STREAM_GRP_PACKET; else request.message_id = MSG_STREAM_STOP_STREAM_GRP_PACKET; @@ -126,7 +126,7 @@ static int mixart_set_pipe_state(struct mixart_mgr *mgr, return -EINVAL; }

- if(start) { + if (start) { u32 stat;

group_state.pipe_count = 0; /* in case of start same command once again with pipe_count=0 */ @@ -171,10 +171,10 @@ static int mixart_set_clock(struct mixart_mgr *mgr, case PIPE_CLOCK_SET: break; case PIPE_RUNNING: - if(rate != 0) + if (rate != 0) break; default: - if(rate == 0) + if (rate == 0) return 0; /* nothing to do */ else { snd_printk(KERN_ERR "error mixart_set_clock(%d) called with wrong pipe->status !\n", rate); @@ -202,7 +202,7 @@ static int mixart_set_clock(struct mixart_mgr *mgr, return -EINVAL; }

- if(rate) pipe->status = PIPE_CLOCK_SET; + if (rate) pipe->status = PIPE_CLOCK_SET; else pipe->status = PIPE_RUNNING;

return 0; @@ -220,7 +220,7 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture, struct mixart_pipe *pipe; struct mixart_msg request;

- if(capture) { + if (capture) { if (pcm_number == MIXART_PCM_ANALOG) { pipe = &(chip->pipe_in_ana); /* analog inputs */ } else { @@ -239,12 +239,12 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture, }

/* a new stream is opened and there are already all streams in use */ - if( (monitoring == 0) && (pipe->references >= stream_count) ) { + if ((monitoring == 0) && (pipe->references >= stream_count)) { return NULL; }

/* pipe is not yet defined */ - if( pipe->status == PIPE_UNDEFINED ) { + if (pipe->status == PIPE_UNDEFINED) { int err, i; struct { struct mixart_streaming_group_req sgroup_req; @@ -268,11 +268,11 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture, buf->sgroup_req.latency = 256; buf->sgroup_req.connector = pipe->uid_left_connector; /* the left connector */

- for (i=0; i<stream_count; i++) { + for (i = 0; i < stream_count; i++) { int j; struct mixart_flowinfo *flowinfo; struct mixart_bufferinfo *bufferinfo; - + /* we don't yet know the format, so config 16 bit pcm audio for instance */ buf->sgroup_req.stream_info[i].size_max_byte_frame = 1024; buf->sgroup_req.stream_info[i].size_max_sample_frame = 256; @@ -280,7 +280,7 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture,

/* find the right bufferinfo_array */ j = (chip->chip_idx * MIXART_MAX_STREAM_PER_CARD) + (pcm_number * (MIXART_PLAYBACK_STREAMS + MIXART_CAPTURE_STREAMS)) + i; - if(capture) j += MIXART_PLAYBACK_STREAMS; /* in the array capture is behind playback */ + if (capture) j += MIXART_PLAYBACK_STREAMS; /* in the array capture is behind playback */

buf->sgroup_req.flow_entry[i] = j;

@@ -294,13 +294,13 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture,

/* construct the identifier of the stream buffer received in the interrupts ! */ bufferinfo[j].buffer_id = (chip->chip_idx << MIXART_NOTIFY_CARD_OFFSET) + (pcm_number << MIXART_NOTIFY_PCM_OFFSET ) + i; - if(capture) { + if (capture) { bufferinfo[j].buffer_id |= MIXART_NOTIFY_CAPT_MASK; } }

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(buf->sgroup_resp), &buf->sgroup_resp); - if((err < 0) || (buf->sgroup_resp.status != 0)) { + if ((err < 0) || (buf->sgroup_resp.status != 0)) { snd_printk(KERN_ERR "error MSG_STREAM_ADD_**PUT_GROUP err=%x stat=%x !\n", err, buf->sgroup_resp.status); kfree(buf); return NULL; @@ -314,7 +314,7 @@ snd_mixart_add_ref_pipe(struct snd_mixart *chip, int pcm_number, int capture, kfree(buf); }

- if(monitoring) pipe->monitoring = 1; + if (monitoring) pipe->monitoring = 1; else pipe->references++;

return pipe; @@ -326,28 +326,28 @@ int snd_mixart_kill_ref_pipe(struct mixart_mgr *mgr, { int err = 0;

- if(pipe->status == PIPE_UNDEFINED) + if (pipe->status == PIPE_UNDEFINED) return 0;

- if(monitoring) + if (monitoring) pipe->monitoring = 0; else pipe->references--;

- if((pipe->references <= 0) && (pipe->monitoring == 0)) { + if ((pipe->references <= 0) && (pipe->monitoring == 0)) {

struct mixart_msg request; struct mixart_delete_group_resp delete_resp;

/* release the clock */ err = mixart_set_clock( mgr, pipe, 0); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "mixart_set_clock(0) return error!\n"); }

/* stop the pipe */ err = mixart_set_pipe_state(mgr, pipe, 0); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "error stopping pipe!\n"); }

@@ -376,7 +376,7 @@ static int mixart_set_stream_state(struct mixart_stream *stream, int start) struct mixart_stream_state_req stream_state_req; struct mixart_msg request;

- if(!stream->substream) + if (!stream->substream) return -EINVAL;

memset(&stream_state_req, 0, sizeof(stream_state_req)); @@ -416,7 +416,7 @@ static int snd_mixart_trigger(struct snd_pcm_substream *subs, int cmd) snd_printdd("SNDRV_PCM_TRIGGER_START\n");

/* START_STREAM */ - if( mixart_set_stream_state(stream, 1) ) + if (mixart_set_stream_state(stream, 1)) return -EINVAL;

stream->status = MIXART_STREAM_STATUS_RUNNING; @@ -425,7 +425,7 @@ static int snd_mixart_trigger(struct snd_pcm_substream *subs, int cmd) case SNDRV_PCM_TRIGGER_STOP:

/* STOP_STREAM */ - if( mixart_set_stream_state(stream, 0) ) + if (mixart_set_stream_state(stream, 0)) return -EINVAL;

stream->status = MIXART_STREAM_STATUS_OPEN; @@ -479,12 +479,12 @@ static int snd_mixart_prepare(struct snd_pcm_substream *subs)

/* only the first stream can choose the sample rate */ /* the further opened streams will be limited to its frequency (see open) */ - if(chip->mgr->ref_count_rate == 1) + if (chip->mgr->ref_count_rate == 1) chip->mgr->sample_rate = subs->runtime->rate;

/* set the clock only once (first stream) on the same pipe */ - if(stream->pipe->references == 1) { - if( mixart_set_clock(chip->mgr, stream->pipe, subs->runtime->rate) ) + if (stream->pipe->references == 1) { + if (mixart_set_clock(chip->mgr, stream->pipe, subs->runtime->rate)) return -EINVAL; }

@@ -508,7 +508,7 @@ static int mixart_set_format(struct mixart_stream *stream, snd_pcm_format_t form stream_param.number_of_channel = stream->channels;

stream_param.sampling_freq = chip->mgr->sample_rate; - if(stream_param.sampling_freq == 0) + if (stream_param.sampling_freq == 0) stream_param.sampling_freq = 44100; /* if frequency not yet defined, use some default */

switch(format){ @@ -564,7 +564,7 @@ static int mixart_set_format(struct mixart_stream *stream, snd_pcm_format_t form request.size = sizeof(stream_param);

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(resp), &resp); - if((err < 0) || resp.error_code) { + if ((err < 0) || resp.error_code) { snd_printk(KERN_ERR "MSG_STREAM_SET_INPUT_STAGE_PARAM err=%x; resp=%x\n", err, resp.error_code); return -EINVAL; } @@ -594,9 +594,9 @@ static int snd_mixart_hw_params(struct snd_pcm_substream *subs, mutex_lock(&mgr->setup_mutex);

/* update the stream levels */ - if( stream->pcm_number <= MIXART_PCM_DIGITAL ) { + if (stream->pcm_number <= MIXART_PCM_DIGITAL) { int is_aes = stream->pcm_number > MIXART_PCM_ANALOG; - if( subs->stream == SNDRV_PCM_STREAM_PLAYBACK ) + if (subs->stream == SNDRV_PCM_STREAM_PLAYBACK) mixart_update_playback_stream_level(chip, is_aes, subs->number); else mixart_update_capture_stream_level( chip, is_aes); @@ -606,7 +606,7 @@ static int snd_mixart_hw_params(struct snd_pcm_substream *subs,

/* set the format to the board */ err = mixart_set_format(stream, format); - if(err < 0) { + if (err < 0) { return err; }

@@ -616,10 +616,10 @@ static int snd_mixart_hw_params(struct snd_pcm_substream *subs, if (err > 0) { struct mixart_bufferinfo *bufferinfo; int i = (chip->chip_idx * MIXART_MAX_STREAM_PER_CARD) + (stream->pcm_number * (MIXART_PLAYBACK_STREAMS+MIXART_CAPTURE_STREAMS)) + subs->number; - if( subs->stream == SNDRV_PCM_STREAM_CAPTURE ) { + if (subs->stream == SNDRV_PCM_STREAM_CAPTURE) { i += MIXART_PLAYBACK_STREAMS; /* in array capture is behind playback */ } - + bufferinfo = (struct mixart_bufferinfo *)chip->mgr->bufferinfo.area; bufferinfo[i].buffer_address = subs->runtime->dma_addr; bufferinfo[i].available_length = subs->runtime->dma_bytes; @@ -704,7 +704,7 @@ static int snd_mixart_playback_open(struct snd_pcm_substream *subs)

mutex_lock(&mgr->setup_mutex);

- if ( pcm == chip->pcm ) { + if (pcm == chip->pcm) { pcm_number = MIXART_PCM_ANALOG; runtime->hw = snd_mixart_analog_caps; } else { @@ -734,7 +734,7 @@ static int snd_mixart_playback_open(struct snd_pcm_substream *subs)

/* start the pipe if necessary */ err = mixart_set_pipe_state(chip->mgr, pipe, 1); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "error starting pipe!\n"); snd_mixart_kill_ref_pipe(chip->mgr, pipe, 0); err = -EINVAL; @@ -753,13 +753,13 @@ static int snd_mixart_playback_open(struct snd_pcm_substream *subs) snd_pcm_hw_constraint_step(runtime, 0, SNDRV_PCM_HW_PARAM_BUFFER_SIZE, 64);

/* if a sample rate is already used, another stream cannot change */ - if(mgr->ref_count_rate++) { - if(mgr->sample_rate) { + if (mgr->ref_count_rate++) { + if (mgr->sample_rate) { runtime->hw.rate_min = runtime->hw.rate_max = mgr->sample_rate; } }

- _exit_open: +_exit_open: mutex_unlock(&mgr->setup_mutex);

return err; @@ -779,7 +779,7 @@ static int snd_mixart_capture_open(struct snd_pcm_substream *subs)

mutex_lock(&mgr->setup_mutex);

- if ( pcm == chip->pcm ) { + if (pcm == chip->pcm) { pcm_number = MIXART_PCM_ANALOG; runtime->hw = snd_mixart_analog_caps; } else { @@ -812,7 +812,7 @@ static int snd_mixart_capture_open(struct snd_pcm_substream *subs)

/* start the pipe if necessary */ err = mixart_set_pipe_state(chip->mgr, pipe, 1); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "error starting pipe!\n"); snd_mixart_kill_ref_pipe(chip->mgr, pipe, 0); err = -EINVAL; @@ -831,13 +831,13 @@ static int snd_mixart_capture_open(struct snd_pcm_substream *subs) snd_pcm_hw_constraint_step(runtime, 0, SNDRV_PCM_HW_PARAM_BUFFER_SIZE, 64);

/* if a sample rate is already used, another stream cannot change */ - if(mgr->ref_count_rate++) { - if(mgr->sample_rate) { + if (mgr->ref_count_rate++) { + if (mgr->sample_rate) { runtime->hw.rate_min = runtime->hw.rate_max = mgr->sample_rate; } }

- _exit_open: +_exit_open: mutex_unlock(&mgr->setup_mutex);

return err; @@ -856,7 +856,7 @@ static int snd_mixart_close(struct snd_pcm_substream *subs) snd_printdd("snd_mixart_close C%d/P%d/Sub%d\n", chip->chip_idx, stream->pcm_number, subs->number);

/* sample rate released */ - if(--mgr->ref_count_rate == 0) { + if (--mgr->ref_count_rate == 0) { mgr->sample_rate = 0; }

@@ -1011,7 +1011,7 @@ static int __devinit snd_mixart_create(struct mixart_mgr *mgr, struct snd_card * };

mgr->chip[idx] = chip = kzalloc(sizeof(*chip), GFP_KERNEL); - if (! chip) { + if (!chip) { snd_printk(KERN_ERR "cannot allocate chip\n"); return -ENOMEM; } @@ -1038,7 +1038,7 @@ int snd_mixart_create_pcm(struct snd_mixart* chip) if (err < 0) return err;

- if(chip->mgr->board_type == MIXART_DAUGHTER_TYPE_AES) { + if (chip->mgr->board_type == MIXART_DAUGHTER_TYPE_AES) {

err = snd_mixart_pcm_digital(chip); if (err < 0) @@ -1068,7 +1068,7 @@ static int snd_mixart_free(struct mixart_mgr *mgr) free_irq(mgr->irq, mgr);

/* reset board if some firmware was loaded */ - if(mgr->dsp_loaded) { + if (mgr->dsp_loaded) { snd_mixart_reset_board(mgr); snd_printdd("reset miXart !\n"); } @@ -1081,12 +1081,12 @@ static int snd_mixart_free(struct mixart_mgr *mgr) pci_release_regions(mgr->pci);

/* free flowarray */ - if(mgr->flowinfo.area) { + if (mgr->flowinfo.area) { snd_dma_free_pages(&mgr->flowinfo); mgr->flowinfo.area = NULL; } /* free bufferarray */ - if(mgr->bufferinfo.area) { + if (mgr->bufferinfo.area) { snd_dma_free_pages(&mgr->bufferinfo); mgr->bufferinfo.area = NULL; } @@ -1120,7 +1120,7 @@ static long long snd_mixart_BA0_llseek(struct snd_info_entry *entry, default: return -EINVAL; } - if(file->f_pos > MIXART_BA0_SIZE) + if (file->f_pos > MIXART_BA0_SIZE) file->f_pos = MIXART_BA0_SIZE; return file->f_pos; } @@ -1146,7 +1146,7 @@ static long long snd_mixart_BA1_llseek(struct snd_info_entry *entry, default: return -EINVAL; } - if(file->f_pos > MIXART_BA1_SIZE) + if (file->f_pos > MIXART_BA1_SIZE) file->f_pos = MIXART_BA1_SIZE; return file->f_pos; } @@ -1161,11 +1161,11 @@ static long snd_mixart_BA0_read(struct snd_info_entry *entry, void *file_private struct mixart_mgr *mgr = entry->private_data;

count = count & ~3; /* make sure the read size is a multiple of 4 bytes */ - if(count <= 0) + if (count <= 0) return 0; - if(pos + count > MIXART_BA0_SIZE) + if (pos + count > MIXART_BA0_SIZE) count = (long)(MIXART_BA0_SIZE - pos); - if(copy_to_user_fromio(buf, MIXART_MEM( mgr, pos ), count)) + if (copy_to_user_fromio(buf, MIXART_MEM( mgr, pos ), count)) return -EFAULT; return count; } @@ -1180,11 +1180,11 @@ static long snd_mixart_BA1_read(struct snd_info_entry *entry, void *file_private struct mixart_mgr *mgr = entry->private_data;

count = count & ~3; /* make sure the read size is a multiple of 4 bytes */ - if(count <= 0) + if (count <= 0) return 0; - if(pos + count > MIXART_BA1_SIZE) + if (pos + count > MIXART_BA1_SIZE) count = (long)(MIXART_BA1_SIZE - pos); - if(copy_to_user_fromio(buf, MIXART_REG( mgr, pos ), count)) + if (copy_to_user_fromio(buf, MIXART_REG( mgr, pos ), count)) return -EFAULT; return count; } @@ -1200,25 +1200,25 @@ static struct snd_info_entry_ops snd_mixart_proc_ops_BA1 = { };

-static void snd_mixart_proc_read(struct snd_info_entry *entry, +static void snd_mixart_proc_read(struct snd_info_entry *entry, struct snd_info_buffer *buffer) { - struct snd_mixart *chip = entry->private_data; - u32 ref; + struct snd_mixart *chip = entry->private_data; + u32 ref;

snd_iprintf(buffer, "Digigram miXart (alsa card %d)\n\n", chip->chip_idx);

/* stats available when embedded OS is running */ if (chip->mgr->dsp_loaded & ( 1 << MIXART_MOTHERBOARD_ELF_INDEX)) { snd_iprintf(buffer, "- hardware -\n"); - switch (chip->mgr->board_type ) { + switch (chip->mgr->board_type) { case MIXART_DAUGHTER_TYPE_NONE : snd_iprintf(buffer, "\tmiXart8 (no daughter board)\n\n"); break; case MIXART_DAUGHTER_TYPE_AES : snd_iprintf(buffer, "\tmiXart8 AES/EBU\n\n"); break; case MIXART_DAUGHTER_TYPE_COBRANET : snd_iprintf(buffer, "\tmiXart8 Cobranet\n\n"); break; default: snd_iprintf(buffer, "\tUNKNOWN!\n\n"); break; }

- snd_iprintf(buffer, "- system load -\n"); + snd_iprintf(buffer, "- system load -\n");

/* get perf reference */

@@ -1241,18 +1241,18 @@ static void __devinit snd_mixart_proc_init(struct snd_mixart *chip) struct snd_info_entry *entry;

/* text interface to read perf and temp meters */ - if (! snd_card_proc_new(chip->card, "board_info", &entry)) { + if (!snd_card_proc_new(chip->card, "board_info", &entry)) { entry->private_data = chip; entry->c.text.read = snd_mixart_proc_read; }

- if (! snd_card_proc_new(chip->card, "mixart_BA0", &entry)) { + if (!snd_card_proc_new(chip->card, "mixart_BA0", &entry)) { entry->content = SNDRV_INFO_CONTENT_DATA; - entry->private_data = chip->mgr; + entry->private_data = chip->mgr; entry->c.ops = &snd_mixart_proc_ops_BA0; entry->size = MIXART_BA0_SIZE; } - if (! snd_card_proc_new(chip->card, "mixart_BA1", &entry)) { + if (!snd_card_proc_new(chip->card, "mixart_BA1", &entry)) { entry->content = SNDRV_INFO_CONTENT_DATA; entry->private_data = chip->mgr; entry->c.ops = &snd_mixart_proc_ops_BA1; @@ -1278,7 +1278,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci, */ if (dev >= SNDRV_CARDS) return -ENODEV; - if (! enable[dev]) { + if (!enable[dev]) { dev++; return -ENOENT; } @@ -1298,7 +1298,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci, /* */ mgr = kzalloc(sizeof(*mgr), GFP_KERNEL); - if (! mgr) { + if (!mgr) { pci_disable_device(pci); return -ENOMEM; } @@ -1367,7 +1367,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci, snprintf(tmpid, sizeof(tmpid), "%s-%d", id[dev] ? id[dev] : "MIXART", i); card = snd_card_new(idx, tmpid, THIS_MODULE, 0);

- if (! card) { + if (!card) { snd_printk(KERN_ERR "cannot allocate the card %d\n", i); snd_mixart_free(mgr); return -ENOMEM; @@ -1382,7 +1382,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci, return err; }

- if(i==0) { + if (i==0) { /* init proc interface only for chip0 */ snd_mixart_proc_init(mgr->chip[i]); } @@ -1398,7 +1398,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci,

/* create array of streaminfo */ size = PAGE_ALIGN( (MIXART_MAX_STREAM_PER_CARD * MIXART_MAX_CARDS * - sizeof(struct mixart_flowinfo)) ); + sizeof(struct mixart_flowinfo))); if (snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, snd_dma_pci_data(pci), size, &mgr->flowinfo) < 0) { snd_mixart_free(mgr); @@ -1409,7 +1409,7 @@ static int __devinit snd_mixart_probe(struct pci_dev *pci,

/* create array of bufferinfo */ size = PAGE_ALIGN( (MIXART_MAX_STREAM_PER_CARD * MIXART_MAX_CARDS * - sizeof(struct mixart_bufferinfo)) ); + sizeof(struct mixart_bufferinfo))); if (snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, snd_dma_pci_data(pci), size, &mgr->bufferinfo) < 0) { snd_mixart_free(mgr); diff --git a/sound/pci/mixart/mixart_core.c b/sound/pci/mixart/mixart_core.c index b9a06c2..26b5f46 100644 --- a/sound/pci/mixart/mixart_core.c +++ b/sound/pci/mixart/mixart_core.c @@ -23,7 +23,7 @@ #include <linux/interrupt.h> #include <linux/mutex.h>

-#include <asm/io.h> +#include <linux/io.h> #include <sound/core.h> #include "mixart.h" #include "mixart_hwdep.h" @@ -65,7 +65,7 @@ static int retrieve_msg_frame(struct mixart_mgr *mgr, u32 *msg_frame)

/* increment the tail index */ tailptr += 4; - if( tailptr >= (MSG_OUTBOUND_POST_STACK+MSG_BOUND_STACK_SIZE) ) + if (tailptr >= (MSG_OUTBOUND_POST_STACK+MSG_BOUND_STACK_SIZE)) tailptr = MSG_OUTBOUND_POST_STACK; writel_be(tailptr, MIXART_MEM(mgr, MSG_OUTBOUND_POST_TAIL));

@@ -73,7 +73,7 @@ static int retrieve_msg_frame(struct mixart_mgr *mgr, u32 *msg_frame) }

static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp, - u32 msg_frame_address ) + u32 msg_frame_address) { unsigned long flags; u32 headptr; @@ -92,7 +92,7 @@ static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp, resp->uid.object_id = readl_be(MIXART_MEM(mgr, msg_frame_address + 8)); /* uidDest */ resp->uid.desc = readl_be(MIXART_MEM(mgr, msg_frame_address + 12)); /* */

- if( (size < MSG_DESCRIPTOR_SIZE) || (resp->size < (size - MSG_DESCRIPTOR_SIZE))) { + if ((size < MSG_DESCRIPTOR_SIZE) || (resp->size < (size - MSG_DESCRIPTOR_SIZE))) { err = -EINVAL; snd_printk(KERN_ERR "problem with response size = %d\n", size); goto _clean_exit; @@ -105,7 +105,7 @@ static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp, /* swap if necessary */ #ifndef __BIG_ENDIAN size /= 4; /* u32 size */ - for(i=0; i < size; i++) { + for (i = 0; i < size; i++) { ((u32*)resp->data)[i] = be32_to_cpu(((u32*)resp->data)[i]); } #endif @@ -115,7 +115,7 @@ static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp, */ headptr = readl_be(MIXART_MEM(mgr, MSG_OUTBOUND_FREE_HEAD));

- if( (headptr < MSG_OUTBOUND_FREE_STACK) || ( headptr >= (MSG_OUTBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE))) { + if ((headptr < MSG_OUTBOUND_FREE_STACK) || ( headptr >= (MSG_OUTBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE))) { err = -EINVAL; goto _clean_exit; } @@ -125,12 +125,12 @@ static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp,

/* increment the outbound free head */ headptr += 4; - if( headptr >= (MSG_OUTBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE) ) + if (headptr >= (MSG_OUTBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE)) headptr = MSG_OUTBOUND_FREE_STACK;

writel_be(headptr, MIXART_MEM(mgr, MSG_OUTBOUND_FREE_HEAD));

- _clean_exit: +_clean_exit: spin_unlock_irqrestore(&mgr->msg_lock, flags);

return err; @@ -141,7 +141,7 @@ static int get_msg(struct mixart_mgr *mgr, struct mixart_msg *resp, * send a message to miXart. return: the msg_frame used for this message */ /* call with mgr->msg_lock held! */ -static int send_msg( struct mixart_mgr *mgr, +static int send_msg(struct mixart_mgr *mgr, struct mixart_msg *msg, int max_answersize, int mark_pending, @@ -165,7 +165,7 @@ static int send_msg( struct mixart_mgr *mgr, return -EBUSY; }

- if( (tailptr < MSG_INBOUND_FREE_STACK) || (tailptr >= (MSG_INBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE))) { + if ((tailptr < MSG_INBOUND_FREE_STACK) || (tailptr >= (MSG_INBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE))) { return -EINVAL; }

@@ -174,7 +174,7 @@ static int send_msg( struct mixart_mgr *mgr,

/* increment the inbound free tail */ tailptr += 4; - if( tailptr >= (MSG_INBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE) ) + if (tailptr >= (MSG_INBOUND_FREE_STACK+MSG_BOUND_STACK_SIZE)) tailptr = MSG_INBOUND_FREE_STACK;

writel_be(tailptr, MIXART_MEM(mgr, MSG_INBOUND_FREE_TAIL)); @@ -182,24 +182,24 @@ static int send_msg( struct mixart_mgr *mgr, /* TODO : use memcpy_toio() with intermediate buffer to copy the message */

/* copy message descriptor to card memory */ - writel_be( msg->size + MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address) ); /* size of descriptor + request */ - writel_be( msg->message_id , MIXART_MEM(mgr, msg_frame_address + 4) ); /* dwMessageID */ - writel_be( msg->uid.object_id, MIXART_MEM(mgr, msg_frame_address + 8) ); /* uidDest */ - writel_be( msg->uid.desc, MIXART_MEM(mgr, msg_frame_address + 12) ); /* */ - writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 16) ); /* SizeHeader */ - writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 20) ); /* OffsetDLL_T16 */ - writel_be( msg->size, MIXART_MEM(mgr, msg_frame_address + 24) ); /* SizeDLL_T16 */ - writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 28) ); /* OffsetDLL_DRV */ - writel_be( 0, MIXART_MEM(mgr, msg_frame_address + 32) ); /* SizeDLL_DRV */ - writel_be( MSG_DESCRIPTOR_SIZE + max_answersize, MIXART_MEM(mgr, msg_frame_address + 36) ); /* dwExpectedAnswerSize */ + writel_be( msg->size + MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address)); /* size of descriptor + request */ + writel_be( msg->message_id , MIXART_MEM(mgr, msg_frame_address + 4)); /* dwMessageID */ + writel_be( msg->uid.object_id, MIXART_MEM(mgr, msg_frame_address + 8)); /* uidDest */ + writel_be( msg->uid.desc, MIXART_MEM(mgr, msg_frame_address + 12)); /* */ + writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 16)); /* SizeHeader */ + writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 20)); /* OffsetDLL_T16 */ + writel_be( msg->size, MIXART_MEM(mgr, msg_frame_address + 24)); /* SizeDLL_T16 */ + writel_be( MSG_DESCRIPTOR_SIZE, MIXART_MEM(mgr, msg_frame_address + 28)); /* OffsetDLL_DRV */ + writel_be( 0, MIXART_MEM(mgr, msg_frame_address + 32)); /* SizeDLL_DRV */ + writel_be( MSG_DESCRIPTOR_SIZE + max_answersize, MIXART_MEM(mgr, msg_frame_address + 36)); /* dwExpectedAnswerSize */

/* copy message data to card memory */ - for( i=0; i < msg->size; i+=4 ) { - writel_be( *(u32*)(msg->data + i), MIXART_MEM(mgr, MSG_HEADER_SIZE + msg_frame_address + i) ); + for (i = 0; i < msg->size; i += 4) { + writel_be( *(u32*)(msg->data + i), MIXART_MEM(mgr, MSG_HEADER_SIZE + msg_frame_address + i) ); }

- if( mark_pending ) { - if( *msg_event ) { + if (mark_pending) { + if (*msg_event) { /* the pending event is the notification we wait for ! */ mgr->pending_event = *msg_event; } @@ -218,7 +218,7 @@ static int send_msg( struct mixart_mgr *mgr, /* post the frame */ headptr = readl_be(MIXART_MEM(mgr, MSG_INBOUND_POST_HEAD));

- if( (headptr < MSG_INBOUND_POST_STACK) || (headptr >= (MSG_INBOUND_POST_STACK+MSG_BOUND_STACK_SIZE))) { + if ((headptr < MSG_INBOUND_POST_STACK) || (headptr >= (MSG_INBOUND_POST_STACK+MSG_BOUND_STACK_SIZE))) { return -EINVAL; }

@@ -226,7 +226,7 @@ static int send_msg( struct mixart_mgr *mgr,

/* increment the inbound post head */ headptr += 4; - if( headptr >= (MSG_INBOUND_POST_STACK+MSG_BOUND_STACK_SIZE) ) + if (headptr >= (MSG_INBOUND_POST_STACK+MSG_BOUND_STACK_SIZE)) headptr = MSG_INBOUND_POST_STACK;

writel_be(headptr, MIXART_MEM(mgr, MSG_INBOUND_POST_HEAD)); @@ -262,7 +262,7 @@ int snd_mixart_send_msg(struct mixart_mgr *mgr, struct mixart_msg *request, int timeout = schedule_timeout(MSG_TIMEOUT_JIFFIES); remove_wait_queue(&mgr->msg_sleep, &wait);

- if (! timeout) { + if (!timeout) { /* error - no ack */ mutex_unlock(&mgr->msg_mutex); snd_printk(KERN_ERR "error: no reponse on msg %x\n", msg_frame); @@ -277,7 +277,7 @@ int snd_mixart_send_msg(struct mixart_mgr *mgr, struct mixart_msg *request, int

err = get_msg(mgr, &resp, msg_frame);

- if( request->message_id != resp.message_id ) + if (request->message_id != resp.message_id) snd_printk(KERN_ERR "REPONSE ERROR!\n");

mutex_unlock(&mgr->msg_mutex); @@ -306,7 +306,7 @@ int snd_mixart_send_msg_wait_notif(struct mixart_mgr *mgr, spin_lock_irq(&mgr->msg_lock); /* send the message */ err = send_msg(mgr, request, MSG_DEFAULT_SIZE, 1, &notif_event); /* send and mark the notification event pending */ - if(err) { + if (err) { spin_unlock_irq(&mgr->msg_lock); mutex_unlock(&mgr->msg_mutex); return err; @@ -318,7 +318,7 @@ int snd_mixart_send_msg_wait_notif(struct mixart_mgr *mgr, timeout = schedule_timeout(MSG_TIMEOUT_JIFFIES); remove_wait_queue(&mgr->msg_sleep, &wait);

- if (! timeout) { + if (!timeout) { /* error - no ack */ mutex_unlock(&mgr->msg_mutex); snd_printk(KERN_ERR "error: notification %x not received\n", notif_event); @@ -354,7 +354,7 @@ static u32 mixart_msg_data[MSG_DEFAULT_SIZE / 4];

void snd_mixart_msg_tasklet(unsigned long arg) { - struct mixart_mgr *mgr = ( struct mixart_mgr*)(arg); + struct mixart_mgr *mgr = (struct mixart_mgr*)(arg); struct mixart_msg resp; u32 msg, addr, type; int err; @@ -377,7 +377,7 @@ void snd_mixart_msg_tasklet(unsigned long arg) resp.data = mixart_msg_data; resp.size = sizeof(mixart_msg_data); err = get_msg(mgr, &resp, addr); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "tasklet: error(%d) reading mf %x\n", err, msg); break; } @@ -387,7 +387,7 @@ void snd_mixart_msg_tasklet(unsigned long arg) case MSG_STREAM_START_OUTPUT_STAGE_PACKET: case MSG_STREAM_STOP_INPUT_STAGE_PACKET: case MSG_STREAM_STOP_OUTPUT_STAGE_PACKET: - if(mixart_msg_data[0]) + if (mixart_msg_data[0]) snd_printk(KERN_ERR "tasklet : error MSG_STREAM_ST***_***PUT_STAGE_PACKET status=%x\n", mixart_msg_data[0]); break; default: @@ -425,7 +425,7 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) spin_lock(&mgr->lock);

it_reg = readl_le(MIXART_REG(mgr, MIXART_PCI_OMISR_OFFSET)); - if( !(it_reg & MIXART_OIDI) ) { + if (!(it_reg & MIXART_OIDI)) { /* this device did not cause the interrupt */ spin_unlock(&mgr->lock); return IRQ_NONE; @@ -439,7 +439,7 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) writel(it_reg, MIXART_REG(mgr, MIXART_PCI_ODBR_OFFSET));

/* clear interrupt */ - writel_le( MIXART_OIDI, MIXART_REG(mgr, MIXART_PCI_OMISR_OFFSET) ); + writel_le( MIXART_OIDI, MIXART_REG(mgr, MIXART_PCI_OMISR_OFFSET));

/* process interrupt */ while (retrieve_msg_frame(mgr, &msg)) { @@ -450,17 +450,17 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) resp.data = mixart_msg_data; resp.size = sizeof(mixart_msg_data); err = get_msg(mgr, &resp, msg & ~MSG_TYPE_MASK); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "interrupt: error(%d) reading mf %x\n", err, msg); break; }

- if(resp.message_id == MSG_SERVICES_TIMER_NOTIFY) { + if (resp.message_id == MSG_SERVICES_TIMER_NOTIFY) { int i; struct mixart_timer_notify *notify; notify = (struct mixart_timer_notify *)mixart_msg_data;

- for(i=0; i<notify->stream_count; i++) { + for (i = 0; i < notify->stream_count; i++) {

u32 buffer_id = notify->streams[i].buffer_id; unsigned int chip_number = (buffer_id & MIXART_NOTIFY_CARD_MASK) >> MIXART_NOTIFY_CARD_OFFSET; /* card0 to 3 */ @@ -503,9 +503,9 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) stream->abs_period_elapsed = new_elapse_pos; } } - stream->buf_period_frag = (u32)( sample_count - stream->abs_period_elapsed ); + stream->buf_period_frag = (u32)(sample_count - stream->abs_period_elapsed);

- if(elapsed) { + if (elapsed) { spin_unlock(&mgr->lock); snd_pcm_period_elapsed(stream->substream); spin_lock(&mgr->lock); @@ -514,12 +514,12 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) } break; } - if(resp.message_id == MSG_SERVICES_REPORT_TRACES) { - if(resp.size > 1) { + if (resp.message_id == MSG_SERVICES_REPORT_TRACES) { + if (resp.size > 1) { #ifndef __BIG_ENDIAN /* Traces are text: the swapped msg_data has to be swapped back ! */ int i; - for(i=0; i<(resp.size/4); i++) { + for (i = 0; i < (resp.size/4); i++) { (mixart_msg_data)[i] = cpu_to_be32((mixart_msg_data)[i]); } #endif @@ -533,7 +533,7 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) break;

case MSG_TYPE_NOTIFY: - if(msg & MSG_CANCEL_NOTIFY_MASK) { + if (msg & MSG_CANCEL_NOTIFY_MASK) { msg &= ~MSG_CANCEL_NOTIFY_MASK; snd_printk(KERN_ERR "canceled notification %x !\n", msg); } @@ -541,7 +541,7 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id) case MSG_TYPE_ANSWER: /* answer or notification to a message we are waiting for*/ spin_lock(&mgr->msg_lock); - if( (msg & ~MSG_TYPE_MASK) == mgr->pending_event ) { + if ((msg & ~MSG_TYPE_MASK) == mgr->pending_event) { wake_up(&mgr->msg_sleep); mgr->pending_event = 0; } @@ -573,11 +573,11 @@ irqreturn_t snd_mixart_interrupt(int irq, void *dev_id)

void snd_mixart_init_mailbox(struct mixart_mgr *mgr) { - writel( 0, MIXART_MEM( mgr, MSG_HOST_RSC_PROTECTION ) ); - writel( 0, MIXART_MEM( mgr, MSG_AGENT_RSC_PROTECTION ) ); + writel( 0, MIXART_MEM( mgr, MSG_HOST_RSC_PROTECTION )); + writel( 0, MIXART_MEM( mgr, MSG_AGENT_RSC_PROTECTION ));

/* allow outbound messagebox to generate interrupts */ - if(mgr->irq >= 0) { + if (mgr->irq >= 0) { writel_le( MIXART_ALLOW_OUTBOUND_DOORBELL, MIXART_REG( mgr, MIXART_PCI_OMIMR_OFFSET)); } return; @@ -593,6 +593,6 @@ void snd_mixart_exit_mailbox(struct mixart_mgr *mgr) void snd_mixart_reset_board(struct mixart_mgr *mgr) { /* reset miXart */ - writel_be( 1, MIXART_REG(mgr, MIXART_BA1_BRUTAL_RESET_OFFSET) ); + writel_be( 1, MIXART_REG(mgr, MIXART_BA1_BRUTAL_RESET_OFFSET)); return; } diff --git a/sound/pci/mixart/mixart_hwdep.c b/sound/pci/mixart/mixart_hwdep.c index 3782b52..18639aa 100644 --- a/sound/pci/mixart/mixart_hwdep.c +++ b/sound/pci/mixart/mixart_hwdep.c @@ -24,7 +24,7 @@ #include <linux/pci.h> #include <linux/firmware.h> #include <linux/vmalloc.h> -#include <asm/io.h> +#include <linux/io.h> #include <sound/core.h> #include "mixart.h" #include "mixart_mixer.h" @@ -53,20 +53,20 @@ static int mixart_wait_nice_for_register_value(struct mixart_mgr *mgr, cond_resched();

read = readl_be( MIXART_MEM( mgr, offset )); - if(is_egal) { - if(read == value) return 0; + if (is_egal) { + if (read == value) return 0; } else { /* wait for different value */ - if(read != value) return 0; + if (read != value) return 0; } - } while ( time_after_eq(end_time, jiffies) ); + } while ( time_after_eq(end_time, jiffies));

return -EBUSY; }

/* - structures needed to upload elf code packets + structures needed to upload elf code packets */ struct snd_mixart_elf32_ehdr { u8 e_ident[16]; @@ -96,27 +96,27 @@ struct snd_mixart_elf32_phdr { u32 p_align; };

-static int mixart_load_elf(struct mixart_mgr *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'}; struct snd_mixart_elf32_ehdr *elf_header; int i;

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] ) + for (i = 0; i < 4; i++) + if (elf32_magic_number[i] != elf_header->e_ident[i]) return -EINVAL;

- if( elf_header->e_phoff != 0 ) { + if (elf_header->e_phoff != 0) { struct snd_mixart_elf32_phdr elf_programheader;

- for( i=0; i < be16_to_cpu(elf_header->e_phnum); i++ ) { + 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));

- memcpy( &elf_programheader, dsp->data + pos, sizeof(elf_programheader) ); + memcpy( &elf_programheader, dsp->data + pos, sizeof(elf_programheader));

- if(elf_programheader.p_type != 0) { - if( elf_programheader.p_filesz != 0 ) { + if (elf_programheader.p_type != 0) { + if (elf_programheader.p_filesz != 0) { memcpy_toio( MIXART_MEM( mgr, be32_to_cpu(elf_programheader.p_vaddr)), dsp->data + be32_to_cpu( elf_programheader.p_offset ), be32_to_cpu( elf_programheader.p_filesz )); @@ -147,7 +147,7 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) 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) { + if (!connector || !audio_info_req || !audio_info) { err = -ENOMEM; goto __error; } @@ -162,21 +162,21 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) 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)) { + if ((err < 0) || (connector->error_code) || (connector->uid_count > MIXART_MAX_PHYS_CONNECTORS)) { snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_PLAY_CONNECTOR\n"); err = -EINVAL; goto __error; }

- for(k=0; k < connector->uid_count; k++) { + for (k=0; k < connector->uid_count; k++) { struct mixart_pipe *pipe;

- if(k < MIXART_FIRST_DIG_AUDIO_ID) { + if (k < MIXART_FIRST_DIG_AUDIO_ID) { pipe = &mgr->chip[k/2]->pipe_out_ana; } else { pipe = &mgr->chip[(k-MIXART_FIRST_DIG_AUDIO_ID)/2]->pipe_out_dig; } - if(k & 1) { + if (k & 1) { pipe->uid_right_connector = connector->uid[k]; /* odd */ } else { pipe->uid_left_connector = connector->uid[k]; /* even */ @@ -191,7 +191,7 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) request.size = sizeof(*audio_info_req);

err = snd_mixart_send_msg(mgr, &request, sizeof(*audio_info), audio_info); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "error MSG_CONNECTOR_GET_AUDIO_INFO\n"); goto __error; } @@ -204,21 +204,21 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) 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)) { + if ((err < 0) || (connector->error_code) || (connector->uid_count > MIXART_MAX_PHYS_CONNECTORS)) { snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_RECORD_CONNECTOR\n"); err = -EINVAL; goto __error; }

- for(k=0; k < connector->uid_count; k++) { + for (k=0; k < connector->uid_count; k++) { struct mixart_pipe *pipe;

- if(k < MIXART_FIRST_DIG_AUDIO_ID) { + if (k < MIXART_FIRST_DIG_AUDIO_ID) { pipe = &mgr->chip[k/2]->pipe_in_ana; } else { pipe = &mgr->chip[(k-MIXART_FIRST_DIG_AUDIO_ID)/2]->pipe_in_dig; } - if(k & 1) { + if (k & 1) { pipe->uid_right_connector = connector->uid[k]; /* odd */ } else { pipe->uid_left_connector = connector->uid[k]; /* even */ @@ -233,7 +233,7 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) request.size = sizeof(*audio_info_req);

err = snd_mixart_send_msg(mgr, &request, sizeof(*audio_info), audio_info); - if( err < 0 ) { + if (err < 0) { snd_printk(KERN_ERR "error MSG_CONNECTOR_GET_AUDIO_INFO\n"); goto __error; } @@ -241,7 +241,7 @@ static int mixart_enum_connectors(struct mixart_mgr *mgr) } err = 0;

- __error: +__error: kfree(connector); kfree(audio_info_req); kfree(audio_info); @@ -269,7 +269,7 @@ static int mixart_enum_physio(struct mixart_mgr *mgr)

err = snd_mixart_send_msg(mgr, &request, sizeof(console_mgr), &console_mgr);

- if( (err < 0) || (console_mgr.error_code != 0) ) { + if ((err < 0) || (console_mgr.error_code != 0)) { snd_printk(KERN_DEBUG "error MSG_CONSOLE_GET_CLOCK_UID : err=%x\n", console_mgr.error_code); return -EINVAL; } @@ -283,8 +283,8 @@ static int mixart_enum_physio(struct mixart_mgr *mgr) request.size = sizeof(console_mgr.uid);

err = snd_mixart_send_msg(mgr, &request, sizeof(phys_io), &phys_io); - if( (err < 0) || ( phys_io.error_code != 0 ) ) { - snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_PHYSICAL_IO err(%x) error_code(%x)\n", err, phys_io.error_code ); + if ((err < 0) || ( phys_io.error_code != 0 )) { + snd_printk(KERN_ERR "error MSG_SYSTEM_ENUM_PHYSICAL_IO err(%x) error_code(%x)\n", err, phys_io.error_code); return -EINVAL; }

@@ -292,9 +292,9 @@ static int mixart_enum_physio(struct mixart_mgr *mgr) if (phys_io.nb_uid < MIXART_MAX_CARDS * 2) return -EINVAL;

- for(k=0; k<mgr->num_cards; k++) { + for (k=0; k<mgr->num_cards; k++) { mgr->chip[k]->uid_in_analog_physio = phys_io.uid[k]; - mgr->chip[k]->uid_out_analog_physio = phys_io.uid[phys_io.nb_uid/2 + k]; + mgr->chip[k]->uid_out_analog_physio = phys_io.uid[phys_io.nb_uid/2 + k]; }

return 0; @@ -307,9 +307,9 @@ static int mixart_first_init(struct mixart_mgr *mgr) int err; struct mixart_msg request;

- if((err = mixart_enum_connectors(mgr)) < 0) return err; + if ((err = mixart_enum_connectors(mgr)) < 0) return err;

- if((err = mixart_enum_physio(mgr)) < 0) return err; + if ((err = mixart_enum_physio(mgr)) < 0) return err;

/* send a synchro command to card (necessary to do this before first MSG_STREAM_START_STREAM_GRP_PACKET) */ /* though why not here */ @@ -319,7 +319,7 @@ static int mixart_first_init(struct mixart_mgr *mgr) request.size = 0; /* this command has no data. response is a 32 bit status */ err = snd_mixart_send_msg(mgr, &request, sizeof(k), &k); - if( (err < 0) || (k != 0) ) { + if ((err < 0) || (k != 0)) { snd_printk(KERN_ERR "error MSG_SYSTEM_SEND_SYNCHRO_CMD\n"); return err == 0 ? -EINVAL : err; } @@ -345,22 +345,22 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw status_daught = readl_be( MIXART_MEM( mgr,MIXART_PSEUDOREG_DXLX_STATUS_OFFSET ));

/* motherboard xilinx status 5 will say that the board is performing a reset */ - if( status_xilinx == 5 ) { - snd_printk( KERN_ERR "miXart is resetting !\n"); + if (status_xilinx == 5) { + snd_printk(KERN_ERR "miXart is resetting !\n"); return -EAGAIN; /* try again later */ }

switch (index) { case MIXART_MOTHERBOARD_XLX_INDEX:

- /* xilinx already loaded ? */ - if( status_xilinx == 4 ) { - snd_printk( KERN_DEBUG "xilinx is already loaded !\n"); + /* xilinx already loaded ? */ + if (status_xilinx == 4) { + snd_printk(KERN_DEBUG "xilinx is already loaded !\n"); return 0; } /* the status should be 0 == "idle" */ - if( status_xilinx != 0 ) { - snd_printk( KERN_ERR "xilinx load error ! status = %d\n", status_xilinx); + if (status_xilinx != 0) { + snd_printk(KERN_ERR "xilinx load error ! status = %d\n", status_xilinx); return -EIO; /* modprob -r may help ? */ }

@@ -380,7 +380,7 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw

/* copy xilinx code */ memcpy_toio( MIXART_MEM( mgr, MIXART_MOTHERBOARD_XLX_BASE_ADDRESS), dsp->data, dsp->size); - + /* set xilinx status to copy finished */ writel_be( 2, MIXART_MEM( mgr, MIXART_PSEUDOREG_MXLX_STATUS_OFFSET ));

@@ -389,33 +389,33 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw

case MIXART_MOTHERBOARD_ELF_INDEX:

- if( status_elf == 4 ) { - snd_printk( KERN_DEBUG "elf file already loaded !\n"); + if (status_elf == 4) { + snd_printk(KERN_DEBUG "elf file already loaded !\n"); return 0; }

/* the status should be 0 == "idle" */ - if( status_elf != 0 ) { - snd_printk( KERN_ERR "elf load error ! status = %d\n", status_elf); + if (status_elf != 0) { + snd_printk(KERN_ERR "elf load error ! status = %d\n", status_elf); return -EIO; /* modprob -r may help ? */ }

/* wait for xilinx status == 4 */ err = mixart_wait_nice_for_register_value( mgr, MIXART_PSEUDOREG_MXLX_STATUS_OFFSET, 1, 4, 500); /* 5sec */ if (err < 0) { - snd_printk( KERN_ERR "xilinx was not loaded or could not be started\n"); + snd_printk(KERN_ERR "xilinx was not loaded or could not be started\n"); return err; }

/* init some data on the card */ - writel_be( 0, MIXART_MEM( mgr, MIXART_PSEUDOREG_BOARDNUMBER ) ); /* set miXart boardnumber to 0 */ - writel_be( 0, MIXART_MEM( mgr, MIXART_FLOWTABLE_PTR ) ); /* reset pointer to flow table on miXart */ + writel_be( 0, MIXART_MEM( mgr, MIXART_PSEUDOREG_BOARDNUMBER )); /* set miXart boardnumber to 0 */ + writel_be( 0, MIXART_MEM( mgr, MIXART_FLOWTABLE_PTR )); /* reset pointer to flow table on miXart */

/* set elf status to copying */ writel_be( 1, MIXART_MEM( mgr, MIXART_PSEUDOREG_ELF_STATUS_OFFSET ));

/* process the copying of the elf packets */ - err = mixart_load_elf( mgr, dsp ); + err = mixart_load_elf( mgr, dsp); if (err < 0) return err;

/* set elf status to copy finished */ @@ -424,12 +424,12 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw /* wait for elf status == 4 */ err = mixart_wait_nice_for_register_value( mgr, MIXART_PSEUDOREG_ELF_STATUS_OFFSET, 1, 4, 300); /* 3sec */ if (err < 0) { - snd_printk( KERN_ERR "elf could not be started\n"); + snd_printk(KERN_ERR "elf could not be started\n"); return err; }

/* miXart waits at this point on the pointer to the flow table */ - writel_be( (u32)mgr->flowinfo.addr, MIXART_MEM( mgr, MIXART_FLOWTABLE_PTR ) ); /* give pointer of flow table to miXart */ + writel_be( (u32)mgr->flowinfo.addr, MIXART_MEM( mgr, MIXART_FLOWTABLE_PTR )); /* give pointer of flow table to miXart */

return 0; /* return, another xilinx file has to be loaded before */

@@ -437,15 +437,15 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw default:

/* elf and xilinx should be loaded */ - if( (status_elf != 4) || (status_xilinx != 4) ) { - printk( KERN_ERR "xilinx or elf not successfully loaded\n"); + if ((status_elf != 4) || (status_xilinx != 4)) { + printk(KERN_ERR "xilinx or elf not successfully loaded\n"); return -EIO; /* modprob -r may help ? */ }

/* wait for daughter detection != 0 */ err = mixart_wait_nice_for_register_value( mgr, MIXART_PSEUDOREG_DBRD_PRESENCE_OFFSET, 0, 0, 30); /* 300msec */ if (err < 0) { - snd_printk( KERN_ERR "error starting elf file\n"); + snd_printk(KERN_ERR "error starting elf file\n"); return err; }

@@ -455,16 +455,16 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw if (mgr->board_type == MIXART_DAUGHTER_TYPE_NONE) break; /* no daughter board; the file does not have to be loaded, continue after the switch */

- /* only if aesebu daughter board presence (elf code must run) */ - if (mgr->board_type != MIXART_DAUGHTER_TYPE_AES ) + /* only if aesebu daughter board presence (elf code must run) */ + if (mgr->board_type != MIXART_DAUGHTER_TYPE_AES) return -EINVAL;

/* daughter should be idle */ - if( status_daught != 0 ) { - printk( KERN_ERR "daughter load error ! status = %d\n", status_daught); + if (status_daught != 0) { + printk(KERN_ERR "daughter load error ! status = %d\n", status_daught); return -EIO; /* modprob -r may help ? */ } - + /* check daughterboard xilinx validity */ if (((u32*)(dsp->data))[0] == 0xffffffff) return -EINVAL; @@ -480,7 +480,7 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw /* wait for status == 2 */ err = mixart_wait_nice_for_register_value( mgr, MIXART_PSEUDOREG_DXLX_STATUS_OFFSET, 1, 2, 30); /* 300msec */ if (err < 0) { - snd_printk( KERN_ERR "daughter board load error\n"); + snd_printk(KERN_ERR "daughter board load error\n"); return err; }

@@ -502,7 +502,7 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw /* wait for daughter status == 3 */ err = mixart_wait_nice_for_register_value( mgr, MIXART_PSEUDOREG_DXLX_STATUS_OFFSET, 1, 3, 300); /* 3sec */ if (err < 0) { - snd_printk( KERN_ERR "daughter board could not be initialised\n"); + snd_printk(KERN_ERR "daughter board could not be initialised\n"); return err; }

@@ -512,7 +512,7 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw /* first communication with embedded */ err = mixart_first_init(mgr); if (err < 0) { - snd_printk( KERN_ERR "miXart could not be set up\n"); + snd_printk(KERN_ERR "miXart could not be set up\n"); return err; }

@@ -615,7 +615,7 @@ static int mixart_hwdep_dsp_load(struct snd_hwdep *hw,

fw.size = dsp->length; fw.data = vmalloc(dsp->length); - if (! fw.data) { + if (!fw.data) { snd_printk(KERN_ERR "miXart: cannot allocate image size %d\n", (int)dsp->length); return -ENOMEM; diff --git a/sound/pci/mixart/mixart_hwdep.h b/sound/pci/mixart/mixart_hwdep.h index a46f508..ce1f109 100644 --- a/sound/pci/mixart/mixart_hwdep.h +++ b/sound/pci/mixart/mixart_hwdep.h @@ -36,12 +36,12 @@

/* Daughter board Type */ -#define DAUGHTER_TYPE_MASK 0x0F -#define DAUGHTER_VER_MASK 0xF0 +#define DAUGHTER_TYPE_MASK 0x0F +#define DAUGHTER_VER_MASK 0xF0 #define DAUGHTER_TYPEVER_MASK (DAUGHTER_TYPE_MASK|DAUGHTER_VER_MASK) - -#define MIXART_DAUGHTER_TYPE_NONE 0x00 -#define MIXART_DAUGHTER_TYPE_COBRANET 0x08 + +#define MIXART_DAUGHTER_TYPE_NONE 0x00 +#define MIXART_DAUGHTER_TYPE_COBRANET 0x08 #define MIXART_DAUGHTER_TYPE_AES 0x0E

@@ -63,28 +63,28 @@ #define MIXART_PSEUDOREG_PERF_INTERR_LOAD_OFFSET MIXART_PSEUDOREG+0x74 /* interrupt handling load */

/* motherboard xilinx loader info */ -#define MIXART_PSEUDOREG_MXLX_BASE_ADDR_OFFSET MIXART_PSEUDOREG+0x9C /* 0x00600000 */ -#define MIXART_PSEUDOREG_MXLX_SIZE_OFFSET MIXART_PSEUDOREG+0xA0 /* xilinx size in bytes */ -#define MIXART_PSEUDOREG_MXLX_STATUS_OFFSET MIXART_PSEUDOREG+0xA4 /* status = EMBEBBED_STAT_XXX */ +#define MIXART_PSEUDOREG_MXLX_BASE_ADDR_OFFSET MIXART_PSEUDOREG+0x9C /* 0x00600000 */ +#define MIXART_PSEUDOREG_MXLX_SIZE_OFFSET MIXART_PSEUDOREG+0xA0 /* xilinx size in bytes */ +#define MIXART_PSEUDOREG_MXLX_STATUS_OFFSET MIXART_PSEUDOREG+0xA4 /* status = EMBEBBED_STAT_XXX */

/* elf loader info */ -#define MIXART_PSEUDOREG_ELF_STATUS_OFFSET MIXART_PSEUDOREG+0xB0 /* status = EMBEBBED_STAT_XXX */ +#define MIXART_PSEUDOREG_ELF_STATUS_OFFSET MIXART_PSEUDOREG+0xB0 /* status = EMBEBBED_STAT_XXX */

-/* +/* * after the elf code is loaded, and the flowtable info was passed to it, * the driver polls on this address, until it shows 1 (presence) or 2 (absence) * once it is non-zero, the daughter board type may be read */ -#define MIXART_PSEUDOREG_DBRD_PRESENCE_OFFSET MIXART_PSEUDOREG+0x990 +#define MIXART_PSEUDOREG_DBRD_PRESENCE_OFFSET MIXART_PSEUDOREG+0x990

/* Global info structure */ #define MIXART_PSEUDOREG_DBRD_TYPE_OFFSET MIXART_PSEUDOREG+0x994 /* Type and version of daughterboard */

/* daughterboard xilinx loader info */ -#define MIXART_PSEUDOREG_DXLX_BASE_ADDR_OFFSET MIXART_PSEUDOREG+0x998 /* get the address here where to write the file */ -#define MIXART_PSEUDOREG_DXLX_SIZE_OFFSET MIXART_PSEUDOREG+0x99C /* xilinx size in bytes */ -#define MIXART_PSEUDOREG_DXLX_STATUS_OFFSET MIXART_PSEUDOREG+0x9A0 /* status = EMBEBBED_STAT_XXX */ +#define MIXART_PSEUDOREG_DXLX_BASE_ADDR_OFFSET MIXART_PSEUDOREG+0x998 /* get the address here where to write the file */ +#define MIXART_PSEUDOREG_DXLX_SIZE_OFFSET MIXART_PSEUDOREG+0x99C /* xilinx size in bytes */ +#define MIXART_PSEUDOREG_DXLX_STATUS_OFFSET MIXART_PSEUDOREG+0x9A0 /* status = EMBEBBED_STAT_XXX */

/* */ #define MIXART_FLOWTABLE_PTR 0x3000 /* pointer to flow table */ diff --git a/sound/pci/mixart/mixart_mixer.c b/sound/pci/mixart/mixart_mixer.c index 3ba6174..388c257 100644 --- a/sound/pci/mixart/mixart_mixer.c +++ b/sound/pci/mixart/mixart_mixer.c @@ -310,25 +310,25 @@ static int mixart_update_analog_audio_level(struct snd_mixart* chip, int is_capt memset(&io_level, 0, sizeof(io_level)); io_level.channel = -1; /* left and right */

- for(i=0; i<2; i++) { - if(is_capture) { + for (i = 0; i < 2; i++) { + if (is_capture) { io_level.level[i].analog_level = mixart_analog_level[chip->analog_capture_volume[i]]; } else { - if(chip->analog_playback_active[i]) + if (chip->analog_playback_active[i]) io_level.level[i].analog_level = mixart_analog_level[chip->analog_playback_volume[i]]; else io_level.level[i].analog_level = mixart_analog_level[MIXART_ANALOG_PLAYBACK_LEVEL_MIN]; } }

- if(is_capture) request.uid = chip->uid_in_analog_physio; + if (is_capture) request.uid = chip->uid_in_analog_physio; else request.uid = chip->uid_out_analog_physio; request.message_id = MSG_PHYSICALIO_SET_LEVEL; request.data = &io_level; request.size = sizeof(io_level);

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(resp), &resp); - if((err<0) || (resp.error_code)) { + if ((err<0) || (resp.error_code)) { snd_printk(KERN_DEBUG "error MSG_PHYSICALIO_SET_LEVEL card(%d) is_capture(%d) error_code(%x)\n", chip->chip_idx, is_capture, resp.error_code); return -EINVAL; } @@ -342,7 +342,7 @@ static int mixart_analog_vol_info(struct snd_kcontrol *kcontrol, struct snd_ctl_ { uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER; uinfo->count = 2; - if(kcontrol->private_value == 0) { /* playback */ + if (kcontrol->private_value == 0) { /* playback */ uinfo->value.integer.min = MIXART_ANALOG_PLAYBACK_LEVEL_MIN; /* -96 dB */ uinfo->value.integer.max = MIXART_ANALOG_PLAYBACK_LEVEL_MAX; /* 0 dB */ } else { /* capture */ @@ -356,7 +356,7 @@ static int mixart_analog_vol_get(struct snd_kcontrol *kcontrol, struct snd_ctl_e { struct snd_mixart *chip = snd_kcontrol_chip(kcontrol); mutex_lock(&chip->mgr->mixer_mutex); - if(kcontrol->private_value == 0) { /* playback */ + if (kcontrol->private_value == 0) { /* playback */ ucontrol->value.integer.value[0] = chip->analog_playback_volume[0]; ucontrol->value.integer.value[1] = chip->analog_playback_volume[1]; } else { /* capture */ @@ -731,7 +731,7 @@ int mixart_update_playback_stream_level(struct snd_mixart* chip, int is_aes, int set_level.nb_of_stream = 1; set_level.stream_level.desc.stream_idx = idx;

- if(is_aes) { + if (is_aes) { pipe = &chip->pipe_out_dig; /* AES playback */ idx += MIXART_PLAYBACK_STREAMS; } else { @@ -739,13 +739,13 @@ int mixart_update_playback_stream_level(struct snd_mixart* chip, int is_aes, int }

/* only when pipe exists ! */ - if(pipe->status == PIPE_UNDEFINED) + if (pipe->status == PIPE_UNDEFINED) return 0;

set_level.stream_level.desc.uid_pipe = pipe->group_uid;

- for(i=0; i<2; i++) { - if(chip->digital_playback_active[idx][i]) + for (i = 0; i < 2; i++) { + if (chip->digital_playback_active[idx][i]) volume[i] = chip->digital_playback_volume[idx][i]; else volume[i] = MIXART_DIGITAL_LEVEL_MIN; @@ -761,7 +761,7 @@ int mixart_update_playback_stream_level(struct snd_mixart* chip, int is_aes, int request.size = sizeof(set_level);

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(status), &status); - if((err<0) || status) { + if ((err<0) || status) { snd_printk(KERN_DEBUG "error MSG_STREAM_SET_OUT_STREAM_LEVEL card(%d) status(%x)\n", chip->chip_idx, status); return -EINVAL; } @@ -776,7 +776,7 @@ int mixart_update_capture_stream_level(struct snd_mixart* chip, int is_aes) struct mixart_set_in_audio_level_req set_level; u32 status;

- if(is_aes) { + if (is_aes) { idx = 1; pipe = &chip->pipe_in_dig; } else { @@ -785,7 +785,7 @@ int mixart_update_capture_stream_level(struct snd_mixart* chip, int is_aes) }

/* only when pipe exists ! */ - if(pipe->status == PIPE_UNDEFINED) + if (pipe->status == PIPE_UNDEFINED) return 0;

memset(&set_level, 0, sizeof(set_level)); @@ -793,7 +793,7 @@ int mixart_update_capture_stream_level(struct snd_mixart* chip, int is_aes) set_level.level[0].connector = pipe->uid_left_connector; set_level.level[1].connector = pipe->uid_right_connector;

- for(i=0; i<2; i++) { + for (i = 0; i < 2; i++) { set_level.level[i].valid_mask1 = MIXART_AUDIO_LEVEL_DIGITAL_MASK; set_level.level[i].digital_level = mixart_digital_level[chip->digital_capture_volume[idx][i]]; } @@ -804,7 +804,7 @@ int mixart_update_capture_stream_level(struct snd_mixart* chip, int is_aes) request.size = sizeof(set_level);

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(status), &status); - if((err<0) || status) { + if ((err<0) || status) { snd_printk(KERN_DEBUG "error MSG_STREAM_SET_IN_AUDIO_LEVEL card(%d) status(%x)\n", chip->chip_idx, status); return -EINVAL; } @@ -833,12 +833,12 @@ static int mixart_pcm_vol_get(struct snd_kcontrol *kcontrol, struct snd_ctl_elem int is_capture = kcontrol->private_value & MIXART_VOL_REC_MASK; int is_aes = kcontrol->private_value & MIXART_VOL_AES_MASK; mutex_lock(&chip->mgr->mixer_mutex); - if(is_capture) { - if(is_aes) stored_volume = chip->digital_capture_volume[1]; /* AES capture */ + if (is_capture) { + if (is_aes) stored_volume = chip->digital_capture_volume[1]; /* AES capture */ else stored_volume = chip->digital_capture_volume[0]; /* analog capture */ } else { snd_BUG_ON(idx >= MIXART_PLAYBACK_STREAMS); - if(is_aes) stored_volume = chip->digital_playback_volume[MIXART_PLAYBACK_STREAMS + idx]; /* AES playback */ + if (is_aes) stored_volume = chip->digital_playback_volume[MIXART_PLAYBACK_STREAMS + idx]; /* AES playback */ else stored_volume = chip->digital_playback_volume[idx]; /* analog playback */ } ucontrol->value.integer.value[0] = stored_volume[0]; @@ -911,7 +911,7 @@ static int mixart_pcm_sw_get(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_ int idx = snd_ctl_get_ioffidx(kcontrol, &ucontrol->id); /* index */ snd_BUG_ON(idx >= MIXART_PLAYBACK_STREAMS); mutex_lock(&chip->mgr->mixer_mutex); - if(kcontrol->private_value & MIXART_VOL_AES_MASK) /* AES playback */ + if (kcontrol->private_value & MIXART_VOL_AES_MASK) /* AES playback */ idx += MIXART_PLAYBACK_STREAMS; ucontrol->value.integer.value[0] = chip->digital_playback_active[idx][0]; ucontrol->value.integer.value[1] = chip->digital_playback_active[idx][1]; @@ -961,10 +961,10 @@ static int mixart_update_monitoring(struct snd_mixart* chip, int channel) struct mixart_set_out_audio_level audio_level; u32 resp;

- if(chip->pipe_out_ana.status == PIPE_UNDEFINED) + if (chip->pipe_out_ana.status == PIPE_UNDEFINED) return -EINVAL; /* no pipe defined */

- if(!channel) request.uid = chip->pipe_out_ana.uid_left_connector; + if (!channel) request.uid = chip->pipe_out_ana.uid_left_connector; else request.uid = chip->pipe_out_ana.uid_right_connector; request.message_id = MSG_CONNECTOR_SET_OUT_AUDIO_LEVEL; request.data = &audio_level; @@ -976,7 +976,7 @@ static int mixart_update_monitoring(struct snd_mixart* chip, int channel) audio_level.monitor_mute1 = !chip->monitoring_active[channel!=0];

err = snd_mixart_send_msg(chip->mgr, &request, sizeof(resp), &resp); - if((err<0) || resp) { + if ((err<0) || resp) { snd_printk(KERN_DEBUG "error MSG_CONNECTOR_SET_OUT_AUDIO_LEVEL card(%d) resp(%x)\n", chip->chip_idx, resp); return -EINVAL; } @@ -1097,7 +1097,7 @@ static void mixart_reset_audio_levels(struct snd_mixart *chip) /* analog volumes can be set even if there is no pipe */ mixart_update_analog_audio_level(chip, 0); /* analog levels for capture only on the first two chips */ - if(chip->chip_idx < 2) { + if (chip->chip_idx < 2) { mixart_update_analog_audio_level(chip, 1); } return; @@ -1111,7 +1111,7 @@ int snd_mixart_create_mixer(struct mixart_mgr *mgr)

mutex_init(&mgr->mixer_mutex); /* can be in another place */

- for(i=0; i<mgr->num_cards; i++) { + for (i = 0; i < mgr->num_cards; i++) { struct snd_kcontrol_new temp; chip = mgr->chip[i];

@@ -1126,7 +1126,7 @@ int snd_mixart_create_mixer(struct mixart_mgr *mgr) return err;

/* analog input level control only on first two chips !*/ - if(i<2) { + if (i<2) { temp = mixart_control_analog_level; temp.name = "Master Capture Volume"; temp.private_value = 1; /* capture */ @@ -1147,7 +1147,7 @@ int snd_mixart_create_mixer(struct mixart_mgr *mgr) if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&temp, chip))) < 0) return err;

- if(mgr->board_type == MIXART_DAUGHTER_TYPE_AES) { + if (mgr->board_type == MIXART_DAUGHTER_TYPE_AES) { temp.name = "AES Playback Volume"; temp.count = MIXART_PLAYBACK_STREAMS; temp.private_value = MIXART_VOL_AES_MASK; /* playback AES/EBU */ @@ -1166,7 +1166,7 @@ int snd_mixart_create_mixer(struct mixart_mgr *mgr) if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&temp, chip))) < 0) return err;

- if(mgr->board_type == MIXART_DAUGHTER_TYPE_AES) { + if (mgr->board_type == MIXART_DAUGHTER_TYPE_AES) { temp.name = "AES Playback Switch"; temp.private_value = MIXART_VOL_AES_MASK; /* playback AES/EBU */ if ((err = snd_ctl_add(chip->card, snd_ctl_new1(&temp, chip))) < 0)