ASoC: Intel: Skylake: code cleanup for pin fixup limitation
authorTakashi Sakamoto <takashi.sakamoto@miraclelinux.com>
Wed, 8 Mar 2017 08:47:02 +0000 (17:47 +0900)
committerMark Brown <broonie@kernel.org>
Mon, 13 Mar 2017 16:07:41 +0000 (16:07 +0000)
As a commit 4cd9899f0d16 ("ASoC: Intel: Skylake: Add multiple pin
formats") describes, 'fixups is applied to pin 0 only'. On the other
hand, the commit left some codes as what they were. This might
confuses readers.

This commit fixes the issue. This doesn't change driver behaviour at all.

Fixes: 4cd9899f0d16 ("ASoC: Intel: Skylake: Add multiple pin formats")
Signed-off-by: Takashi Sakamoto <takashi.sakamoto@miraclelinux.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Tested-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/skylake/skl-topology.c

index 17a9a55026a420d5ad34a778f07357541ec951c6..e7836a2e18f063c038b750342fbac37f7621faad 100644 (file)
@@ -310,13 +310,11 @@ static void skl_tplg_update_buffer_size(struct skl_sst *ctx,
                multiplier = 5;
 
        mcfg->ibs = DIV_ROUND_UP(in_fmt->s_freq, 1000) *
-                       (mcfg->in_fmt->channels) *
-                       (mcfg->in_fmt->bit_depth >> 3) *
+                       in_fmt->channels * (in_fmt->bit_depth >> 3) *
                        multiplier;
 
-       mcfg->obs = DIV_ROUND_UP(mcfg->out_fmt->s_freq, 1000) *
-                       (mcfg->out_fmt->channels) *
-                       (mcfg->out_fmt->bit_depth >> 3) *
+       mcfg->obs = DIV_ROUND_UP(out_fmt->s_freq, 1000) *
+                       out_fmt->channels * (out_fmt->bit_depth >> 3) *
                        multiplier;
 }