ASoC: SOF: ipc4-topology: clarify calculation precedence
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Tue, 20 Sep 2022 14:54:04 +0000 (16:54 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 20 Sep 2022 18:07:36 +0000 (19:07 +0100)
cppcheck warning:

sound/soc/sof/ipc4-topology.c:334:64: style: Clarify calculation
precedence for '&' and '?'. [clarifyCalculation]
 type = fw_module->man4_module_entry.type & SOF_IPC4_MODULE_DP ? 1 : 0;
                                                               ^

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Chao Song <chao.song@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20220920145405.2089147-2-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/ipc4-topology.c

index 64929dc..1503e3c 100644 (file)
@@ -331,7 +331,7 @@ static int sof_ipc4_widget_setup_msg(struct snd_sof_widget *swidget, struct sof_
        msg->extension = SOF_IPC4_MOD_EXT_PPL_ID(swidget->pipeline_id);
        msg->extension |= SOF_IPC4_MOD_EXT_CORE_ID(swidget->core);
 
-       type = fw_module->man4_module_entry.type & SOF_IPC4_MODULE_DP ? 1 : 0;
+       type = (fw_module->man4_module_entry.type & SOF_IPC4_MODULE_DP) ? 1 : 0;
        msg->extension |= SOF_IPC4_MOD_EXT_DOMAIN(type);
 
        return 0;