On Wednesday 22 January 2014 07:41 AM, David Miller wrote: > From: Mugunthan V N <mugunthanvnm@xxxxxx> > Date: Mon, 20 Jan 2014 17:38:38 +0530 > >> + if (!enable && ((priv->slaves[0].ndev->flags & IFF_PROMISC) || >> + (priv->slaves[1].ndev->flags & IFF_PROMISC))) { > This assumption that there are exactly 2 slaves is not valid. > > Use the appropriate for_each_slave() et al. abstractions to access > the slaves. Will fix this in next version. Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html