diff options
author | Justin Stitt <justinstitt@google.com> | 2023-08-15 20:35:59 +0000 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2023-08-17 19:25:40 -0700 |
commit | f3add6dec36d9d747929918ba1d7ce8866e1c054 (patch) | |
tree | fe291d4ea3a4d58e2378d46942cad4e0cc2b029c | |
parent | 0c2d8227ba7881306d8404a3a2ffd97ac479fc0a (diff) | |
download | linux-f3add6dec36d9d747929918ba1d7ce8866e1c054.tar.gz linux-f3add6dec36d9d747929918ba1d7ce8866e1c054.tar.bz2 linux-f3add6dec36d9d747929918ba1d7ce8866e1c054.zip |
net: mdio: fix -Wvoid-pointer-to-enum-cast warning
When building with clang 18 I see the following warning:
| drivers/net/mdio/mdio-xgene.c:338:13: warning: cast to smaller integer
| type 'enum xgene_mdio_id' from 'const void *' [-Wvoid-pointer-to-enum-cast]
| 338 | mdio_id = (enum xgene_mdio_id)of_id->data;
This is due to the fact that `of_id->data` is a void* while `enum
xgene_mdio_id` has the size of an int. This leads to truncation and
possible data loss.
Link: https://github.com/ClangBuiltLinux/linux/issues/1910
Reported-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Justin Stitt <justinstitt@google.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://lore.kernel.org/r/20230815-void-drivers-net-mdio-mdio-xgene-v1-1-5304342e0659@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
-rw-r--r-- | drivers/net/mdio/mdio-xgene.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/mdio/mdio-xgene.c b/drivers/net/mdio/mdio-xgene.c index 683e8f8319ab..1af7a4d9f86c 100644 --- a/drivers/net/mdio/mdio-xgene.c +++ b/drivers/net/mdio/mdio-xgene.c @@ -335,7 +335,7 @@ static int xgene_mdio_probe(struct platform_device *pdev) of_id = of_match_device(xgene_mdio_of_match, &pdev->dev); if (of_id) { - mdio_id = (enum xgene_mdio_id)of_id->data; + mdio_id = (uintptr_t)of_id->data; } else { #ifdef CONFIG_ACPI const struct acpi_device_id *acpi_id; |