On Wed, Apr 23, 2014 at 5:24 PM, K. Y. Srinivasan <k...@microsoft.com> wrote:
> drivers/net/hyperv/hyperv_net.h | 14 +++ > drivers/net/hyperv/netvsc.c | 226 > +++++++++++++++++++++++++++++++++++++-- > drivers/net/hyperv/netvsc_drv.c | 3 +-> 3 files changed, 234 > insertions(+), 9 deletions(-) I just looked over netvsc.c and it could definitely use a more consistent coding style. Your use of goto exit/cleanup in some functions and returning directly on errors in others could use a cleanup. Please consider doing that while you are touching those files. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/