ASoC: Intel: Skylake: Remove return check for skl_codec_create()
authorguneshwor.o.singh@intel.com <guneshwor.o.singh@intel.com>
Fri, 28 Jul 2017 10:42:15 +0000 (16:12 +0530)
committerMark Brown <broonie@kernel.org>
Tue, 1 Aug 2017 12:34:05 +0000 (13:34 +0100)
Since skl_codec_create() always returns 0, make it return void and remove
return check.

Signed-off-by: Guneshwor Singh <guneshwor.o.singh@intel.com>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/skylake/skl.c

index b9e1310673f0eb2d70ed5dffa2a1237b8b10f47c..ab651517e8b8979233108696c90780006f036ae1 100644 (file)
@@ -528,7 +528,7 @@ static int probe_codec(struct hdac_ext_bus *ebus, int addr)
 }
 
 /* Codec initialization */
-static int skl_codec_create(struct hdac_ext_bus *ebus)
+static void skl_codec_create(struct hdac_ext_bus *ebus)
 {
        struct hdac_bus *bus = ebus_to_hbus(ebus);
        int c, max_slots;
@@ -559,8 +559,6 @@ static int skl_codec_create(struct hdac_ext_bus *ebus)
                        }
                }
        }
-
-       return 0;
 }
 
 static const struct hdac_bus_ops bus_core_ops = {
@@ -612,9 +610,7 @@ static void skl_probe_work(struct work_struct *work)
                dev_info(bus->dev, "no hda codecs found!\n");
 
        /* create codec instances */
-       err = skl_codec_create(ebus);
-       if (err < 0)
-               goto out_err;
+       skl_codec_create(ebus);
 
        if (IS_ENABLED(CONFIG_SND_SOC_HDAC_HDMI)) {
                err = snd_hdac_display_power(bus, false);