diff options
| author | Lars-Peter Clausen <lars@metafoo.de> | 2014-08-17 16:18:19 +0200 |
|---|---|---|
| committer | Stephane Desneux <stephane.desneux@open.eurogiciel.org> | 2015-02-04 11:16:20 +0100 |
| commit | 5a9961ad6472149ba1d3f2ec24e6712cc53203f3 (patch) | |
| tree | 44e581dcff06dbf171a8d5b90d0a52271011b7aa /sound | |
| parent | cddffe783c890836910950d6a3d59ebc42167fbd (diff) | |
| download | kernel-common-5a9961ad6472149ba1d3f2ec24e6712cc53203f3.tar.gz kernel-common-5a9961ad6472149ba1d3f2ec24e6712cc53203f3.tar.bz2 kernel-common-5a9961ad6472149ba1d3f2ec24e6712cc53203f3.zip | |
ASoC: rcar: Use && instead of & for boolean expressions
Sparse spits out the following warning:
sound/soc/sh/rcar/gen.c:250:21: warning: dubious: x & !y
It does this because sometimes mixing boolean and bit-wise logic has not the
intended result. In this case we are fine, but replacing the bit-wise '&' with
the boolean '&&' silences the sparse warning. The generated code for both cases
is the same.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@linaro.org>
(cherry picked from commit c8e6e960733f4a5835265c15429fced4d2f1595e)
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/sh/rcar/gen.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/sh/rcar/gen.c b/sound/soc/sh/rcar/gen.c index 3fdf3be7b99a..f95e7ab135e8 100644 --- a/sound/soc/sh/rcar/gen.c +++ b/sound/soc/sh/rcar/gen.c @@ -247,7 +247,7 @@ rsnd_gen2_dma_addr(struct rsnd_priv *priv, }; /* it shouldn't happen */ - if (use_dvc & !use_src) + if (use_dvc && !use_src) dev_err(dev, "DVC is selected without SRC\n"); /* use SSIU or SSI ? */ |