* [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs
@ 2018-04-20 8:16 Pavan Nikhilesh
2018-04-20 9:22 ` Kovacevic, Marko
2018-04-20 9:24 ` Kovacevic, Marko
0 siblings, 2 replies; 4+ messages in thread
From: Pavan Nikhilesh @ 2018-04-20 8:16 UTC (permalink / raw)
To: thomas, jerin.jacob, santosh.shukla, john.mcnamara, marko.kovacevic
Cc: dev, Pavan Nikhilesh
Fixes: 26cb0a72412f ("doc: add octeontx platform guide")
Fixes: 48191dde6acf ("doc: add octeontx mempool guide")
Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
---
doc/guides/eventdevs/octeontx.rst | 2 +-
doc/guides/platform/octeontx.rst | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/doc/guides/eventdevs/octeontx.rst b/doc/guides/eventdevs/octeontx.rst
index f77bc5c54..3dcf6afbd 100644
--- a/doc/guides/eventdevs/octeontx.rst
+++ b/doc/guides/eventdevs/octeontx.rst
@@ -39,7 +39,7 @@ Supported OCTEONTX SoCs
Prerequisites
-------------
-See :doc: `../platform/octeontx` for setup information.
+See :doc:`../platform/octeontx` for setup information.
Pre-Installation Configuration
------------------------------
diff --git a/doc/guides/platform/octeontx.rst b/doc/guides/platform/octeontx.rst
index dc1aa4fe1..535504998 100644
--- a/doc/guides/platform/octeontx.rst
+++ b/doc/guides/platform/octeontx.rst
@@ -16,11 +16,11 @@ Common Offload HW Block Drivers
-------------------------------
1. **Eventdev Driver**
- See :doc: `../eventdevs/octeontx.rst` for octeontx ssovf eventdev driver
+ See :doc:`../eventdevs/octeontx.rst` for octeontx ssovf eventdev driver
information.
2. **Mempool Driver**
- See :doc: `../mempool/octeontx.rst` for octeontx fpavf mempool driver
+ See :doc:`../mempool/octeontx.rst` for octeontx fpavf mempool driver
information.
Steps To Setup Platform
--
2.17.0
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs
2018-04-20 8:16 [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs Pavan Nikhilesh
@ 2018-04-20 9:22 ` Kovacevic, Marko
2018-04-20 9:24 ` Kovacevic, Marko
1 sibling, 0 replies; 4+ messages in thread
From: Kovacevic, Marko @ 2018-04-20 9:22 UTC (permalink / raw)
To: Pavan Nikhilesh, thomas, jerin.jacob, santosh.shukla, Mcnamara, John; +Cc: dev
>
> Fixes: 26cb0a72412f ("doc: add octeontx platform guide")
> Fixes: 48191dde6acf ("doc: add octeontx mempool guide")
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> ---
> doc/guides/eventdevs/octeontx.rst | 2 +-
> doc/guides/platform/octeontx.rst | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/doc/guides/eventdevs/octeontx.rst
> b/doc/guides/eventdevs/octeontx.rst
> index f77bc5c54..3dcf6afbd 100644
> --- a/doc/guides/eventdevs/octeontx.rst
> +++ b/doc/guides/eventdevs/octeontx.rst
> @@ -39,7 +39,7 @@ Supported OCTEONTX SoCs Prerequisites
> -------------
>
> -See :doc: `../platform/octeontx` for setup information.
> +See :doc:`../platform/octeontx` for setup information.
>
> Pre-Installation Configuration
> ------------------------------
> diff --git a/doc/guides/platform/octeontx.rst
> b/doc/guides/platform/octeontx.rst
> index dc1aa4fe1..535504998 100644
> --- a/doc/guides/platform/octeontx.rst
> +++ b/doc/guides/platform/octeontx.rst
> @@ -16,11 +16,11 @@ Common Offload HW Block Drivers
> -------------------------------
>
> 1. **Eventdev Driver**
> - See :doc: `../eventdevs/octeontx.rst` for octeontx ssovf eventdev driver
> + See :doc:`../eventdevs/octeontx.rst` for octeontx ssovf eventdev
> + driver
> information.
>
> 2. **Mempool Driver**
> - See :doc: `../mempool/octeontx.rst` for octeontx fpavf mempool driver
> + See :doc:`../mempool/octeontx.rst` for octeontx fpavf mempool driver
> information.
Hi Pavan,
Can you please remove the .rst from the two files ":doc: `../eventdevs/octeontx.rst`" , :doc: `../mempool/octeontx.rst`
as it is failing to find the html part if the rst is added onto the end and in not referenced to the file.
And as I was looking at this I noticed at the bottom there is the doc: `../linux_gsg/indes.rst`
Can you remove the .rst from it too and just add the fixes line to your patch.
Otherwise you can add my ack after as iv tested it and it works.
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs
2018-04-20 8:16 [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs Pavan Nikhilesh
2018-04-20 9:22 ` Kovacevic, Marko
@ 2018-04-20 9:24 ` Kovacevic, Marko
2018-04-20 9:28 ` Pavan Nikhilesh
1 sibling, 1 reply; 4+ messages in thread
From: Kovacevic, Marko @ 2018-04-20 9:24 UTC (permalink / raw)
To: Pavan Nikhilesh, thomas, jerin.jacob, santosh.shukla, Mcnamara, John; +Cc: dev
And there is a space you need to just get rid of at the bottom
:doc: `../linux_gsg/index.rst`
:doc:`../linux_gsg/index`
Marko K
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs
2018-04-20 9:24 ` Kovacevic, Marko
@ 2018-04-20 9:28 ` Pavan Nikhilesh
0 siblings, 0 replies; 4+ messages in thread
From: Pavan Nikhilesh @ 2018-04-20 9:28 UTC (permalink / raw)
To: Kovacevic, Marko, jerin.jacob, santosh.shukla, Mcnamara, John; +Cc: dev
Hi Marko,
Will send out v2 with the appropriate changes.
Thanks,
Pavan
On Fri, Apr 20, 2018 at 09:24:34AM +0000, Kovacevic, Marko wrote:
> And there is a space you need to just get rid of at the bottom
>
> :doc: `../linux_gsg/index.rst`
>
> :doc:`../linux_gsg/index`
>
> Marko K
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-04-20 9:28 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-20 8:16 [dpdk-dev] [PATCH] doc: fix minor typos in OcteonTx platform and eventdev docs Pavan Nikhilesh
2018-04-20 9:22 ` Kovacevic, Marko
2018-04-20 9:24 ` Kovacevic, Marko
2018-04-20 9:28 ` Pavan Nikhilesh
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).