* [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c
@ 2014-01-14 16:14 Masanari Iida
2014-01-14 22:25 ` Brown, Aaron F
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Masanari Iida @ 2014-01-14 16:14 UTC (permalink / raw)
To: jeffrey.t.kirsher, jesse.brandeburg, bruce.w.allan, e1000-devel,
netdev, linux-kernel, trivial
Cc: Masanari Iida
cppcheck detected following warning in ixgbe_fcoe.c
(warning) %d in format string (no. 1) requires 'int' but the
argument type is 'unsigned int'.
Signed-off-by: Masanari Iida <standby24x7@gmail.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
index f58db45..0872617 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
@@ -585,7 +585,7 @@ static int ixgbe_fcoe_dma_pool_alloc(struct ixgbe_fcoe *fcoe,
struct dma_pool *pool;
char pool_name[32];
- snprintf(pool_name, 32, "ixgbe_fcoe_ddp_%d", cpu);
+ snprintf(pool_name, 32, "ixgbe_fcoe_ddp_%u", cpu);
pool = dma_pool_create(pool_name, dev, IXGBE_FCPTR_MAX,
IXGBE_FCPTR_ALIGN, PAGE_SIZE);
--
1.8.5.2.309.ga25014b
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c
2014-01-14 16:14 [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c Masanari Iida
@ 2014-01-14 22:25 ` Brown, Aaron F
2014-01-15 22:55 ` David Miller
2014-01-21 23:04 ` Brown, Aaron F
2 siblings, 0 replies; 5+ messages in thread
From: Brown, Aaron F @ 2014-01-14 22:25 UTC (permalink / raw)
To: standby24x7
Cc: trivial, e1000-devel, netdev, Allan, Bruce W, linux-kernel,
Brandeburg, Jesse
On Wed, 2014-01-15 at 01:14 +0900, Masanari Iida wrote:
> cppcheck detected following warning in ixgbe_fcoe.c
> (warning) %d in format string (no. 1) requires 'int' but the
> argument type is 'unsigned int'.
>
> Signed-off-by: Masanari Iida <standby24x7@gmail.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks Masanari, I have added it to our internal queue for testing.
------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c
2014-01-14 16:14 [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c Masanari Iida
2014-01-14 22:25 ` Brown, Aaron F
@ 2014-01-15 22:55 ` David Miller
2014-01-15 23:34 ` Brown, Aaron F
2014-01-21 23:04 ` Brown, Aaron F
2 siblings, 1 reply; 5+ messages in thread
From: David Miller @ 2014-01-15 22:55 UTC (permalink / raw)
To: standby24x7
Cc: jeffrey.t.kirsher, jesse.brandeburg, bruce.w.allan, e1000-devel,
netdev, linux-kernel, trivial
From: Masanari Iida <standby24x7@gmail.com>
Date: Wed, 15 Jan 2014 01:14:42 +0900
> cppcheck detected following warning in ixgbe_fcoe.c
> (warning) %d in format string (no. 1) requires 'int' but the
> argument type is 'unsigned int'.
>
> Signed-off-by: Masanari Iida <standby24x7@gmail.com>
Intel folks, please be sure to pick this up.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c
2014-01-15 22:55 ` David Miller
@ 2014-01-15 23:34 ` Brown, Aaron F
0 siblings, 0 replies; 5+ messages in thread
From: Brown, Aaron F @ 2014-01-15 23:34 UTC (permalink / raw)
To: davem
Cc: Kirsher, Jeffrey T, Brandeburg, Jesse, Allan, Bruce W,
standby24x7, e1000-devel, netdev, linux-kernel, trivial
On Wed, 2014-01-15 at 14:55 -0800, David Miller wrote:
> From: Masanari Iida <standby24x7@gmail.com>
> Date: Wed, 15 Jan 2014 01:14:42 +0900
>
> > cppcheck detected following warning in ixgbe_fcoe.c
> > (warning) %d in format string (no. 1) requires 'int' but the
> > argument type is 'unsigned int'.
> >
> > Signed-off-by: Masanari Iida <standby24x7@gmail.com>
>
> Intel folks, please be sure to pick this up.
Yup, already in our queue.
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c
2014-01-14 16:14 [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c Masanari Iida
2014-01-14 22:25 ` Brown, Aaron F
2014-01-15 22:55 ` David Miller
@ 2014-01-21 23:04 ` Brown, Aaron F
2 siblings, 0 replies; 5+ messages in thread
From: Brown, Aaron F @ 2014-01-21 23:04 UTC (permalink / raw)
To: standby24x7
Cc: Kirsher, Jeffrey T, Brandeburg, Jesse, Allan, Bruce W,
e1000-devel, netdev, linux-kernel, trivial
On Wed, 2014-01-15 at 01:14 +0900, Masanari Iida wrote:
> cppcheck detected following warning in ixgbe_fcoe.c
> (warning) %d in format string (no. 1) requires 'int' but the
> argument type is 'unsigned int'.
>
> Signed-off-by: Masanari Iida <standby24x7@gmail.com>
Tested-By: Jack Morgan <jack.morgan@intel.com>
Signed-off-by: Aaron Brown <aaron.f.brown@intel.com>
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
> index f58db45..0872617 100644
> --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
> +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
> @@ -585,7 +585,7 @@ static int ixgbe_fcoe_dma_pool_alloc(struct ixgbe_fcoe *fcoe,
> struct dma_pool *pool;
> char pool_name[32];
>
> - snprintf(pool_name, 32, "ixgbe_fcoe_ddp_%d", cpu);
> + snprintf(pool_name, 32, "ixgbe_fcoe_ddp_%u", cpu);
>
> pool = dma_pool_create(pool_name, dev, IXGBE_FCPTR_MAX,
> IXGBE_FCPTR_ALIGN, PAGE_SIZE);
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-01-21 23:04 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-14 16:14 [PATCH] [trivial] ixgbe: Fix format string in ixgbe_fcoe.c Masanari Iida
2014-01-14 22:25 ` Brown, Aaron F
2014-01-15 22:55 ` David Miller
2014-01-15 23:34 ` Brown, Aaron F
2014-01-21 23:04 ` Brown, Aaron F
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).