ALSA: echoaudio: remove all snd_printk
removed all references of snd_printk with the standard dev_* macro. [a few places degraded to dev_dbg(), too -- tiwai] Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:

committed by
Takashi Iwai

parent
e369086968
commit
b5b4a41b39
@@ -80,7 +80,7 @@ static int send_vector(struct echoaudio *chip, u32 command)
|
||||
udelay(1);
|
||||
}
|
||||
|
||||
DE_ACT((KERN_ERR "timeout on send_vector\n"));
|
||||
dev_err(chip->card->dev, "timeout on send_vector\n");
|
||||
return -EBUSY;
|
||||
}
|
||||
|
||||
@@ -104,7 +104,7 @@ static int write_dsp(struct echoaudio *chip, u32 data)
|
||||
}
|
||||
|
||||
chip->bad_board = TRUE; /* Set TRUE until DSP re-loaded */
|
||||
DE_ACT((KERN_ERR "write_dsp: Set bad_board to TRUE\n"));
|
||||
dev_dbg(chip->card->dev, "write_dsp: Set bad_board to TRUE\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
@@ -127,7 +127,7 @@ static int read_dsp(struct echoaudio *chip, u32 *data)
|
||||
}
|
||||
|
||||
chip->bad_board = TRUE; /* Set TRUE until DSP re-loaded */
|
||||
DE_INIT((KERN_ERR "read_dsp: Set bad_board to TRUE\n"));
|
||||
dev_err(chip->card->dev, "read_dsp: Set bad_board to TRUE\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
@@ -154,8 +154,9 @@ static int read_sn(struct echoaudio *chip)
|
||||
return -EIO;
|
||||
}
|
||||
}
|
||||
DE_INIT(("Read serial number %08x %08x %08x %08x %08x\n",
|
||||
sn[0], sn[1], sn[2], sn[3], sn[4]));
|
||||
dev_dbg(chip->card->dev,
|
||||
"Read serial number %08x %08x %08x %08x %08x\n",
|
||||
sn[0], sn[1], sn[2], sn[3], sn[4]);
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -205,12 +206,12 @@ static int load_asic_generic(struct echoaudio *chip, u32 cmd, short asic)
|
||||
goto la_error;
|
||||
}
|
||||
|
||||
DE_INIT(("ASIC loaded\n"));
|
||||
dev_dbg(chip->card->dev, "ASIC loaded\n");
|
||||
free_firmware(fw, chip);
|
||||
return 0;
|
||||
|
||||
la_error:
|
||||
DE_INIT(("failed on write_dsp\n"));
|
||||
dev_err(chip->card->dev, "failed on write_dsp\n");
|
||||
free_firmware(fw, chip);
|
||||
return -EIO;
|
||||
}
|
||||
@@ -241,8 +242,9 @@ static int install_resident_loader(struct echoaudio *chip)
|
||||
loader is already installed, host flag 5 will be on. */
|
||||
status = get_dsp_register(chip, CHI32_STATUS_REG);
|
||||
if (status & CHI32_STATUS_REG_HF5) {
|
||||
DE_INIT(("Resident loader already installed; status is 0x%x\n",
|
||||
status));
|
||||
dev_dbg(chip->card->dev,
|
||||
"Resident loader already installed; status is 0x%x\n",
|
||||
status);
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -283,12 +285,14 @@ static int install_resident_loader(struct echoaudio *chip)
|
||||
|
||||
/* Write the count to the DSP */
|
||||
if (write_dsp(chip, words)) {
|
||||
DE_INIT(("install_resident_loader: Failed to write word count!\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"install_resident_loader: Failed to write word count!\n");
|
||||
goto irl_error;
|
||||
}
|
||||
/* Write the DSP address */
|
||||
if (write_dsp(chip, address)) {
|
||||
DE_INIT(("install_resident_loader: Failed to write DSP address!\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"install_resident_loader: Failed to write DSP address!\n");
|
||||
goto irl_error;
|
||||
}
|
||||
/* Write out this block of code to the DSP */
|
||||
@@ -297,7 +301,8 @@ static int install_resident_loader(struct echoaudio *chip)
|
||||
|
||||
data = ((u32)code[index] << 16) + code[index + 1];
|
||||
if (write_dsp(chip, data)) {
|
||||
DE_INIT(("install_resident_loader: Failed to write DSP code\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"install_resident_loader: Failed to write DSP code\n");
|
||||
goto irl_error;
|
||||
}
|
||||
index += 2;
|
||||
@@ -312,11 +317,11 @@ static int install_resident_loader(struct echoaudio *chip)
|
||||
}
|
||||
|
||||
if (i == 200) {
|
||||
DE_INIT(("Resident loader failed to set HF5\n"));
|
||||
dev_err(chip->card->dev, "Resident loader failed to set HF5\n");
|
||||
goto irl_error;
|
||||
}
|
||||
|
||||
DE_INIT(("Resident loader successfully installed\n"));
|
||||
dev_dbg(chip->card->dev, "Resident loader successfully installed\n");
|
||||
free_firmware(fw, chip);
|
||||
return 0;
|
||||
|
||||
@@ -334,14 +339,14 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
int index, words, i;
|
||||
|
||||
if (chip->dsp_code == code) {
|
||||
DE_INIT(("DSP is already loaded!\n"));
|
||||
dev_warn(chip->card->dev, "DSP is already loaded!\n");
|
||||
return 0;
|
||||
}
|
||||
chip->bad_board = TRUE; /* Set TRUE until DSP loaded */
|
||||
chip->dsp_code = NULL; /* Current DSP code not loaded */
|
||||
chip->asic_loaded = FALSE; /* Loading the DSP code will reset the ASIC */
|
||||
|
||||
DE_INIT(("load_dsp: Set bad_board to TRUE\n"));
|
||||
dev_dbg(chip->card->dev, "load_dsp: Set bad_board to TRUE\n");
|
||||
|
||||
/* If this board requires a resident loader, install it. */
|
||||
#ifdef DSP_56361
|
||||
@@ -351,7 +356,8 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
|
||||
/* Send software reset command */
|
||||
if (send_vector(chip, DSP_VC_RESET) < 0) {
|
||||
DE_INIT(("LoadDsp: send_vector DSP_VC_RESET failed, Critical Failure\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"LoadDsp: send_vector DSP_VC_RESET failed, Critical Failure\n");
|
||||
return -EIO;
|
||||
}
|
||||
/* Delay 10us */
|
||||
@@ -366,7 +372,8 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
}
|
||||
|
||||
if (i == 1000) {
|
||||
DE_INIT(("load_dsp: Timeout waiting for CHI32_STATUS_REG_HF3\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: Timeout waiting for CHI32_STATUS_REG_HF3\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
@@ -403,29 +410,34 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
index += 2;
|
||||
|
||||
if (write_dsp(chip, words) < 0) {
|
||||
DE_INIT(("load_dsp: failed to write number of DSP words\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: failed to write number of DSP words\n");
|
||||
return -EIO;
|
||||
}
|
||||
if (write_dsp(chip, address) < 0) {
|
||||
DE_INIT(("load_dsp: failed to write DSP address\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: failed to write DSP address\n");
|
||||
return -EIO;
|
||||
}
|
||||
if (write_dsp(chip, mem_type) < 0) {
|
||||
DE_INIT(("load_dsp: failed to write DSP memory type\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: failed to write DSP memory type\n");
|
||||
return -EIO;
|
||||
}
|
||||
/* Code */
|
||||
for (i = 0; i < words; i++, index+=2) {
|
||||
data = ((u32)code[index] << 16) + code[index + 1];
|
||||
if (write_dsp(chip, data) < 0) {
|
||||
DE_INIT(("load_dsp: failed to write DSP data\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: failed to write DSP data\n");
|
||||
return -EIO;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (write_dsp(chip, 0) < 0) { /* We're done!!! */
|
||||
DE_INIT(("load_dsp: Failed to write final zero\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: Failed to write final zero\n");
|
||||
return -EIO;
|
||||
}
|
||||
udelay(10);
|
||||
@@ -438,12 +450,14 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
get_dsp_register(chip, CHI32_CONTROL_REG) & ~0x1b00);
|
||||
|
||||
if (write_dsp(chip, DSP_FNC_SET_COMMPAGE_ADDR) < 0) {
|
||||
DE_INIT(("load_dsp: Failed to write DSP_FNC_SET_COMMPAGE_ADDR\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: Failed to write DSP_FNC_SET_COMMPAGE_ADDR\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
if (write_dsp(chip, chip->comm_page_phys) < 0) {
|
||||
DE_INIT(("load_dsp: Failed to write comm page address\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: Failed to write comm page address\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
@@ -452,19 +466,21 @@ static int load_dsp(struct echoaudio *chip, u16 *code)
|
||||
We don't actually use the serial number but we have to
|
||||
get it as part of the DSP init voodoo. */
|
||||
if (read_sn(chip) < 0) {
|
||||
DE_INIT(("load_dsp: Failed to read serial number\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: Failed to read serial number\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
chip->dsp_code = code; /* Show which DSP code loaded */
|
||||
chip->bad_board = FALSE; /* DSP OK */
|
||||
DE_INIT(("load_dsp: OK!\n"));
|
||||
dev_dbg(chip->card->dev, "load_dsp: OK!\n");
|
||||
return 0;
|
||||
}
|
||||
udelay(100);
|
||||
}
|
||||
|
||||
DE_INIT(("load_dsp: DSP load timed out waiting for HF4\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"load_dsp: DSP load timed out waiting for HF4\n");
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
@@ -658,7 +674,7 @@ static void get_audio_meters(struct echoaudio *chip, long *meters)
|
||||
static int restore_dsp_rettings(struct echoaudio *chip)
|
||||
{
|
||||
int i, o, err;
|
||||
DE_INIT(("restore_dsp_settings\n"));
|
||||
dev_dbg(chip->card->dev, "restore_dsp_settings\n");
|
||||
|
||||
if ((err = check_asic_status(chip)) < 0)
|
||||
return err;
|
||||
@@ -755,7 +771,7 @@ static int restore_dsp_rettings(struct echoaudio *chip)
|
||||
if (send_vector(chip, DSP_VC_UPDATE_FLAGS) < 0)
|
||||
return -EIO;
|
||||
|
||||
DE_INIT(("restore_dsp_rettings done\n"));
|
||||
dev_dbg(chip->card->dev, "restore_dsp_rettings done\n");
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -835,7 +851,8 @@ static void set_audio_format(struct echoaudio *chip, u16 pipe_index,
|
||||
break;
|
||||
}
|
||||
}
|
||||
DE_ACT(("set_audio_format[%d] = %x\n", pipe_index, dsp_format));
|
||||
dev_dbg(chip->card->dev,
|
||||
"set_audio_format[%d] = %x\n", pipe_index, dsp_format);
|
||||
chip->comm_page->audio_format[pipe_index] = cpu_to_le16(dsp_format);
|
||||
}
|
||||
|
||||
@@ -848,7 +865,7 @@ Same thing for pause_ and stop_ -trasport below. */
|
||||
static int start_transport(struct echoaudio *chip, u32 channel_mask,
|
||||
u32 cyclic_mask)
|
||||
{
|
||||
DE_ACT(("start_transport %x\n", channel_mask));
|
||||
dev_dbg(chip->card->dev, "start_transport %x\n", channel_mask);
|
||||
|
||||
if (wait_handshake(chip))
|
||||
return -EIO;
|
||||
@@ -866,7 +883,7 @@ static int start_transport(struct echoaudio *chip, u32 channel_mask,
|
||||
return 0;
|
||||
}
|
||||
|
||||
DE_ACT(("start_transport: No pipes to start!\n"));
|
||||
dev_err(chip->card->dev, "start_transport: No pipes to start!\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
@@ -874,7 +891,7 @@ static int start_transport(struct echoaudio *chip, u32 channel_mask,
|
||||
|
||||
static int pause_transport(struct echoaudio *chip, u32 channel_mask)
|
||||
{
|
||||
DE_ACT(("pause_transport %x\n", channel_mask));
|
||||
dev_dbg(chip->card->dev, "pause_transport %x\n", channel_mask);
|
||||
|
||||
if (wait_handshake(chip))
|
||||
return -EIO;
|
||||
@@ -893,7 +910,7 @@ static int pause_transport(struct echoaudio *chip, u32 channel_mask)
|
||||
return 0;
|
||||
}
|
||||
|
||||
DE_ACT(("pause_transport: No pipes to stop!\n"));
|
||||
dev_warn(chip->card->dev, "pause_transport: No pipes to stop!\n");
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -901,7 +918,7 @@ static int pause_transport(struct echoaudio *chip, u32 channel_mask)
|
||||
|
||||
static int stop_transport(struct echoaudio *chip, u32 channel_mask)
|
||||
{
|
||||
DE_ACT(("stop_transport %x\n", channel_mask));
|
||||
dev_dbg(chip->card->dev, "stop_transport %x\n", channel_mask);
|
||||
|
||||
if (wait_handshake(chip))
|
||||
return -EIO;
|
||||
@@ -920,7 +937,7 @@ static int stop_transport(struct echoaudio *chip, u32 channel_mask)
|
||||
return 0;
|
||||
}
|
||||
|
||||
DE_ACT(("stop_transport: No pipes to stop!\n"));
|
||||
dev_warn(chip->card->dev, "stop_transport: No pipes to stop!\n");
|
||||
return 0;
|
||||
}
|
||||
|
||||
@@ -937,7 +954,8 @@ static inline int is_pipe_allocated(struct echoaudio *chip, u16 pipe_index)
|
||||
stopped and unallocated. */
|
||||
static int rest_in_peace(struct echoaudio *chip)
|
||||
{
|
||||
DE_ACT(("rest_in_peace() open=%x\n", chip->pipe_alloc_mask));
|
||||
dev_dbg(chip->card->dev,
|
||||
"rest_in_peace() open=%x\n", chip->pipe_alloc_mask);
|
||||
|
||||
/* Stops all active pipes (just to be sure) */
|
||||
stop_transport(chip, chip->active_mask);
|
||||
@@ -965,7 +983,8 @@ static int init_dsp_comm_page(struct echoaudio *chip)
|
||||
{
|
||||
/* Check if the compiler added extra padding inside the structure */
|
||||
if (offsetof(struct comm_page, midi_output) != 0xbe0) {
|
||||
DE_INIT(("init_dsp_comm_page() - Invalid struct comm_page structure\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"init_dsp_comm_page() - Invalid struct comm_page structure\n");
|
||||
return -EPERM;
|
||||
}
|
||||
|
||||
@@ -999,7 +1018,7 @@ static int init_dsp_comm_page(struct echoaudio *chip)
|
||||
*/
|
||||
static int init_line_levels(struct echoaudio *chip)
|
||||
{
|
||||
DE_INIT(("init_line_levels\n"));
|
||||
dev_dbg(chip->card->dev, "init_line_levels\n");
|
||||
memset(chip->output_gain, ECHOGAIN_MUTED, sizeof(chip->output_gain));
|
||||
memset(chip->input_gain, ECHOGAIN_MUTED, sizeof(chip->input_gain));
|
||||
memset(chip->monitor_gain, ECHOGAIN_MUTED, sizeof(chip->monitor_gain));
|
||||
@@ -1051,7 +1070,8 @@ static int allocate_pipes(struct echoaudio *chip, struct audiopipe *pipe,
|
||||
u32 channel_mask;
|
||||
char is_cyclic;
|
||||
|
||||
DE_ACT(("allocate_pipes: ch=%d int=%d\n", pipe_index, interleave));
|
||||
dev_dbg(chip->card->dev,
|
||||
"allocate_pipes: ch=%d int=%d\n", pipe_index, interleave);
|
||||
|
||||
if (chip->bad_board)
|
||||
return -EIO;
|
||||
@@ -1061,7 +1081,8 @@ static int allocate_pipes(struct echoaudio *chip, struct audiopipe *pipe,
|
||||
for (channel_mask = i = 0; i < interleave; i++)
|
||||
channel_mask |= 1 << (pipe_index + i);
|
||||
if (chip->pipe_alloc_mask & channel_mask) {
|
||||
DE_ACT(("allocate_pipes: channel already open\n"));
|
||||
dev_err(chip->card->dev,
|
||||
"allocate_pipes: channel already open\n");
|
||||
return -EAGAIN;
|
||||
}
|
||||
|
||||
@@ -1078,7 +1099,7 @@ static int allocate_pipes(struct echoaudio *chip, struct audiopipe *pipe,
|
||||
it moves data. The DMA counter is in units of bytes, not samples. */
|
||||
pipe->dma_counter = &chip->comm_page->position[pipe_index];
|
||||
*pipe->dma_counter = 0;
|
||||
DE_ACT(("allocate_pipes: ok\n"));
|
||||
dev_dbg(chip->card->dev, "allocate_pipes: ok\n");
|
||||
return pipe_index;
|
||||
}
|
||||
|
||||
@@ -1089,7 +1110,7 @@ static int free_pipes(struct echoaudio *chip, struct audiopipe *pipe)
|
||||
u32 channel_mask;
|
||||
int i;
|
||||
|
||||
DE_ACT(("free_pipes: Pipe %d\n", pipe->index));
|
||||
dev_dbg(chip->card->dev, "free_pipes: Pipe %d\n", pipe->index);
|
||||
if (snd_BUG_ON(!is_pipe_allocated(chip, pipe->index)))
|
||||
return -EINVAL;
|
||||
if (snd_BUG_ON(pipe->state != PIPE_STATE_STOPPED))
|
||||
@@ -1131,7 +1152,7 @@ static int sglist_add_mapping(struct echoaudio *chip, struct audiopipe *pipe,
|
||||
list[head].size = cpu_to_le32(length);
|
||||
pipe->sglist_head++;
|
||||
} else {
|
||||
DE_ACT(("SGlist: too many fragments\n"));
|
||||
dev_err(chip->card->dev, "SGlist: too many fragments\n");
|
||||
return -ENOMEM;
|
||||
}
|
||||
return 0;
|
||||
|
Reference in New Issue
Block a user