Commit 112ac808 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Takashi Iwai

ALSA: sound/ps3: Fix checkpatch issues

Signed-off-by: default avatarGeert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 07a2039b
...@@ -165,8 +165,7 @@ static const struct snd_pcm_hardware snd_ps3_pcm_hw = { ...@@ -165,8 +165,7 @@ static const struct snd_pcm_hardware snd_ps3_pcm_hw = {
.fifo_size = PS3_AUDIO_FIFO_SIZE .fifo_size = PS3_AUDIO_FIFO_SIZE
}; };
static struct snd_pcm_ops snd_ps3_pcm_spdif_ops = static struct snd_pcm_ops snd_ps3_pcm_spdif_ops = {
{
.open = snd_ps3_pcm_open, .open = snd_ps3_pcm_open,
.close = snd_ps3_pcm_close, .close = snd_ps3_pcm_close,
.prepare = snd_ps3_pcm_prepare, .prepare = snd_ps3_pcm_prepare,
...@@ -183,7 +182,7 @@ static int snd_ps3_verify_dma_stop(struct snd_ps3_card_info *card, ...@@ -183,7 +182,7 @@ static int snd_ps3_verify_dma_stop(struct snd_ps3_card_info *card,
int dma_ch, done, retries, stop_forced = 0; int dma_ch, done, retries, stop_forced = 0;
uint32_t status; uint32_t status;
for (dma_ch = 0; dma_ch < 8; dma_ch ++) { for (dma_ch = 0; dma_ch < 8; dma_ch++) {
retries = count; retries = count;
do { do {
status = read_reg(PS3_AUDIO_KICK(dma_ch)) & status = read_reg(PS3_AUDIO_KICK(dma_ch)) &
...@@ -259,9 +258,7 @@ static void snd_ps3_kick_dma(struct snd_ps3_card_info *card) ...@@ -259,9 +258,7 @@ static void snd_ps3_kick_dma(struct snd_ps3_card_info *card)
/* /*
* convert virtual addr to ioif bus addr. * convert virtual addr to ioif bus addr.
*/ */
static dma_addr_t v_to_bus(struct snd_ps3_card_info *card, static dma_addr_t v_to_bus(struct snd_ps3_card_info *card, void *paddr, int ch)
void * paddr,
int ch)
{ {
return card->dma_start_bus_addr[ch] + return card->dma_start_bus_addr[ch] +
(paddr - card->dma_start_vaddr[ch]); (paddr - card->dma_start_vaddr[ch]);
...@@ -321,7 +318,7 @@ static int snd_ps3_program_dma(struct snd_ps3_card_info *card, ...@@ -321,7 +318,7 @@ static int snd_ps3_program_dma(struct snd_ps3_card_info *card,
spin_lock_irqsave(&card->dma_lock, irqsave); spin_lock_irqsave(&card->dma_lock, irqsave);
for (ch = 0; ch < 2; ch++) { for (ch = 0; ch < 2; ch++) {
start_vaddr = card->dma_next_transfer_vaddr[0]; start_vaddr = card->dma_next_transfer_vaddr[0];
for (stage = 0; stage < fill_stages; stage ++) { for (stage = 0; stage < fill_stages; stage++) {
dma_ch = stage * 2 + ch; dma_ch = stage * 2 + ch;
if (silent) if (silent)
dma_addr = card->null_buffer_start_dma_addr; dma_addr = card->null_buffer_start_dma_addr;
...@@ -619,7 +616,7 @@ static int snd_ps3_change_avsetting(struct snd_ps3_card_info *card) ...@@ -619,7 +616,7 @@ static int snd_ps3_change_avsetting(struct snd_ps3_card_info *card)
PS3_AUDIO_AO_3WMCTRL_ASOEN(2) | PS3_AUDIO_AO_3WMCTRL_ASOEN(2) |
PS3_AUDIO_AO_3WMCTRL_ASOEN(3)), PS3_AUDIO_AO_3WMCTRL_ASOEN(3)),
0); 0);
wmb(); /* ensure the hardware sees the change */ wmb(); /* ensure the hardware sees the change */
/* wait for actually stopped */ /* wait for actually stopped */
retries = 1000; retries = 1000;
while ((read_reg(PS3_AUDIO_AO_3WMCTRL) & while ((read_reg(PS3_AUDIO_AO_3WMCTRL) &
...@@ -798,20 +795,20 @@ static struct snd_kcontrol_new spdif_ctls[] = { ...@@ -798,20 +795,20 @@ static struct snd_kcontrol_new spdif_ctls[] = {
{ {
.access = SNDRV_CTL_ELEM_ACCESS_READ, .access = SNDRV_CTL_ELEM_ACCESS_READ,
.iface = SNDRV_CTL_ELEM_IFACE_PCM, .iface = SNDRV_CTL_ELEM_IFACE_PCM,
.name = SNDRV_CTL_NAME_IEC958("",PLAYBACK,CON_MASK), .name = SNDRV_CTL_NAME_IEC958("", PLAYBACK, CON_MASK),
.info = snd_ps3_spdif_mask_info, .info = snd_ps3_spdif_mask_info,
.get = snd_ps3_spdif_cmask_get, .get = snd_ps3_spdif_cmask_get,
}, },
{ {
.access = SNDRV_CTL_ELEM_ACCESS_READ, .access = SNDRV_CTL_ELEM_ACCESS_READ,
.iface = SNDRV_CTL_ELEM_IFACE_PCM, .iface = SNDRV_CTL_ELEM_IFACE_PCM,
.name = SNDRV_CTL_NAME_IEC958("",PLAYBACK,PRO_MASK), .name = SNDRV_CTL_NAME_IEC958("", PLAYBACK, PRO_MASK),
.info = snd_ps3_spdif_mask_info, .info = snd_ps3_spdif_mask_info,
.get = snd_ps3_spdif_pmask_get, .get = snd_ps3_spdif_pmask_get,
}, },
{ {
.iface = SNDRV_CTL_ELEM_IFACE_PCM, .iface = SNDRV_CTL_ELEM_IFACE_PCM,
.name = SNDRV_CTL_NAME_IEC958("",PLAYBACK,DEFAULT), .name = SNDRV_CTL_NAME_IEC958("", PLAYBACK, DEFAULT),
.info = snd_ps3_spdif_mask_info, .info = snd_ps3_spdif_mask_info,
.get = snd_ps3_spdif_default_get, .get = snd_ps3_spdif_default_get,
.put = snd_ps3_spdif_default_put, .put = snd_ps3_spdif_default_put,
...@@ -1020,11 +1017,12 @@ static int __init snd_ps3_driver_probe(struct ps3_system_bus_device *dev) ...@@ -1020,11 +1017,12 @@ static int __init snd_ps3_driver_probe(struct ps3_system_bus_device *dev)
* its size should be lager than PS3_AUDIO_FIFO_STAGE_SIZE * 2 * its size should be lager than PS3_AUDIO_FIFO_STAGE_SIZE * 2
* PAGE_SIZE is enogh * PAGE_SIZE is enogh
*/ */
if (!(the_card.null_buffer_start_vaddr = the_card.null_buffer_start_vaddr =
dma_alloc_coherent(&the_card.ps3_dev->core, dma_alloc_coherent(&the_card.ps3_dev->core,
PAGE_SIZE, PAGE_SIZE,
&the_card.null_buffer_start_dma_addr, &the_card.null_buffer_start_dma_addr,
GFP_KERNEL))) { GFP_KERNEL);
if (!the_card.null_buffer_start_vaddr) {
pr_info("%s: nullbuffer alloc failed\n", __func__); pr_info("%s: nullbuffer alloc failed\n", __func__);
goto clean_preallocate; goto clean_preallocate;
} }
...@@ -1148,7 +1146,7 @@ static irqreturn_t snd_ps3_interrupt(int irq, void *dev_id) ...@@ -1148,7 +1146,7 @@ static irqreturn_t snd_ps3_interrupt(int irq, void *dev_id)
SND_PS3_DMA_FILLTYPE_SILENT_FIRSTFILL : SND_PS3_DMA_FILLTYPE_SILENT_FIRSTFILL :
SND_PS3_DMA_FILLTYPE_SILENT_RUNNING); SND_PS3_DMA_FILLTYPE_SILENT_RUNNING);
snd_ps3_kick_dma(card); snd_ps3_kick_dma(card);
card->silent --; card->silent--;
} else { } else {
snd_ps3_program_dma(card, snd_ps3_program_dma(card,
(underflow_occured) ? (underflow_occured) ?
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment