Message ID | 1386821345-30555-1-git-send-email-balbi@ti.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
Balbi On Thursday 12 December 2013 09:39 AM, Felipe Balbi wrote: > From: Mugunthan V N <mugunthanvnm@ti.com> > > When only one port of the two port is pinned out, then dt probe is failing > because second port phy is not found. fixing this by checking the number of > slaves and breaking the loop. > > Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com> > Signed-off-by: Felipe Balbi <balbi@ti.com> This patch is already applied to net branch and its commit id is 3a27bfac17fe375539c4e0a53478679645eb5ae2. Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Thu, Dec 12, 2013 at 01:41:11PM +0530, Mugunthan V N wrote: > On Thursday 12 December 2013 09:39 AM, Felipe Balbi wrote: > > From: Mugunthan V N <mugunthanvnm@ti.com> > > > > When only one port of the two port is pinned out, then dt probe is failing > > because second port phy is not found. fixing this by checking the number of > > slaves and breaking the loop. > > > > Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com> > > Signed-off-by: Felipe Balbi <balbi@ti.com> > > This patch is already applied to net branch and its commit id is > 3a27bfac17fe375539c4e0a53478679645eb5ae2. Ok, cool. Should it be backported to v3.12 stable tree ? I can see it's applicable there too.
diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 7536a4c..a91f0c9 100644 --- a/drivers/net/ethernet/ti/cpsw.c +++ b/drivers/net/ethernet/ti/cpsw.c @@ -1816,6 +1816,8 @@ static int cpsw_probe_dt(struct cpsw_platform_data *data, } i++; + if (i == data->slaves) + break; } return 0;