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