2011-02-12 17:47:08

by Stanislav Fomichev

[permalink] [raw]
Subject: [PATCH] brcm80211: fix warnings in wl_check_firmwares

Use %zu instead of %d for size_t

Signed-off-by: Stanislav Fomichev <[email protected]>
---
drivers/staging/brcm80211/sys/wl_mac80211.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/brcm80211/sys/wl_mac80211.c b/drivers/staging/brcm80211/sys/wl_mac80211.c
index 5faa521..0c87baa 100644
--- a/drivers/staging/brcm80211/sys/wl_mac80211.c
+++ b/drivers/staging/brcm80211/sys/wl_mac80211.c
@@ -1822,12 +1822,12 @@ int wl_check_firmwares(struct wl_info *wl)
WL_ERROR("%s: invalid bin/hdr fw\n", __func__);
rc = -EBADF;
} else if (fw_hdr->size % sizeof(struct wl_fw_hdr)) {
- WL_ERROR("%s: non integral fw hdr file size %d/%zu\n",
+ WL_ERROR("%s: non integral fw hdr file size %zu/%zu\n",
__func__, fw_hdr->size,
sizeof(struct wl_fw_hdr));
rc = -EBADF;
} else if (fw->size < MIN_FW_SIZE || fw->size > MAX_FW_SIZE) {
- WL_ERROR("%s: out of bounds fw file size %d\n",
+ WL_ERROR("%s: out of bounds fw file size %zu\n",
__func__, fw->size);
rc = -EBADF;
} else {
--
1.7.1


2011-02-18 21:04:16

by Greg KH

[permalink] [raw]
Subject: Re: [PATCH] brcm80211: fix warnings in wl_check_firmwares

On Sat, Feb 12, 2011 at 08:47:00PM +0300, Stanislav Fomichev wrote:
> Use %zu instead of %d for size_t
>
> Signed-off-by: Stanislav Fomichev <[email protected]>
> ---
> drivers/staging/brcm80211/sys/wl_mac80211.c | 4 ++--

This doesn't apply to linux-next as this file isn't around anymore :(

thanks,

greg k-h

2011-02-19 09:25:22

by Stanislav Fomichev

[permalink] [raw]
Subject: Re: [PATCH] brcm80211: fix warnings in wl_check_firmwares

> On Sat, Feb 12, 2011 at 08:47:00PM +0300, Stanislav Fomichev wrote:
> > Use %zu instead of %d for size_t
> >
> > Signed-off-by: Stanislav Fomichev <[email protected]>
> > ---
> > drivers/staging/brcm80211/sys/wl_mac80211.c | 4 ++--
>
> This doesn't apply to linux-next as this file isn't around anymore :(
This is not relevant anymore as I see you included the other patch with
the same title (which is based on the right tree).

--
Stas