Re: [PATCH] of: mdio: fix compile warning in of_mdiobus_register_phy()

From: Grant Likely
Date: Tue May 27 2014 - 07:16:39 EST


On Fri, 23 May 2014 23:33:55 +0200, Christian Engelmayer <cengelma@xxxxxx> 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@xxxxxx>
> ---
> 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@xxxxxxxxxx>

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 linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/