On Mon, Feb 08, 2016 at 12:23:07PM +0000, Estrin, Alex wrote: > > 2. This function declared as a void, it is an error to add "return 0". > > > ipoib_mcast_join() is declared as int. Please see a very first change. > > > -static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast) > > +static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast) > You are right. Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html