diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-08 10:46:53 +0100 |
---|---|---|
committer | Mark Brown <broonie@linaro.org> | 2013-11-24 14:42:44 +0100 |
commit | a37377314ff068c83f425979142263a17a6f18af (patch) | |
tree | bd74c160350e817ed94c0cb93eaffecc89896412 /sound/soc/sh | |
parent | ASoC: rcar: rename GEN2_SRU to GEN2_SCU (diff) | |
download | linux-a37377314ff068c83f425979142263a17a6f18af.tar.xz linux-a37377314ff068c83f425979142263a17a6f18af.zip |
ASoC: rcar: some dubious one-bit signed bitfields
Because these are signed they can either be 0 or -1 instead of 0 and 1
as intended. It doesn't cause a problem from what I can see, but it's
dangerous and Sparse complains:
sound/soc/sh/rcar/rsnd.h:177:25:
error: dubious one-bit signed bitfield
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'sound/soc/sh')
-rw-r--r-- | sound/soc/sh/rcar/rsnd.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sound/soc/sh/rcar/rsnd.h b/sound/soc/sh/rcar/rsnd.h index 9e463e50e7e6..b5ac3a2afc58 100644 --- a/sound/soc/sh/rcar/rsnd.h +++ b/sound/soc/sh/rcar/rsnd.h @@ -174,11 +174,11 @@ struct rsnd_dai { struct rsnd_dai_stream playback; struct rsnd_dai_stream capture; - int clk_master:1; - int bit_clk_inv:1; - int frm_clk_inv:1; - int sys_delay:1; - int data_alignment:1; + unsigned int clk_master:1; + unsigned int bit_clk_inv:1; + unsigned int frm_clk_inv:1; + unsigned int sys_delay:1; + unsigned int data_alignment:1; }; #define rsnd_dai_nr(priv) ((priv)->dai_nr) |