ALSA: ak411x: simplify snd_ak4113_create() a bit
"err" is always a negative error code here, so there is no point in checking. Removing the check silences a static checker warning and makes the code a bit more clear. Also we don't need to initialize "err". Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
parent
e311782acd
commit
034f90b393
@ -73,7 +73,7 @@ int snd_ak4113_create(struct snd_card *card, ak4113_read_t *read,
|
|||||||
void *private_data, struct ak4113 **r_ak4113)
|
void *private_data, struct ak4113 **r_ak4113)
|
||||||
{
|
{
|
||||||
struct ak4113 *chip;
|
struct ak4113 *chip;
|
||||||
int err = 0;
|
int err;
|
||||||
unsigned char reg;
|
unsigned char reg;
|
||||||
static struct snd_device_ops ops = {
|
static struct snd_device_ops ops = {
|
||||||
.dev_free = snd_ak4113_dev_free,
|
.dev_free = snd_ak4113_dev_free,
|
||||||
@ -109,7 +109,7 @@ int snd_ak4113_create(struct snd_card *card, ak4113_read_t *read,
|
|||||||
|
|
||||||
__fail:
|
__fail:
|
||||||
snd_ak4113_free(chip);
|
snd_ak4113_free(chip);
|
||||||
return err < 0 ? err : -EIO;
|
return err;
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(snd_ak4113_create);
|
EXPORT_SYMBOL_GPL(snd_ak4113_create);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user