From: Arnd Bergmann <arnd@arndb.de> To: "David S. Miller" <davem@davemloft.net> Cc: linux-arm-kernel@lists.infradead.org, Arnd Bergmann <arnd@arndb.de>, netdev@vger.kernel.org, Prashant Sreedharan <prashant@broadcom.com>, Michael Chan <mchan@broadcom.com>, linux-kernel@vger.kernel.org Subject: [PATCH 9/9] net: tg3: avoid uninitialized variable warning Date: Wed, 27 Jan 2016 15:04:59 +0100 [thread overview] Message-ID: <1453903507-3427225-10-git-send-email-arnd@arndb.de> (raw) In-Reply-To: <1453903507-3427225-1-git-send-email-arnd@arndb.de> The tg3_set_eeprom() function correctly initializes the 'start' variable, but gcc generates a false warning: drivers/net/ethernet/broadcom/tg3.c: In function 'tg3_set_eeprom': drivers/net/ethernet/broadcom/tg3.c:12057:4: warning: 'start' may be used uninitialized in this function [-Wmaybe-uninitialized] I have not come up with a way to restructure the code in a way that avoids the warning without making it less readable, so this adds an initialization for the declaration to shut up that warning. Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/net/ethernet/broadcom/tg3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c index 9293675df7ba..49eea8981332 100644 --- a/drivers/net/ethernet/broadcom/tg3.c +++ b/drivers/net/ethernet/broadcom/tg3.c @@ -12016,7 +12016,7 @@ static int tg3_set_eeprom(struct net_device *dev, struct ethtool_eeprom *eeprom, int ret; u32 offset, len, b_offset, odd_len; u8 *buf; - __be32 start, end; + __be32 start = 0, end; if (tg3_flag(tp, NO_NVRAM) || eeprom->magic != TG3_EEPROM_MAGIC) -- 2.7.0
WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 9/9] net: tg3: avoid uninitialized variable warning Date: Wed, 27 Jan 2016 15:04:59 +0100 [thread overview] Message-ID: <1453903507-3427225-10-git-send-email-arnd@arndb.de> (raw) In-Reply-To: <1453903507-3427225-1-git-send-email-arnd@arndb.de> The tg3_set_eeprom() function correctly initializes the 'start' variable, but gcc generates a false warning: drivers/net/ethernet/broadcom/tg3.c: In function 'tg3_set_eeprom': drivers/net/ethernet/broadcom/tg3.c:12057:4: warning: 'start' may be used uninitialized in this function [-Wmaybe-uninitialized] I have not come up with a way to restructure the code in a way that avoids the warning without making it less readable, so this adds an initialization for the declaration to shut up that warning. Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/net/ethernet/broadcom/tg3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c index 9293675df7ba..49eea8981332 100644 --- a/drivers/net/ethernet/broadcom/tg3.c +++ b/drivers/net/ethernet/broadcom/tg3.c @@ -12016,7 +12016,7 @@ static int tg3_set_eeprom(struct net_device *dev, struct ethtool_eeprom *eeprom, int ret; u32 offset, len, b_offset, odd_len; u8 *buf; - __be32 start, end; + __be32 start = 0, end; if (tg3_flag(tp, NO_NVRAM) || eeprom->magic != TG3_EEPROM_MAGIC) -- 2.7.0
next prev parent reply other threads:[~2016-01-27 14:08 UTC|newest] Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-01-27 14:04 [PATCH 0/9] network driver fixes Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 1/9] net: davinci_cpdma: use dma_addr_t for DMA address Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 2/9] net: hp100: remove unnecessary #ifdefs Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 3/9] net: bgmac: clarify CONFIG_BCMA dependency Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 16:11 ` Paul Gortmaker 2016-01-27 16:11 ` Paul Gortmaker 2016-01-27 16:11 ` Paul Gortmaker 2016-01-28 8:49 ` Arnd Bergmann 2016-01-28 8:49 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 4/9] net: moxart: use correct accessors for DMA memory Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-28 12:36 ` David Laight 2016-01-28 12:36 ` David Laight 2016-01-28 16:53 ` Arnd Bergmann 2016-01-28 16:53 ` Arnd Bergmann 2016-01-28 16:58 ` Arnd Bergmann 2016-01-28 16:58 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 5/9] net: fddi/defxx: avoid warning about uninitialized variable use Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 15:15 ` Maciej W. Rozycki 2016-01-27 15:15 ` Maciej W. Rozycki 2016-01-27 14:04 ` [PATCH 6/9] net: vxge: avoid unused function warnings Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 14:04 ` [PATCH 7/9] net: macb: avoid uninitialized variables Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 15:51 ` Nicolas Ferre 2016-01-27 15:51 ` Nicolas Ferre 2016-01-27 16:04 ` Nicolas Ferre 2016-01-27 16:04 ` Nicolas Ferre 2016-01-28 16:32 ` Arnd Bergmann 2016-01-28 16:32 ` Arnd Bergmann 2016-01-28 13:27 ` Sergei Shtylyov 2016-01-28 13:27 ` Sergei Shtylyov 2016-01-27 14:04 ` [PATCH 8/9] net: nb8800: avoid uninitialized variable warning Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann 2016-01-27 14:13 ` Måns Rullgård 2016-01-27 14:13 ` Måns Rullgård 2016-01-27 15:21 ` Arnd Bergmann 2016-01-27 15:21 ` Arnd Bergmann 2016-01-27 14:04 ` Arnd Bergmann [this message] 2016-01-27 14:04 ` [PATCH 9/9] net: tg3: " Arnd Bergmann 2016-01-29 0:14 ` [PATCH 0/9] network driver fixes David Miller 2016-01-29 0:14 ` David Miller 2016-01-29 12:56 ` Arnd Bergmann 2016-01-29 12:56 ` Arnd Bergmann
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=1453903507-3427225-10-git-send-email-arnd@arndb.de \ --to=arnd@arndb.de \ --cc=davem@davemloft.net \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mchan@broadcom.com \ --cc=netdev@vger.kernel.org \ --cc=prashant@broadcom.com \ /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: linkBe 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.