diff options
author | Daniel Kurtz <djkurtz@chromium.org> | 2015-07-22 13:23:26 +0300 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-07-23 17:58:33 +0300 |
commit | a70f0d027c83350ad46ca3e86e4c781bfed7fcc2 (patch) | |
tree | d1016bdb21837a1f740b856c544f980ff485ae29 /drivers/extcon | |
parent | bc0195aad0daa2ad5b0d76cce22b167bc3435590 (diff) | |
download | linux-a70f0d027c83350ad46ca3e86e4c781bfed7fcc2.tar.xz |
regulator: tps6586x: silence pointer-to-int-cast
of_regulator_match.driver_data is (void *). tps6586x uses it to store an
anonymous enum value (those TPS6586X_ID_ values).
Later, it tries to extract the ID by casting directly to an int, which is a
no-no ([-Wpointer-to-int-cast]):
drivers/regulator/tps6586x-regulator.c: In function 'tps6586x_parse_regulator_dt':
drivers/regulator/tps6586x-regulator.c:430:8: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
id = (int)tps6586x_matches[i].driver_data;
^
Instead of casting to int, uintptr_t is better suited for receiving and
comparing integers extracted from void *. This is especially true on
64-bit systems where sizeof(void *) != sizeof(int).
Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/extcon')
0 files changed, 0 insertions, 0 deletions