2016-04-26 18:27, Rasesh Mody: > > From: Bruce Richardson [mailto:bruce.richardson at intel.com] > > Sent: Tuesday, April 26, 2016 6:03 AM > > > > On Mon, Apr 25, 2016 at 10:12:59PM -0700, Rasesh Mody wrote: > > > Signed-off-by: Harish Patil <harish.patil at qlogic.com> > > > Signed-off-by: Rasesh Mody <rasesh.mody at qlogic.com> > > > Signed-off-by: Sony Chacko <sony.chacko at qlogic.com> > > > --- > > > MAINTAINERS | 7 + > > > doc/guides/nics/index.rst | 1 + > > > doc/guides/nics/overview.rst | 86 +++++----- > > > doc/guides/nics/qede.rst | 314 > > +++++++++++++++++++++++++++++++++++++ > > > drivers/net/qede/LICENSE.qede_pmd | 28 ++++ > > > 5 files changed, 393 insertions(+), 43 deletions(-) create mode > > > 100644 doc/guides/nics/qede.rst create mode 100644 > > > drivers/net/qede/LICENSE.qede_pmd > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS index 1953ea2..ba4053a 100644 > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -332,6 +332,13 @@ M: Rasesh Mody <rasesh.mody at qlogic.com> > > > F: drivers/net/bnx2x/ > > > F: doc/guides/nics/bnx2x.rst > > > > > > +QLogic qede PMD > > > +M: Harish Patil <harish.patil at qlogic.com> > > > +M: Rasesh Mody <rasesh.mody at qlogic.com> > > > +M: Sony Chacko <sony.chacko at qlogic.com> > > > +F: drivers/net/qede/ > > > +F: doc/guides/nics/qede.rst > > > + > > > RedHat virtio > > > M: Huawei Xie <huawei.xie at intel.com> > > > M: Yuanhan Liu <yuanhan.liu at linux.intel.com> diff --git > > > a/doc/guides/nics/index.rst b/doc/guides/nics/index.rst index > > > 769f677..d67f056 100644 > > > --- a/doc/guides/nics/index.rst > > > +++ b/doc/guides/nics/index.rst > > > @@ -53,6 +53,7 @@ Network Interface Controller Drivers > > > vhost > > > vmxnet3 > > > pcap_ring > > > + qede > > > > Minor nit - this addition should be between "nfp" and "szedata2" as the > > list is > > in alphabetical order apart from the last entry which covers two virtual > > NICs > > in the one section. > > > > /Bruce > > During our v5 submission we received a comment to keep the logic order by > adding qede below bnx2x, hence the change. We can place it under "nfp" and > "szedata2" to maintain the alphabetical order.
I think you are talking of 2 different things: MAINTAINERS file et doc/guides/nics. For MAINTAINERS, this patch is right. For doc, it is not.
