* [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled
@ 2011-06-02 15:19 Mike Frysinger
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Mike Frysinger @ 2011-06-02 15:19 UTC (permalink / raw)
To: u-boot
The ctrl variable is only used when autoneg support is disabled, so only
declare it under those conditions to avoid an unused variable warning.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
drivers/net/designware.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/drivers/net/designware.c b/drivers/net/designware.c
index 3f5eeb7..96f20d1 100644
--- a/drivers/net/designware.c
+++ b/drivers/net/designware.c
@@ -368,11 +368,13 @@ static int configure_phy(struct eth_device *dev)
{
struct dw_eth_dev *priv = dev->priv;
int phy_addr;
- u16 bmcr, ctrl;
+ u16 bmcr;
#if defined(CONFIG_DW_AUTONEG)
u16 bmsr;
u32 timeout;
u16 anlpar, btsr;
+#else
+ u16 ctrl;
#endif
#if defined(CONFIG_DW_SEARCH_PHY)
--
1.7.5.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage
2011-06-02 15:19 [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Mike Frysinger
@ 2011-06-02 15:19 ` Mike Frysinger
2011-06-06 4:21 ` Vipin Kumar
2011-07-25 20:21 ` Wolfgang Denk
2011-06-06 4:21 ` [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Vipin Kumar
2011-07-25 20:20 ` Wolfgang Denk
2 siblings, 2 replies; 6+ messages in thread
From: Mike Frysinger @ 2011-06-02 15:19 UTC (permalink / raw)
To: u-boot
When CONFIG_DW_SEARCH_PHY is disabled, the local phy_addr variable never
gets initialized causes random behavior at runtime and a gcc warning. So
set it by default to the stored phy address.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
drivers/net/designware.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/net/designware.c b/drivers/net/designware.c
index 96f20d1..bfa4384 100644
--- a/drivers/net/designware.c
+++ b/drivers/net/designware.c
@@ -383,6 +383,8 @@ static int configure_phy(struct eth_device *dev)
priv->address = phy_addr;
else
return -1;
+#else
+ phy_addr = priv->address;
#endif
if (dw_reset_phy(dev) < 0)
return -1;
--
1.7.5.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled
2011-06-02 15:19 [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Mike Frysinger
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
@ 2011-06-06 4:21 ` Vipin Kumar
2011-07-25 20:20 ` Wolfgang Denk
2 siblings, 0 replies; 6+ messages in thread
From: Vipin Kumar @ 2011-06-06 4:21 UTC (permalink / raw)
To: u-boot
On 6/2/2011 8:49 PM, Mike Frysinger wrote:
> The ctrl variable is only used when autoneg support is disabled, so only
> declare it under those conditions to avoid an unused variable warning.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> drivers/net/designware.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/designware.c b/drivers/net/designware.c
> index 3f5eeb7..96f20d1 100644
> --- a/drivers/net/designware.c
> +++ b/drivers/net/designware.c
> @@ -368,11 +368,13 @@ static int configure_phy(struct eth_device *dev)
> {
> struct dw_eth_dev *priv = dev->priv;
> int phy_addr;
> - u16 bmcr, ctrl;
> + u16 bmcr;
> #if defined(CONFIG_DW_AUTONEG)
> u16 bmsr;
> u32 timeout;
> u16 anlpar, btsr;
> +#else
> + u16 ctrl;
> #endif
>
> #if defined(CONFIG_DW_SEARCH_PHY)
Reviewed-by: Vipin Kumar <vipin.kumar@st.com>
Regards
Vipin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
@ 2011-06-06 4:21 ` Vipin Kumar
2011-07-25 20:21 ` Wolfgang Denk
1 sibling, 0 replies; 6+ messages in thread
From: Vipin Kumar @ 2011-06-06 4:21 UTC (permalink / raw)
To: u-boot
On 6/2/2011 8:49 PM, Mike Frysinger wrote:
> When CONFIG_DW_SEARCH_PHY is disabled, the local phy_addr variable never
> gets initialized causes random behavior at runtime and a gcc warning. So
> set it by default to the stored phy address.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> drivers/net/designware.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/designware.c b/drivers/net/designware.c
> index 96f20d1..bfa4384 100644
> --- a/drivers/net/designware.c
> +++ b/drivers/net/designware.c
> @@ -383,6 +383,8 @@ static int configure_phy(struct eth_device *dev)
> priv->address = phy_addr;
> else
> return -1;
> +#else
> + phy_addr = priv->address;
> #endif
> if (dw_reset_phy(dev) < 0)
> return -1;
Reviewed-by: Vipin Kumar <vipin.kumar@st.com>
Thanks Mike
Regards
Vipin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled
2011-06-02 15:19 [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Mike Frysinger
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
2011-06-06 4:21 ` [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Vipin Kumar
@ 2011-07-25 20:20 ` Wolfgang Denk
2 siblings, 0 replies; 6+ messages in thread
From: Wolfgang Denk @ 2011-07-25 20:20 UTC (permalink / raw)
To: u-boot
Dear Mike Frysinger,
In message <1307027978-13418-1-git-send-email-vapier@gentoo.org> you wrote:
> The ctrl variable is only used when autoneg support is disabled, so only
> declare it under those conditions to avoid an unused variable warning.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> drivers/net/designware.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"No proper program contains an indication which as an operator-
applied occurrence identifies an operator-defining occurrence which
as an indication-applied occurrence identifies an indication-defining
occurrence different from the one identified by the given indication
as an indication-applied occurrence." - ALGOL 68 Report
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
2011-06-06 4:21 ` Vipin Kumar
@ 2011-07-25 20:21 ` Wolfgang Denk
1 sibling, 0 replies; 6+ messages in thread
From: Wolfgang Denk @ 2011-07-25 20:21 UTC (permalink / raw)
To: u-boot
Dear Mike Frysinger,
In message <1307027978-13418-2-git-send-email-vapier@gentoo.org> you wrote:
> When CONFIG_DW_SEARCH_PHY is disabled, the local phy_addr variable never
> gets initialized causes random behavior at runtime and a gcc warning. So
> set it by default to the stored phy address.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> drivers/net/designware.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
Applied (with minimal changes to the commit message which is missing
a 'which" or so), thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I see that Microsoft's campaign to destroy all knowledge of any
operating environment but its own environment-of-the-year has
succeeded in creating a generation of users who don't understand the
concept of a shell...
-- L. Peter Deutsch in <m0x5jNX-000R2UC@lamp.aladdin.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-07-25 20:21 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-02 15:19 [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Mike Frysinger
2011-06-02 15:19 ` [U-Boot] [PATCH] net: designware: fix uninitialized phy_addr usage Mike Frysinger
2011-06-06 4:21 ` Vipin Kumar
2011-07-25 20:21 ` Wolfgang Denk
2011-06-06 4:21 ` [U-Boot] [PATCH] net: designware: fix unused warning when CONFIG_DW_AUTONEG is enabled Vipin Kumar
2011-07-25 20:20 ` Wolfgang Denk
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.