diff mbox series

[v1,1/1] scsi: mpi3mr: Fix printf() specifier

Message ID 20240602140420.3247493-1-andriy.shevchenko@linux.intel.com (mailing list archive)
State Not Applicable
Headers show
Series [v1,1/1] scsi: mpi3mr: Fix printf() specifier | expand

Commit Message

Andy Shevchenko June 2, 2024, 2:04 p.m. UTC
Compiler is not happy about format string

  format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘unsigned int’ [-Werror=format=]
  note: in expansion of macro ‘ioc_warn’
  1367 |       ioc_warn(mrioc, "skipping port %u, max allowed value is %lu\n",

Fix the specifier to make it possible to build.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/scsi/mpi3mr/mpi3mr_transport.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andy Shevchenko June 2, 2024, 3:36 p.m. UTC | #1
On Sun, Jun 02, 2024 at 05:04:20PM +0300, Andy Shevchenko wrote:
> Compiler is not happy about format string
> 
>   format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘unsigned int’ [-Werror=format=]
>   note: in expansion of macro ‘ioc_warn’
>   1367 |       ioc_warn(mrioc, "skipping port %u, max allowed value is %lu\n",
> 
> Fix the specifier to make it possible to build.

Already fixed by 9f365cb8bbd0 ("scsi: mpi3mr: Use proper format specifier in
mpi3mr_sas_port_add()").
diff mbox series

Patch

diff --git a/drivers/scsi/mpi3mr/mpi3mr_transport.c b/drivers/scsi/mpi3mr/mpi3mr_transport.c
index 329cc6ec3b58..82aa4e418c5a 100644
--- a/drivers/scsi/mpi3mr/mpi3mr_transport.c
+++ b/drivers/scsi/mpi3mr/mpi3mr_transport.c
@@ -1364,7 +1364,7 @@  static struct mpi3mr_sas_port *mpi3mr_sas_port_add(struct mpi3mr_ioc *mrioc,
 			continue;
 
 		if (i > sizeof(mr_sas_port->phy_mask) * 8) {
-			ioc_warn(mrioc, "skipping port %u, max allowed value is %lu\n",
+			ioc_warn(mrioc, "skipping port %u, max allowed value is %zu\n",
 			    i, sizeof(mr_sas_port->phy_mask) * 8);
 			goto out_fail;
 		}