From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758211Ab2BMV0r (ORCPT ); Mon, 13 Feb 2012 16:26:47 -0500 Received: from mailout-de.gmx.net ([213.165.64.22]:33373 "HELO mailout-de.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1756008Ab2BMV0p (ORCPT ); Mon, 13 Feb 2012 16:26:45 -0500 X-Authenticated: #10250065 X-Provags-ID: V01U2FsdGVkX18ADj4jTqkA7vV8BZurUqwxGDSSs7EQ4jFCLDCokz 3X/qjT5VtFshTZ Message-ID: <4F398006.6080000@gmx.de> Date: Mon, 13 Feb 2012 21:26:30 +0000 From: Florian Tobias Schandinat User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20120207 Icedove/3.0.11 MIME-Version: 1.0 To: Danny Kukawka CC: Antonino Daplas , linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 13/16] riva/fbdev: fix several -Wuninitialized References: <1327960820-11867-1-git-send-email-danny.kukawka@bisect.de> <1327960820-11867-14-git-send-email-danny.kukawka@bisect.de> In-Reply-To: <1327960820-11867-14-git-send-email-danny.kukawka@bisect.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Y-GMX-Trusted: 0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/30/2012 10:00 PM, Danny Kukawka wrote: > Fix several -Wuninitialized compiler warnings by changing the > order of getting modedb in riva_update_default_var() to set > first the fallback and then the prefered timing. > > Signed-off-by: Danny Kukawka Applied. Thanks, Florian Tobias Schandinat > --- > drivers/video/riva/fbdev.c | 5 ++--- > 1 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/video/riva/fbdev.c b/drivers/video/riva/fbdev.c > index 2f58cf9..90df1a6 100644 > --- a/drivers/video/riva/fbdev.c > +++ b/drivers/video/riva/fbdev.c > @@ -1816,6 +1816,8 @@ static void __devinit riva_update_default_var(struct fb_var_screeninfo *var, > specs->modedb, specs->modedb_len, > NULL, 8); > } else if (specs->modedb != NULL) { > + /* get first mode in database as fallback */ > + modedb = specs->modedb[0]; > /* get preferred timing */ > if (info->monspecs.misc & FB_MISC_1ST_DETAIL) { > int i; > @@ -1826,9 +1828,6 @@ static void __devinit riva_update_default_var(struct fb_var_screeninfo *var, > break; > } > } > - } else { > - /* otherwise, get first mode in database */ > - modedb = specs->modedb[0]; > } > var->bits_per_pixel = 8; > riva_update_var(var, &modedb);