Staging: cx25821 fixed C99 comments, white spaces and unnecessary braces
This patch fixes some C99 style comments, trailing white spaces and some unnecessary 'if' braces found by checkpatch.pl Signed-off-by: Wouter Van Rooy <wouter.vanrooy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
0686e4f4a2
commit
2a613e41ec
|
@ -42,10 +42,10 @@
|
||||||
|
|
||||||
#define AUDIO_SRAM_CHANNEL SRAM_CH08
|
#define AUDIO_SRAM_CHANNEL SRAM_CH08
|
||||||
|
|
||||||
#define dprintk(level,fmt, arg...) if (debug >= level) \
|
#define dprintk(level, fmt, arg...) if (debug >= level) \
|
||||||
printk(KERN_INFO "%s/1: " fmt, chip->dev->name , ## arg)
|
printk(KERN_INFO "%s/1: " fmt, chip->dev->name , ## arg)
|
||||||
|
|
||||||
#define dprintk_core(level,fmt, arg...) if (debug >= level) \
|
#define dprintk_core(level, fmt, arg...) if (debug >= level) \
|
||||||
printk(KERN_DEBUG "%s/1: " fmt, chip->dev->name , ## arg)
|
printk(KERN_DEBUG "%s/1: " fmt, chip->dev->name , ## arg)
|
||||||
|
|
||||||
/****************************************************************************
|
/****************************************************************************
|
||||||
|
@ -105,7 +105,7 @@ MODULE_PARM_DESC(index, "Index value for cx25821 capture interface(s).");
|
||||||
MODULE_DESCRIPTION("ALSA driver module for cx25821 based capture cards");
|
MODULE_DESCRIPTION("ALSA driver module for cx25821 based capture cards");
|
||||||
MODULE_AUTHOR("Hiep Huynh");
|
MODULE_AUTHOR("Hiep Huynh");
|
||||||
MODULE_LICENSE("GPL");
|
MODULE_LICENSE("GPL");
|
||||||
MODULE_SUPPORTED_DEVICE("{{Conexant,25821}"); //"{{Conexant,23881},"
|
MODULE_SUPPORTED_DEVICE("{{Conexant,25821}"); /*"{{Conexant,23881},"*/
|
||||||
|
|
||||||
static unsigned int debug;
|
static unsigned int debug;
|
||||||
module_param(debug, int, 0644);
|
module_param(debug, int, 0644);
|
||||||
|
@ -135,7 +135,7 @@ MODULE_PARM_DESC(debug, "enable debug messages");
|
||||||
* BOARD Specific: Sets audio DMA
|
* BOARD Specific: Sets audio DMA
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static int _cx25821_start_audio_dma(snd_cx25821_card_t * chip)
|
static int _cx25821_start_audio_dma(snd_cx25821_card_t *chip)
|
||||||
{
|
{
|
||||||
struct cx25821_buffer *buf = chip->buf;
|
struct cx25821_buffer *buf = chip->buf;
|
||||||
struct cx25821_dev *dev = chip->dev;
|
struct cx25821_dev *dev = chip->dev;
|
||||||
|
@ -143,7 +143,7 @@ static int _cx25821_start_audio_dma(snd_cx25821_card_t * chip)
|
||||||
&cx25821_sram_channels[AUDIO_SRAM_CHANNEL];
|
&cx25821_sram_channels[AUDIO_SRAM_CHANNEL];
|
||||||
u32 tmp = 0;
|
u32 tmp = 0;
|
||||||
|
|
||||||
// enable output on the GPIO 0 for the MCLK ADC (Audio)
|
/* enable output on the GPIO 0 for the MCLK ADC (Audio) */
|
||||||
cx25821_set_gpiopin_direction(chip->dev, 0, 0);
|
cx25821_set_gpiopin_direction(chip->dev, 0, 0);
|
||||||
|
|
||||||
/* Make sure RISC/FIFO are off before changing FIFO/RISC settings */
|
/* Make sure RISC/FIFO are off before changing FIFO/RISC settings */
|
||||||
|
@ -158,18 +158,22 @@ static int _cx25821_start_audio_dma(snd_cx25821_card_t * chip)
|
||||||
cx_write(AUD_A_LNGTH, buf->bpl);
|
cx_write(AUD_A_LNGTH, buf->bpl);
|
||||||
|
|
||||||
/* reset counter */
|
/* reset counter */
|
||||||
cx_write(AUD_A_GPCNT_CTL, GP_COUNT_CONTROL_RESET); //GP_COUNT_CONTROL_RESET = 0x3
|
cx_write(AUD_A_GPCNT_CTL, GP_COUNT_CONTROL_RESET);
|
||||||
|
/* GP_COUNT_CONTROL_RESET = 0x3 */
|
||||||
|
|
||||||
atomic_set(&chip->count, 0);
|
atomic_set(&chip->count, 0);
|
||||||
|
|
||||||
//Set the input mode to 16-bit
|
/* Set the input mode to 16-bit */
|
||||||
tmp = cx_read(AUD_A_CFG);
|
tmp = cx_read(AUD_A_CFG);
|
||||||
cx_write(AUD_A_CFG,
|
cx_write(AUD_A_CFG,
|
||||||
tmp | FLD_AUD_DST_PK_MODE | FLD_AUD_DST_ENABLE |
|
tmp | FLD_AUD_DST_PK_MODE | FLD_AUD_DST_ENABLE |
|
||||||
FLD_AUD_CLK_ENABLE);
|
FLD_AUD_CLK_ENABLE);
|
||||||
|
|
||||||
//printk(KERN_INFO "DEBUG: Start audio DMA, %d B/line, cmds_start(0x%x)= %d lines/FIFO, %d periods, %d "
|
/*printk(KERN_INFO "DEBUG: Start audio DMA, %d B/line, "
|
||||||
// "byte buffer\n", buf->bpl, audio_ch->cmds_start, cx_read(audio_ch->cmds_start + 12)>>1,
|
"cmds_start(0x%x)= %d lines/FIFO, %d periods, %d "
|
||||||
// chip->num_periods, buf->bpl * chip->num_periods);
|
"byte buffer\n", buf->bpl, audio_ch->cmds_start, "
|
||||||
|
"cx_read(audio_ch->cmds_start + 12)>>1,
|
||||||
|
chip->num_periods, buf->bpl * chip->num_periods);*/
|
||||||
|
|
||||||
/* Enables corresponding bits at AUD_INT_STAT */
|
/* Enables corresponding bits at AUD_INT_STAT */
|
||||||
cx_write(AUD_A_INT_MSK,
|
cx_write(AUD_A_INT_MSK,
|
||||||
|
@ -182,7 +186,7 @@ static int _cx25821_start_audio_dma(snd_cx25821_card_t * chip)
|
||||||
/* enable audio irqs */
|
/* enable audio irqs */
|
||||||
cx_set(PCI_INT_MSK, chip->dev->pci_irqmask | PCI_MSK_AUD_INT);
|
cx_set(PCI_INT_MSK, chip->dev->pci_irqmask | PCI_MSK_AUD_INT);
|
||||||
|
|
||||||
// Turn on audio downstream fifo and risc enable 0x101
|
/* Turn on audio downstream fifo and risc enable 0x101 */
|
||||||
tmp = cx_read(AUD_INT_DMA_CTL);
|
tmp = cx_read(AUD_INT_DMA_CTL);
|
||||||
cx_set(AUD_INT_DMA_CTL,
|
cx_set(AUD_INT_DMA_CTL,
|
||||||
tmp | (FLD_AUD_DST_A_RISC_EN | FLD_AUD_DST_A_FIFO_EN));
|
tmp | (FLD_AUD_DST_A_RISC_EN | FLD_AUD_DST_A_FIFO_EN));
|
||||||
|
@ -194,7 +198,7 @@ static int _cx25821_start_audio_dma(snd_cx25821_card_t * chip)
|
||||||
/*
|
/*
|
||||||
* BOARD Specific: Resets audio DMA
|
* BOARD Specific: Resets audio DMA
|
||||||
*/
|
*/
|
||||||
static int _cx25821_stop_audio_dma(snd_cx25821_card_t * chip)
|
static int _cx25821_stop_audio_dma(snd_cx25821_card_t *chip)
|
||||||
{
|
{
|
||||||
struct cx25821_dev *dev = chip->dev;
|
struct cx25821_dev *dev = chip->dev;
|
||||||
|
|
||||||
|
@ -232,13 +236,12 @@ static char *cx25821_aud_irqs[32] = {
|
||||||
/*
|
/*
|
||||||
* BOARD Specific: Threats IRQ audio specific calls
|
* BOARD Specific: Threats IRQ audio specific calls
|
||||||
*/
|
*/
|
||||||
static void cx25821_aud_irq(snd_cx25821_card_t * chip, u32 status, u32 mask)
|
static void cx25821_aud_irq(snd_cx25821_card_t *chip, u32 status, u32 mask)
|
||||||
{
|
{
|
||||||
struct cx25821_dev *dev = chip->dev;
|
struct cx25821_dev *dev = chip->dev;
|
||||||
|
|
||||||
if (0 == (status & mask)) {
|
if (0 == (status & mask))
|
||||||
return;
|
return;
|
||||||
}
|
|
||||||
|
|
||||||
cx_write(AUD_A_INT_STAT, status);
|
cx_write(AUD_A_INT_STAT, status);
|
||||||
if (debug > 1 || (status & mask & ~0xff))
|
if (debug > 1 || (status & mask & ~0xff))
|
||||||
|
@ -318,11 +321,11 @@ static irqreturn_t cx25821_irq(int irq, void *dev_id)
|
||||||
if (handled)
|
if (handled)
|
||||||
cx_write(PCI_INT_STAT, pci_status);
|
cx_write(PCI_INT_STAT, pci_status);
|
||||||
|
|
||||||
out:
|
out:
|
||||||
return IRQ_RETVAL(handled);
|
return IRQ_RETVAL(handled);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int dsp_buffer_free(snd_cx25821_card_t * chip)
|
static int dsp_buffer_free(snd_cx25821_card_t *chip)
|
||||||
{
|
{
|
||||||
BUG_ON(!chip->dma_size);
|
BUG_ON(!chip->dma_size);
|
||||||
|
|
||||||
|
@ -363,7 +366,8 @@ static struct snd_pcm_hardware snd_cx25821_digital_hw = {
|
||||||
.period_bytes_max = DEFAULT_FIFO_SIZE / 3,
|
.period_bytes_max = DEFAULT_FIFO_SIZE / 3,
|
||||||
.periods_min = 1,
|
.periods_min = 1,
|
||||||
.periods_max = AUDIO_LINE_SIZE,
|
.periods_max = AUDIO_LINE_SIZE,
|
||||||
.buffer_bytes_max = (AUDIO_LINE_SIZE * AUDIO_LINE_SIZE), //128*128 = 16384 = 1024 * 16
|
.buffer_bytes_max = (AUDIO_LINE_SIZE * AUDIO_LINE_SIZE),
|
||||||
|
/* 128*128 = 16384 = 1024 * 16 */
|
||||||
};
|
};
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -393,18 +397,20 @@ static int snd_cx25821_pcm_open(struct snd_pcm_substream *substream)
|
||||||
|
|
||||||
if (cx25821_sram_channels[AUDIO_SRAM_CHANNEL].fifo_size !=
|
if (cx25821_sram_channels[AUDIO_SRAM_CHANNEL].fifo_size !=
|
||||||
DEFAULT_FIFO_SIZE) {
|
DEFAULT_FIFO_SIZE) {
|
||||||
bpl = cx25821_sram_channels[AUDIO_SRAM_CHANNEL].fifo_size / 3; //since there are 3 audio Clusters
|
/* since there are 3 audio Clusters */
|
||||||
|
bpl = cx25821_sram_channels[AUDIO_SRAM_CHANNEL].fifo_size / 3;
|
||||||
|
|
||||||
bpl &= ~7; /* must be multiple of 8 */
|
bpl &= ~7; /* must be multiple of 8 */
|
||||||
|
|
||||||
if (bpl > AUDIO_LINE_SIZE) {
|
if (bpl > AUDIO_LINE_SIZE)
|
||||||
bpl = AUDIO_LINE_SIZE;
|
bpl = AUDIO_LINE_SIZE;
|
||||||
}
|
|
||||||
runtime->hw.period_bytes_min = bpl;
|
runtime->hw.period_bytes_min = bpl;
|
||||||
runtime->hw.period_bytes_max = bpl;
|
runtime->hw.period_bytes_max = bpl;
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
_error:
|
_error:
|
||||||
dprintk(1, "Error opening PCM!\n");
|
dprintk(1, "Error opening PCM!\n");
|
||||||
return err;
|
return err;
|
||||||
}
|
}
|
||||||
|
@ -445,9 +451,9 @@ static int snd_cx25821_hw_params(struct snd_pcm_substream *substream,
|
||||||
if (NULL == buf)
|
if (NULL == buf)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
if (chip->period_size > AUDIO_LINE_SIZE) {
|
if (chip->period_size > AUDIO_LINE_SIZE)
|
||||||
chip->period_size = AUDIO_LINE_SIZE;
|
chip->period_size = AUDIO_LINE_SIZE;
|
||||||
}
|
|
||||||
|
|
||||||
buf->vb.memory = V4L2_MEMORY_MMAP;
|
buf->vb.memory = V4L2_MEMORY_MMAP;
|
||||||
buf->vb.field = V4L2_FIELD_NONE;
|
buf->vb.field = V4L2_FIELD_NONE;
|
||||||
|
@ -474,7 +480,7 @@ static int snd_cx25821_hw_params(struct snd_pcm_substream *substream,
|
||||||
buf->vb.width, buf->vb.height, 1);
|
buf->vb.width, buf->vb.height, 1);
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
printk(KERN_INFO
|
printk(KERN_INFO
|
||||||
"DEBUG: ERROR after cx25821_risc_databuffer_audio() \n");
|
"DEBUG: ERROR after cx25821_risc_databuffer_audio()\n");
|
||||||
goto error;
|
goto error;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -494,7 +500,7 @@ static int snd_cx25821_hw_params(struct snd_pcm_substream *substream,
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
error:
|
error:
|
||||||
kfree(buf);
|
kfree(buf);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
@ -593,10 +599,10 @@ static struct snd_pcm_ops snd_cx25821_pcm_ops = {
|
||||||
};
|
};
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* ALSA create a PCM device: Called when initializing the board. Sets up the name and hooks up
|
* ALSA create a PCM device: Called when initializing the board.
|
||||||
* the callbacks
|
* Sets up the name and hooks up the callbacks
|
||||||
*/
|
*/
|
||||||
static int snd_cx25821_pcm(snd_cx25821_card_t * chip, int device, char *name)
|
static int snd_cx25821_pcm(snd_cx25821_card_t *chip, int device, char *name)
|
||||||
{
|
{
|
||||||
struct snd_pcm *pcm;
|
struct snd_pcm *pcm;
|
||||||
int err;
|
int err;
|
||||||
|
@ -655,7 +661,7 @@ static void snd_cx25821_dev_free(struct snd_card *card)
|
||||||
{
|
{
|
||||||
snd_cx25821_card_t *chip = card->private_data;
|
snd_cx25821_card_t *chip = card->private_data;
|
||||||
|
|
||||||
//snd_cx25821_free(chip);
|
/*snd_cx25821_free(chip);*/
|
||||||
snd_card_free(chip->card);
|
snd_card_free(chip->card);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -671,13 +677,13 @@ static int cx25821_audio_initdev(struct cx25821_dev *dev)
|
||||||
if (devno >= SNDRV_CARDS) {
|
if (devno >= SNDRV_CARDS) {
|
||||||
printk(KERN_INFO "DEBUG ERROR: devno >= SNDRV_CARDS %s\n",
|
printk(KERN_INFO "DEBUG ERROR: devno >= SNDRV_CARDS %s\n",
|
||||||
__func__);
|
__func__);
|
||||||
return (-ENODEV);
|
return -ENODEV;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!enable[devno]) {
|
if (!enable[devno]) {
|
||||||
++devno;
|
++devno;
|
||||||
printk(KERN_INFO "DEBUG ERROR: !enable[devno] %s\n", __func__);
|
printk(KERN_INFO "DEBUG ERROR: !enable[devno] %s\n", __func__);
|
||||||
return (-ENOENT);
|
return -ENOENT;
|
||||||
}
|
}
|
||||||
|
|
||||||
err = snd_card_create(index[devno], id[devno], THIS_MODULE,
|
err = snd_card_create(index[devno], id[devno], THIS_MODULE,
|
||||||
|
@ -741,7 +747,7 @@ static int cx25821_audio_initdev(struct cx25821_dev *dev)
|
||||||
devno++;
|
devno++;
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
error:
|
error:
|
||||||
snd_card_free(card);
|
snd_card_free(card);
|
||||||
return err;
|
return err;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue