Re: ps3_gelic_wireless: fix format warning

Re: ps3_gelic_wireless: fix format warning

[Geert Uytterhoeven]

On Tue, Mar 2, 2010 at 21:06, Linux Kernel Mailing List
<linux-kernel@vger.kernel.org> wrote:
> Gitweb:     http://git.kernel.org/linus/c6c97bdaa3307d0811f14c99c38c63a8dae36e1f
> Commit:     c6c97bdaa3307d0811f14c99c38c63a8dae36e1f
> Parent:     d3a571971e5af241074947fc80f6284677f6e014
> Author:     John W. Linville <linville@tuxdriver.com>
> AuthorDate: Mon Feb 1 09:55:56 2010 -0500
> Committer:  John W. Linville <linville@tuxdriver.com>
> CommitDate: Mon Feb 1 09:55:56 2010 -0500
>
>    ps3_gelic_wireless: fix format warning
>
>    Signed-off-by: John W. Linville <linville@tuxdriver.com>
> ---
>  drivers/net/ps3_gelic_wireless.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/ps3_gelic_wireless.c b/drivers/net/ps3_gelic_wireless.c
> index d520e9e..448c5b6 100644
> --- a/drivers/net/ps3_gelic_wireless.c
> +++ b/drivers/net/ps3_gelic_wireless.c
> @@ -1449,7 +1449,7 @@ static int gelic_wl_start_scan(struct gelic_wl_info *wl, int always_scan,
>        void *buf = NULL;
>        size_t len;
>
> -       pr_debug("%s: <- always=%d essid_len=%d\n", __func__,
> +       pr_debug("%s: <- always=%d essid_len=%z\n", __func__,

`z' is just a modifier for a (missing) type, so this should be `%zu'.

>                 always_scan, essid_len);
>        if (mutex_lock_interruptible(&wl->scan_lock))
>                return -ERESTARTSYS;

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

Re: ps3_gelic_wireless: fix format warning

[John W. Linville]

On Mon, Mar 08, 2010 at 07:33:27AM +0100, Geert Uytterhoeven wrote:
> On Tue, Mar 2, 2010 at 21:06, Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org> wrote:
> > Gitweb:     http://git.kernel.org/linus/c6c97bdaa3307d0811f14c99c38c63a8dae36e1f
> > Commit:     c6c97bdaa3307d0811f14c99c38c63a8dae36e1f
> > Parent:     d3a571971e5af241074947fc80f6284677f6e014
> > Author:     John W. Linville <linville@tuxdriver.com>
> > AuthorDate: Mon Feb 1 09:55:56 2010 -0500
> > Committer:  John W. Linville <linville@tuxdriver.com>
> > CommitDate: Mon Feb 1 09:55:56 2010 -0500
> >
> >    ps3_gelic_wireless: fix format warning
> >
> >    Signed-off-by: John W. Linville <linville@tuxdriver.com>
> > ---
> >  drivers/net/ps3_gelic_wireless.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/net/ps3_gelic_wireless.c b/drivers/net/ps3_gelic_wireless.c
> > index d520e9e..448c5b6 100644
> > --- a/drivers/net/ps3_gelic_wireless.c
> > +++ b/drivers/net/ps3_gelic_wireless.c
> > @@ -1449,7 +1449,7 @@ static int gelic_wl_start_scan(struct gelic_wl_info *wl, int always_scan,
> >        void *buf = NULL;
> >        size_t len;
> >
> > -       pr_debug("%s: <- always=%d essid_len=%d\n", __func__,
> > +       pr_debug("%s: <- always=%d essid_len=%z\n", __func__,
> 
> `z' is just a modifier for a (missing) type, so this should be `%zu'.

Ugh...yes, correct.  I'll fix it.

Thanks!

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.