From: Michael McConville <mmcco@xxxxxxxxxxx> We can replace the if-statement with the simpler bit toggle operation. Changes in v2: - add proper commit header and description Signed-off-by: Michael McConville <mmcco@xxxxxxxxxxx> Signed-off-by: Tobias Jakobi <tjakobi@xxxxxxxxxxxxxxxxxxxxx> --- sound/soc/samsung/i2s.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/sound/soc/samsung/i2s.c b/sound/soc/samsung/i2s.c index 50635ee..1c484ac 100644 --- a/sound/soc/samsung/i2s.c +++ b/sound/soc/samsung/i2s.c @@ -627,10 +627,7 @@ static int i2s_set_fmt(struct snd_soc_dai *dai, case SND_SOC_DAIFMT_NB_NF: break; case SND_SOC_DAIFMT_NB_IF: - if (tmp & lrp_rlow) - tmp &= ~lrp_rlow; - else - tmp |= lrp_rlow; + tmp ^= lrp_rlow; break; default: dev_err(&i2s->pdev->dev, "Polarity not supported\n"); -- 2.7.3 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html