Re: [PATCH] gma500: fix compiler warnings and format string

[Alan Cox <alan@lxorguk.ukuu.org.uk>]

On Sun, 3 Jul 2011 09:16:13 +0200 (CEST)
Jesper Juhl <jj@chaosbits.net> wrote:

> This patch fixes these compiler warnings:
> 
>   CC      drivers/staging/gma500/psb_gtt.o
> drivers/staging/gma500/psb_gtt.c: In function ‘psb_gtt_insert’:
> drivers/staging/gma500/psb_gtt.c:81:33: warning: unused variable ‘dev_priv’ [-Wunused-variable]
> drivers/staging/gma500/psb_gtt.c: In function ‘psb_gtt_pin’:
> drivers/staging/gma500/psb_gtt.c:237:2: warning: ‘ret’ may be used uninitialized in this function [-Wuninitialized]
> 
> It also fixes up the format string used in DRM_DEBUG() in
> drivers/staging/gma500/psb_gtt.c:psb_gtt_init() - the string required
> two arguments but only one was supplied.
> 
> While reading through the file I also deleted what trailing whitespace
> I encountered.

Unfortunately this file doesn't even exist in the working tree any more.
Please wait until Greg has taken the patch set I sent him to replace the
one that didn't apply, and the patch series after that (and quite
possibly the one after that assuming I can get Cedartrail to work next
week).

Alan