All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anatolij Gustschin <agust@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] board/samsung/universal_c210/universal.c: Fix build warning
Date: Mon, 19 Dec 2011 16:18:28 +0100	[thread overview]
Message-ID: <20111219161828.7495c109@wker> (raw)
In-Reply-To: <20111219160021.2a92ed9a@lmajewski.digital.local>

Hi all,

On Mon, 19 Dec 2011 16:00:21 +0100
Lukasz Majewski <l.majewski@samsung.com> wrote:
...
> > diff --git a/board/samsung/universal_c210/universal.c
> > b/board/samsung/universal_c210/universal.c index 8393e4f..d0ff834
> > 100644 --- a/board/samsung/universal_c210/universal.c
> > +++ b/board/samsung/universal_c210/universal.c
> > @@ -260,7 +260,7 @@ int board_mmc_init(bd_t *bis)
> >  #ifdef CONFIG_USB_GADGET
> >  static int s5pc210_phy_control(int on)
> >  {
> > -	int ret;
> > +	int ret = 0;
> >  	struct pmic *p = get_pmic();
> >  
> >  	if (pmic_probe(p))
> 
> Hi Anatolij,
> 
> Thank you for pointing me out about the new compiler version.
> 
> I've just realized, that I'm still using:
> gcc version 4.4.1 (Sourcery G++ Lite 2009q3-68)
> 
> My question is about the GCC 4.6 version.
> Is it a Codesourcery (Mentor Graphisc), ptxdist Oselas Toolchain,
> buildroot (uClibc), linaro, ELDK or a vanilla GCC 4.6 build from source
> with custom paramters?

I used GCC version 4.2.2 from ELDK 4.2 and GCC version 4.6.1 20110627
from ELDK 5.1 and see this warning with both.

For another patch fixing warnings for s3c_udc_otg driver I
also used same GCC versions, but I see the warnings only
with GCC 4.6.1 from ELDK 5.1.  GCC 4.2.2 from ELDK 4.2 didn't
generate any warning when building s3c_udc_otg driver.

Thanks,
Anatolij

  reply	other threads:[~2011-12-19 15:18 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-19 14:20 [U-Boot] [PATCH] board/samsung/universal_c210/universal.c: Fix build warning Anatolij Gustschin
2011-12-19 15:00 ` Lukasz Majewski
2011-12-19 15:18   ` Anatolij Gustschin [this message]
2011-12-19 15:50     ` Wolfgang Denk
2011-12-20  1:41 ` Minkyu Kang
2011-12-20 22:19 ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111219161828.7495c109@wker \
    --to=agust@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.