* [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy()
@ 2014-05-23 21:33 Christian Engelmayer
2014-05-23 21:57 ` Florian Fainelli
[not found] ` <1400880835-29537-1-git-send-email-cengelma-RbZlAiThDcE@public.gmane.org>
0 siblings, 2 replies; 3+ messages in thread
From: Christian Engelmayer @ 2014-05-23 21:33 UTC (permalink / raw)
To: netdev
Cc: s.hauer, f.fainelli, grant.likely, robh+dt, devicetree,
linux-kernel, Christian Engelmayer
Commit de906af1 (net: phy: make of_set_phy_supported work with genphy driver)
removed the last user of variable 'max_speed' in function
of_mdiobus_register_phy(), leading to compile warning "unused variable
‘max_speed’ [-Wunused-variable]". Thus remove it.
Signed-off-by: Christian Engelmayer <cengelma@gmx.at>
---
Compile tested. Applies against branch master in tree
git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
---
drivers/of/of_mdio.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c
index b857094..7c6e277 100644
--- a/drivers/of/of_mdio.c
+++ b/drivers/of/of_mdio.c
@@ -46,7 +46,6 @@ static int of_mdiobus_register_phy(struct mii_bus *mdio, struct device_node *chi
struct phy_device *phy;
bool is_c45;
int rc;
- u32 max_speed = 0;
u32 phy_id;
is_c45 = of_device_is_compatible(child,
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy()
2014-05-23 21:33 [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy() Christian Engelmayer
@ 2014-05-23 21:57 ` Florian Fainelli
[not found] ` <1400880835-29537-1-git-send-email-cengelma-RbZlAiThDcE@public.gmane.org>
1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2014-05-23 21:57 UTC (permalink / raw)
To: Christian Engelmayer
Cc: netdev, Sascha Hauer, Grant Likely, Rob Herring, devicetree,
linux-kernel
2014-05-23 14:33 GMT-07:00 Christian Engelmayer <cengelma@gmx.at>:
> Commit de906af1 (net: phy: make of_set_phy_supported work with genphy driver)
> removed the last user of variable 'max_speed' in function
> of_mdiobus_register_phy(), leading to compile warning "unused variable
> ‘max_speed’ [-Wunused-variable]". Thus remove it.
>
> Signed-off-by: Christian Engelmayer <cengelma@gmx.at>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
> Compile tested. Applies against branch master in tree
> git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
> ---
> drivers/of/of_mdio.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c
> index b857094..7c6e277 100644
> --- a/drivers/of/of_mdio.c
> +++ b/drivers/of/of_mdio.c
> @@ -46,7 +46,6 @@ static int of_mdiobus_register_phy(struct mii_bus *mdio, struct device_node *chi
> struct phy_device *phy;
> bool is_c45;
> int rc;
> - u32 max_speed = 0;
> u32 phy_id;
>
> is_c45 = of_device_is_compatible(child,
> --
> 1.9.1
>
--
Florian
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy()
[not found] ` <1400880835-29537-1-git-send-email-cengelma-RbZlAiThDcE@public.gmane.org>
@ 2014-05-27 11:16 ` Grant Likely
0 siblings, 0 replies; 3+ messages in thread
From: Grant Likely @ 2014-05-27 11:16 UTC (permalink / raw)
To: Christian Engelmayer, netdev-u79uwXL29TY76Z2rM5mHXA
Cc: s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ,
f.fainelli-Re5JQEeQqe8AvxtiuMwx3w,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, Christian Engelmayer
On Fri, 23 May 2014 23:33:55 +0200, Christian Engelmayer <cengelma@gmx.at> wrote:
> Commit de906af1 (net: phy: make of_set_phy_supported work with genphy driver)
> removed the last user of variable 'max_speed' in function
> of_mdiobus_register_phy(), leading to compile warning "unused variable
> âmax_speedâ [-Wunused-variable]". Thus remove it.
>
> Signed-off-by: Christian Engelmayer <cengelma-RbZlAiThDcE@public.gmane.org>
> ---
> Compile tested. Applies against branch master in tree
> git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
Acked-by: Grant Likely <grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
g.
> ---
> drivers/of/of_mdio.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c
> index b857094..7c6e277 100644
> --- a/drivers/of/of_mdio.c
> +++ b/drivers/of/of_mdio.c
> @@ -46,7 +46,6 @@ static int of_mdiobus_register_phy(struct mii_bus *mdio, struct device_node *chi
> struct phy_device *phy;
> bool is_c45;
> int rc;
> - u32 max_speed = 0;
> u32 phy_id;
>
> is_c45 = of_device_is_compatible(child,
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-05-27 11:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-23 21:33 [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy() Christian Engelmayer
2014-05-23 21:57 ` Florian Fainelli
[not found] ` <1400880835-29537-1-git-send-email-cengelma-RbZlAiThDcE@public.gmane.org>
2014-05-27 11:16 ` Grant Likely
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).