diff mbox

[1/1] drivers: net: cpsw: fix dt probe for one port ethernet

Message ID 1385969019-12880-1-git-send-email-mugunthanvnm@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mugunthan V N Dec. 2, 2013, 7:23 a.m. UTC
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>
---
 drivers/net/ethernet/ti/cpsw.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Miller Dec. 2, 2013, 9:56 p.m. UTC | #1
From: Mugunthan V N <mugunthanvnm@ti.com>
Date: Mon, 2 Dec 2013 12:53:39 +0530

> 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>

Of course, this patch assumes that slaves are pinned out in order.

But, no matter, this makes more configurations work than before so
it's a good patch.

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index 90d41d2..68ef9f2 100644
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -1813,6 +1813,8 @@  static int cpsw_probe_dt(struct cpsw_platform_data *data,
 		}
 
 		i++;
+		if (i == data->slaves)
+			break;
 	}
 
 	return 0;