net: mdio: fix -Wvoid-pointer-to-enum-cast warning
authorJustin Stitt <justinstitt@google.com>
Tue, 15 Aug 2023 20:35:59 +0000 (20:35 +0000)
committerJakub Kicinski <kuba@kernel.org>
Fri, 18 Aug 2023 02:25:40 +0000 (19:25 -0700)
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>
drivers/net/mdio/mdio-xgene.c

index 683e8f8..1af7a4d 100644 (file)
@@ -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;