]> git.hungrycats.org Git - linux/commitdiff
vdpa/octeon_ep: Fix format specifier for pointers in debug messages
authorSrujana Challa <schalla@marvell.com>
Mon, 16 Sep 2024 16:22:55 +0000 (21:52 +0530)
committerMichael S. Tsirkin <mst@redhat.com>
Wed, 25 Sep 2024 11:07:44 +0000 (07:07 -0400)
Updates the debug messages in octep_vdpa_hw.c to use the %p format
specifier for pointers instead of casting them to u64.

Fixes smatch warning:
octep_hw_caps_read() warn: argument 3 to %016llx specifier is cast
from pointer

Fixes: 8b6c724cdab8 ("virtio: vdpa: vDPA driver for Marvell OCTEON DPU devices")
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Closes: https://lore.kernel.org/r/202409160431.bRhZWhiU-lkp@intel.com/
Signed-off-by: Srujana Challa <schalla@marvell.com>
Message-Id: <20240916162255.677774-1-schalla@marvell.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
drivers/vdpa/octeon_ep/octep_vdpa_hw.c

index 11bd76ae18cf937c5dd25ec7c000abe7a4b5c130..1d4767b33315e29e0733c5154440f5d7f242f42d 100644 (file)
@@ -475,11 +475,11 @@ int octep_hw_caps_read(struct octep_hw *oct_hw, struct pci_dev *pdev)
                dev_err(dev, "Incomplete PCI capabilities");
                return -EIO;
        }
-       dev_info(dev, "common cfg mapped at: 0x%016llx\n", (u64)(uintptr_t)oct_hw->common_cfg);
-       dev_info(dev, "device cfg mapped at: 0x%016llx\n", (u64)(uintptr_t)oct_hw->dev_cfg);
-       dev_info(dev, "isr cfg mapped at: 0x%016llx\n", (u64)(uintptr_t)oct_hw->isr);
-       dev_info(dev, "notify base: 0x%016llx, notify off multiplier: %u\n",
-                (u64)(uintptr_t)oct_hw->notify_base, oct_hw->notify_off_multiplier);
+       dev_info(dev, "common cfg mapped at: %p\n", oct_hw->common_cfg);
+       dev_info(dev, "device cfg mapped at: %p\n", oct_hw->dev_cfg);
+       dev_info(dev, "isr cfg mapped at: %p\n", oct_hw->isr);
+       dev_info(dev, "notify base: %p, notify off multiplier: %u\n",
+                oct_hw->notify_base, oct_hw->notify_off_multiplier);
 
        oct_hw->config_size = octep_get_config_size(oct_hw);
        oct_hw->features = octep_hw_get_dev_features(oct_hw);
@@ -511,7 +511,7 @@ int octep_hw_caps_read(struct octep_hw *oct_hw, struct pci_dev *pdev)
        }
        mbox = octep_get_mbox(oct_hw);
        octep_mbox_init(mbox);
-       dev_info(dev, "mbox mapped at: 0x%016llx\n", (u64)(uintptr_t)mbox);
+       dev_info(dev, "mbox mapped at: %p\n", mbox);
 
        return 0;
 }