On Mon, Jan 08, 2018 at 04:20:04PM +0530, Vasundhara Volam wrote: > Hi Andy, > > If you are re-doing the patch, could you modify a minor comment below? > > On Sat, Jan 6, 2018 at 4:28 AM, Andy Gospodarek <a...@greyhouse.net> wrote: > > > diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_dim.c > > b/drivers/net/ethernet/broadcom/bnxt/bnxt_dim.c > > new file mode 100644 > > index 0000000..156e025 > > --- /dev/null > > +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_dim.c > > @@ -0,0 +1,32 @@ > > +/* > > + * Copyright (c) 2017 Broadcom Limited > You have to modify year here. > Also, First line driver description is missing "Broadcom NetXtreme-C/E > network driver." > Could you modify it?
Sure I could make those changes since it looks like I need a v3 anyway. Thanks for the review.