DPDK patches and discussions
 help / color / mirror / Atom feed
From: Akhil Goyal <akhil.goyal@nxp.com>
To: "Trahe, Fiona" <fiona.trahe@intel.com>,
	Rami Rosen <ramirose@gmail.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "Yigit, Ferruh" <ferruh.yigit@intel.com>,
	"De Lara Guarch, Pablo" <pablo.de.lara.guarch@intel.com>,
	"Mcnamara, John" <john.mcnamara@intel.com>,
	 "Kovacevic, Marko" <marko.kovacevic@intel.com>,
	"stable@dpdk.org" <stable@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] doc: add guideines for initial PMD submission
Date: Wed, 10 Apr 2019 10:28:16 +0000	[thread overview]
Message-ID: <VI1PR04MB489397D1A6C4CDD4C4636FA6E62E0@VI1PR04MB4893.eurprd04.prod.outlook.com> (raw)

Hi Rami,

> >
> > This patch for DPDK Contributor's Guidelines indicates the repos
> > against which a new PMD should be prepared; for new network ethernet
> > PMDs it should be dpdk-next-net, and for new crypto PMDs
> > it should be dpdk-next-crypto. Though this may seem obvious, it
> > is not mentioned in DPDK documentation.
> >
> > Cc: stable@dpdk.org
> > Signed-off-by: Rami Rosen <ramirose@gmail.com>
> > ---
> >  doc/guides/contributing/patches.rst | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/doc/guides/contributing/patches.rst
> b/doc/guides/contributing/patches.rst
> > index d8404e623..4dde8e724 100644
> > --- a/doc/guides/contributing/patches.rst
> > +++ b/doc/guides/contributing/patches.rst
> > @@ -148,6 +148,10 @@ Make your planned changes in the cloned ``dpdk``
> repo. Here are some
> > guidelines
> >
> >  * If you add new files or directories you should add your name to the
> ``MAINTAINERS`` file.
> >
> > +* If you add a new network PMD you should prepare the initial submission
> against dpdk-next-net repo.
> > +
> > +* If you add a new crypto PMD you should prepare the initial submission
> against dpdk-next-crypto repo.
> [Fiona] Good idea to add this patch. However despite the name crypto is actual
> broader.... so maybe
> "If you add a new crypto or compression PMD.... "

I believe this information is already captured in the MAINTAINERS file about which PMD goes to which tree.
If we need to document it in the documentation as well, then probably all other stuff should also be captured (like event and bbdev)

-Akhil

             reply	other threads:[~2019-04-10 10:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-10 10:28 Akhil Goyal [this message]
2019-04-10 10:28 ` Akhil Goyal
2019-04-10 14:40 ` Rami Rosen
2019-04-10 14:40   ` Rami Rosen
  -- strict thread matches above, loose matches on Subject: below --
2019-04-10 14:49 Akhil Goyal
2019-04-10 14:49 ` Akhil Goyal
2019-04-09 17:36 Rami Rosen
2019-04-09 17:36 ` Rami Rosen
2019-04-10 10:06 ` Trahe, Fiona
2019-04-10 10:06   ` Trahe, Fiona
2019-04-10 14:36   ` Rami Rosen
2019-04-10 14:36     ` Rami Rosen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=VI1PR04MB489397D1A6C4CDD4C4636FA6E62E0@VI1PR04MB4893.eurprd04.prod.outlook.com \
    --to=akhil.goyal@nxp.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=fiona.trahe@intel.com \
    --cc=john.mcnamara@intel.com \
    --cc=marko.kovacevic@intel.com \
    --cc=pablo.de.lara.guarch@intel.com \
    --cc=ramirose@gmail.com \
    --cc=stable@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).