From: Daniel Trautmann <dtrautm...@ibhsoftec.com>
Date: Mon, 9 Nov 2015 20:24:14 +0100

> @@ -2039,19 +2039,35 @@ static int cpsw_probe_dt(struct cpsw_priv *priv,
>               priv->phy_node = of_parse_phandle(slave_node, "phy-handle", 0);
>               parp = of_get_property(slave_node, "phy_id", &lenp);
>               if ((parp == NULL) || (lenp != (sizeof(void *) * 2))) {
> -                     dev_err(&pdev->dev, "Missing slave[%d] phy_id 
> property\n", i);

I know this has nothing to do with your patch, but that length check
is completely bogus.

It should be "sizeof(__be32) * 2" not "sizeof(void *) * 2".
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to