patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd
@ 2018-07-23 11:27 Shreyansh Jain
  2018-07-23 20:49 ` Ferruh Yigit
  2018-07-25 11:31 ` [dpdk-stable] [PATCH v2] " Shreyansh Jain
  0 siblings, 2 replies; 5+ messages in thread
From: Shreyansh Jain @ 2018-07-23 11:27 UTC (permalink / raw)
  To: dev
  Cc: bernard.iremonger, jingjing.wu, wenzhuo.lu, Shreyansh Jain,
	john.mcnamara, stable

Fixes: e76d7a768ce0 ("doc: fix syntax in testpmd user guide")
Cc: john.mcnamara@intel.com
Cc: stable@dpdk.org

Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
---
Note: Though the git blame suggest e76d7a768c as the last modification
      for fixes, this actually existed when testpmd doc was added.

 doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 1d6bdf9b3..16af95035 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -1993,7 +1993,7 @@ Create a new bonding device::
 
 For example, to create a bonded device in mode 1 on socket 0::
 
-   testpmd> create bonded 1 0
+   testpmd> create bonded device 1 0
    created new bonded device (port X)
 
 add bonding slave
-- 
2.17.1

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd
  2018-07-23 11:27 [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd Shreyansh Jain
@ 2018-07-23 20:49 ` Ferruh Yigit
  2018-07-25  9:00   ` Iremonger, Bernard
  2018-07-25 11:31 ` [dpdk-stable] [PATCH v2] " Shreyansh Jain
  1 sibling, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2018-07-23 20:49 UTC (permalink / raw)
  To: Shreyansh Jain, dev
  Cc: bernard.iremonger, jingjing.wu, wenzhuo.lu, john.mcnamara, stable

On 7/23/2018 12:27 PM, Shreyansh Jain wrote:
> Fixes: e76d7a768ce0 ("doc: fix syntax in testpmd user guide")
> Cc: john.mcnamara@intel.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> ---
> Note: Though the git blame suggest e76d7a768c as the last modification
>       for fixes, this actually existed when testpmd doc was added.

You are right, I think fixes should be initial commit:
Fixes: ac718398f477 ("doc: testpmd application user guide")

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd
  2018-07-23 20:49 ` Ferruh Yigit
@ 2018-07-25  9:00   ` Iremonger, Bernard
  0 siblings, 0 replies; 5+ messages in thread
From: Iremonger, Bernard @ 2018-07-25  9:00 UTC (permalink / raw)
  To: Yigit, Ferruh, Shreyansh Jain, dev
  Cc: Wu, Jingjing, Lu, Wenzhuo, Mcnamara, John, stable

Hi Shreyansh,

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Monday, July 23, 2018 9:49 PM
> To: Shreyansh Jain <shreyansh.jain@nxp.com>; dev@dpdk.org
> Cc: Iremonger, Bernard <bernard.iremonger@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>;
> Mcnamara, John <john.mcnamara@intel.com>; stable@dpdk.org
> Subject: Re: [dpdk-stable] [PATCH] doc: fix incorrect create bonding
> command in testpmd
> 
> On 7/23/2018 12:27 PM, Shreyansh Jain wrote:
> > Fixes: e76d7a768ce0 ("doc: fix syntax in testpmd user guide")
> > Cc: john.mcnamara@intel.com
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > ---
> > Note: Though the git blame suggest e76d7a768c as the last modification
> >       for fixes, this actually existed when testpmd doc was added.
> 
> You are right, I think fixes should be initial commit:
> Fixes: ac718398f477 ("doc: testpmd application user guide")
> 
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Could you send a v2 with the correct Fixes line.

Otherwise

Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [dpdk-stable] [PATCH v2] doc: fix incorrect create bonding command in testpmd
  2018-07-23 11:27 [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd Shreyansh Jain
  2018-07-23 20:49 ` Ferruh Yigit
@ 2018-07-25 11:31 ` Shreyansh Jain
  2018-07-26 20:36   ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon
  1 sibling, 1 reply; 5+ messages in thread
From: Shreyansh Jain @ 2018-07-25 11:31 UTC (permalink / raw)
  To: dev
  Cc: ferruh.yigit, bernard.iremonger, jingjing.wu, wenzhuo.lu,
	john.mcnamara, Shreyansh Jain, stable

Fixes: ac718398f477 ("doc: testpmd application user guide")
Cc: bernard.iremonger@intel.com
Cc: stable@dpdk.org

Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
---
v2:
   Update the Fixes line to represent the first commit
   rather than the formatting patch (e76d7a76).

 doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 1d6bdf9b3..16af95035 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -1993,7 +1993,7 @@ Create a new bonding device::
 
 For example, to create a bonded device in mode 1 on socket 0::
 
-   testpmd> create bonded 1 0
+   testpmd> create bonded device 1 0
    created new bonded device (port X)
 
 add bonding slave
-- 
2.17.1

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v2] doc: fix incorrect create bonding command in testpmd
  2018-07-25 11:31 ` [dpdk-stable] [PATCH v2] " Shreyansh Jain
@ 2018-07-26 20:36   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2018-07-26 20:36 UTC (permalink / raw)
  To: Shreyansh Jain
  Cc: dev, ferruh.yigit, bernard.iremonger, jingjing.wu, wenzhuo.lu,
	john.mcnamara, stable

25/07/2018 13:31, Shreyansh Jain:
> Fixes: ac718398f477 ("doc: testpmd application user guide")
> Cc: bernard.iremonger@intel.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
> Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-07-26 20:36 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-23 11:27 [dpdk-stable] [PATCH] doc: fix incorrect create bonding command in testpmd Shreyansh Jain
2018-07-23 20:49 ` Ferruh Yigit
2018-07-25  9:00   ` Iremonger, Bernard
2018-07-25 11:31 ` [dpdk-stable] [PATCH v2] " Shreyansh Jain
2018-07-26 20:36   ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon

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).