OSDN Git Service

ASoC: Intel: KeemBay: Fix warning potential ! vs ~ typo
authorMichael Sit Wei Hong <michael.wei.hong.sit@intel.com>
Fri, 4 Sep 2020 02:09:04 +0000 (10:09 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 7 Sep 2020 14:28:57 +0000 (15:28 +0100)
To set platform in slave mode setting the MASTER_MODE bit is not needed.
Removing !MASTER_MODE conditional to avoid potential errors and warning.

Signed-off-by: Michael Sit Wei Hong <michael.wei.hong.sit@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20200904020904.19577-1-michael.wei.hong.sit@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/keembay/kmb_platform.c

index ca25a6e..f54b710 100644 (file)
@@ -515,7 +515,7 @@ static int kmb_dai_hw_params(struct snd_pcm_substream *substream,
 
                write_val = ((config->chan_nr / 2) << TDM_CHANNEL_CONFIG_BIT) |
                                (config->data_width << DATA_WIDTH_CONFIG_BIT) |
-                               !MASTER_MODE | TDM_OPERATION;
+                               TDM_OPERATION;
 
                writel(write_val, kmb_i2s->pss_base + I2S_GEN_CFG_0);
                break;