ASoC: Intel: Skylake: Parse multiple manifest data blocks
authorShreyas NC <shreyas.nc@intel.com>
Wed, 23 Aug 2017 14:03:48 +0000 (19:33 +0530)
committerMark Brown <broonie@kernel.org>
Fri, 25 Aug 2017 13:53:47 +0000 (14:53 +0100)
Currently we can parse a single manifest data block. But manifest
private data can have multiple data blocks.

So, fix the parsing logic to parse multiple data blocks by returning
offset of each parsed data block.

Signed-off-by: Shreyas NC <shreyas.nc@intel.com>
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-topology.c

index 350d2387f737f55c78e07380991604acb7b981e3..f14b373f65d73d645b44bf7ad6fea62a363fadcb 100644 (file)
@@ -2721,14 +2721,12 @@ static int skl_tplg_get_manifest_tkn(struct device *dev,
 
                        tkn_count = tkn_count + ret;
                        tkn_elem++;
-                       tuple_size += tkn_count *
-                               sizeof(struct snd_soc_tplg_vendor_value_elem);
-                       break;
                }
+               tuple_size += (tkn_count * sizeof(*tkn_elem));
                tkn_count = 0;
        }
 
-       return 0;
+       return off;
 }
 
 /*
@@ -2751,11 +2749,10 @@ static int skl_tplg_get_manifest_data(struct snd_soc_tplg_manifest *manifest,
        num_blocks = ret;
 
        off += array->size;
-       array = (struct snd_soc_tplg_vendor_array *)
-                       (manifest->priv.data + off);
-
        /* Read the BLOCK_TYPE and BLOCK_SIZE descriptor */
        while (num_blocks > 0) {
+               array = (struct snd_soc_tplg_vendor_array *)
+                               (manifest->priv.data + off);
                ret = skl_tplg_get_desc_blocks(dev, array);
 
                if (ret < 0)
@@ -2789,6 +2786,7 @@ static int skl_tplg_get_manifest_data(struct snd_soc_tplg_manifest *manifest,
                } else {
                        return -EINVAL;
                }
+               off += ret;
        }
 
        return 0;