diff options
author | Justin Stitt <justinstitt@google.com> | 2023-08-16 19:55:29 +0000 |
---|---|---|
committer | Noralf Trønnes <noralf@tronnes.org> | 2023-08-25 13:13:20 +0200 |
commit | 24883eb269f087b5d1068833fced543e020296ca (patch) | |
tree | 7d9512705223fad9290d3aae57fce5ebcf949c37 | |
parent | 17c35883cf5351fd0667083e752a28222a4cec55 (diff) | |
download | linux-24883eb269f087b5d1068833fced543e020296ca.tar.gz linux-24883eb269f087b5d1068833fced543e020296ca.tar.bz2 linux-24883eb269f087b5d1068833fced543e020296ca.zip |
drm/repaper: fix -Wvoid-pointer-to-enum-cast warning
When building with clang 18 I see the following warning:
| drivers/gpu/drm/tiny/repaper.c:952:11: warning: cast to smaller integer
| type 'enum repaper_model' from 'const void *' [-Wvoid-pointer-to-enum-cast]
| 952 | model = (enum repaper_model)match;
|
This is due to the fact that `match` is a void* while `enum repaper_model`
has the size of an int.
Add uintptr_t cast to silence clang warning while also keeping enum cast
for readability and consistency with other `model` assignment just a
few lines below:
| model = (enum repaper_model)spi_id->driver_data;
Link: https://github.com/ClangBuiltLinux/linux/issues/1910
Reported-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Justin Stitt <justinstitt@google.com>
Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20230816-void-drivers-gpu-drm-tiny-repaper-v1-1-9d8d10f0d52f@google.com
-rw-r--r-- | drivers/gpu/drm/tiny/repaper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/tiny/repaper.c b/drivers/gpu/drm/tiny/repaper.c index 13ae148f59b9..73dd4f4289c2 100644 --- a/drivers/gpu/drm/tiny/repaper.c +++ b/drivers/gpu/drm/tiny/repaper.c @@ -949,7 +949,7 @@ static int repaper_probe(struct spi_device *spi) match = device_get_match_data(dev); if (match) { - model = (enum repaper_model)match; + model = (enum repaper_model)(uintptr_t)match; } else { spi_id = spi_get_device_id(spi); model = (enum repaper_model)spi_id->driver_data; |