"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>
                void *private_data, struct ak4113 **r_ak4113)
 {
        struct ak4113 *chip;
-       int err = 0;
+       int err;
        unsigned char reg;
        static struct snd_device_ops ops = {
                .dev_free =     snd_ak4113_dev_free,
 
 __fail:
        snd_ak4113_free(chip);
-       return err < 0 ? err : -EIO;
+       return err;
 }
 EXPORT_SYMBOL_GPL(snd_ak4113_create);