ALSA: fireworks/bebob: Use the same type of variables as function parameters
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Tue, 27 May 2014 15:14:41 +0000 (00:14 +0900)
committerTakashi Iwai <tiwai@suse.de>
Tue, 27 May 2014 15:35:22 +0000 (17:35 +0200)
The second argument of snd_efw_command_get_sampling_rate() means sampling
rate and its type is 'unsigned int'. But 'int' variable is passed as parameter.
It's better to apply the same type for the variable as its argument.

Similally, the type of variable for snd_efw_command_get_clock_source() and
avc_bridgeco_get_plug_type() should be the same type as each argument.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/bebob/bebob_stream.c
sound/firewire/fireworks/fireworks_pcm.c

index fbe938c79e3a5e84e4fbb25f0f679cc14e032788..38aee3344a5dc332947ba50ebbdbd1a04b47fbff 100644 (file)
@@ -852,7 +852,8 @@ static int
 seek_msu_sync_input_plug(struct snd_bebob *bebob)
 {
        u8 plugs[AVC_PLUG_INFO_BUF_BYTES], addr[AVC_BRIDGECO_ADDR_BYTES];
-       unsigned int i, type;
+       unsigned int i;
+       enum avc_bridgeco_plug_type type;
        int err;
 
        /* Get the number of Music Sub Unit for both direction. */
index ed211d062202bfae37841f6687da9e645a095a30..8a34753de210831070643abd8e24b5ae10f89781 100644 (file)
@@ -195,8 +195,8 @@ end:
 static int pcm_open(struct snd_pcm_substream *substream)
 {
        struct snd_efw *efw = substream->private_data;
-       int sampling_rate;
-       unsigned int clock_source;
+       unsigned int sampling_rate;
+       enum snd_efw_clock_source clock_source;
        int err;
 
        err = snd_efw_stream_lock_try(efw);