DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1] doc: update bbdev guide
@ 2020-08-20 20:58 Nicolas Chautru
  2020-09-15  9:56 ` Aidan Goddard
  2020-10-06 20:53 ` Akhil Goyal
  0 siblings, 2 replies; 5+ messages in thread
From: Nicolas Chautru @ 2020-08-20 20:58 UTC (permalink / raw)
  To: dev; +Cc: dave.burley, Nicolas Chautru

Clarify the capability assumptions for LLR and HARQ
compression format.
Correct one historical typo.

Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
---
 doc/guides/prog_guide/bbdev.rst | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/guides/prog_guide/bbdev.rst b/doc/guides/prog_guide/bbdev.rst
index d39167a..6b2bd54 100644
--- a/doc/guides/prog_guide/bbdev.rst
+++ b/doc/guides/prog_guide/bbdev.rst
@@ -13,7 +13,7 @@ optimized libraries to appear as virtual bbdev devices.
 The functional scope of the BBDEV library are those functions in relation to
 the 3GPP Layer 1 signal processing (channel coding, modulation, ...).
 
-The framework currently only supports Turbo Code FEC function.
+The framework currently only supports FEC function.
 
 
 Design Principles
@@ -927,9 +927,11 @@ given below.
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION                                |
 | Set if a device supports input/output HARQ compression             |
+| Data is packed as 6 bits by dropping and saturating the MSBs       |
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_LLR_COMPRESSION                                      |
 | Set if a device supports input LLR compression                     |
+| Data is packed as 6 bits by dropping and saturating the MSBs       |
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_IN_ENABLE                       |
 | Set if a device supports HARQ input to device's internal memory    |
-- 
1.8.3.1


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

* Re: [dpdk-dev] [PATCH v1] doc: update bbdev guide
  2020-08-20 20:58 [dpdk-dev] [PATCH v1] doc: update bbdev guide Nicolas Chautru
@ 2020-09-15  9:56 ` Aidan Goddard
  2020-10-06 20:53 ` Akhil Goyal
  1 sibling, 0 replies; 5+ messages in thread
From: Aidan Goddard @ 2020-09-15  9:56 UTC (permalink / raw)
  To: Nicolas Chautru, dev; +Cc: Dave Burley

Acked-by: Aidan Goddard <aidan.goddard@accelercomm.com>


From: dev <dev-bounces@dpdk.org> on behalf of Nicolas Chautru <nicolas.chautru@intel.com>
Sent: 20 August 2020 21:58
To: dev@dpdk.org <dev@dpdk.org>
Cc: Dave Burley <dave.burley@accelercomm.com>; Nicolas Chautru <nicolas.chautru@intel.com>
Subject: [dpdk-dev] [PATCH v1] doc: update bbdev guide 
 
Clarify the capability assumptions for LLR and HARQ
compression format.
Correct one historical typo.

Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
---
 doc/guides/prog_guide/bbdev.rst | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/guides/prog_guide/bbdev.rst b/doc/guides/prog_guide/bbdev.rst
index d39167a..6b2bd54 100644
--- a/doc/guides/prog_guide/bbdev.rst
+++ b/doc/guides/prog_guide/bbdev.rst
@@ -13,7 +13,7 @@ optimized libraries to appear as virtual bbdev devices.
 The functional scope of the BBDEV library are those functions in relation to
 the 3GPP Layer 1 signal processing (channel coding, modulation, ...).
 
-The framework currently only supports Turbo Code FEC function.
+The framework currently only supports FEC function.
 
 
 Design Principles
@@ -927,9 +927,11 @@ given below.
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION                                |
 | Set if a device supports input/output HARQ compression             |
+| Data is packed as 6 bits by dropping and saturating the MSBs       |
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_LLR_COMPRESSION                                      |
 | Set if a device supports input LLR compression                     |
+| Data is packed as 6 bits by dropping and saturating the MSBs       |
 +--------------------------------------------------------------------+
 |RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_IN_ENABLE                       |
 | Set if a device supports HARQ input to device's internal memory    |
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH v1] doc: update bbdev guide
  2020-08-20 20:58 [dpdk-dev] [PATCH v1] doc: update bbdev guide Nicolas Chautru
  2020-09-15  9:56 ` Aidan Goddard
@ 2020-10-06 20:53 ` Akhil Goyal
  2020-10-06 21:00   ` Chautru, Nicolas
  1 sibling, 1 reply; 5+ messages in thread
From: Akhil Goyal @ 2020-10-06 20:53 UTC (permalink / raw)
  To: Nicolas Chautru, dev; +Cc: dave.burley

> Subject: [dpdk-dev] [PATCH v1] doc: update bbdev guide
> 
> Clarify the capability assumptions for LLR and HARQ
> compression format.
> Correct one historical typo.
> 
> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> ---
Should this be cced to stable for back port?


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

* Re: [dpdk-dev] [PATCH v1] doc: update bbdev guide
  2020-10-06 20:53 ` Akhil Goyal
@ 2020-10-06 21:00   ` Chautru, Nicolas
  2020-10-06 21:10     ` Akhil Goyal
  0 siblings, 1 reply; 5+ messages in thread
From: Chautru, Nicolas @ 2020-10-06 21:00 UTC (permalink / raw)
  To: Akhil Goyal, dev; +Cc: dave.burley

Hi Akhil, 

> -----Original Message-----
> From: Akhil Goyal <akhil.goyal@nxp.com>
> Sent: Tuesday, October 6, 2020 1:53 PM
> To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org
> Cc: dave.burley@accelercomm.com
> Subject: RE: [dpdk-dev] [PATCH v1] doc: update bbdev guide
> 
> > Subject: [dpdk-dev] [PATCH v1] doc: update bbdev guide
> >
> > Clarify the capability assumptions for LLR and HARQ compression
> > format.
> > Correct one historical typo.
> >
> > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> > ---
> Should this be cced to stable for back port?

I have no strong opinion given this is minor doc clarification update. 
Also the related feature is not really exercised until 20.11 really (with new PMD).
Still if you think it is required the change would refer to that commit 3f3f608142cf48
Let me know if I should resubmit.

Thanks
Nic


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

* Re: [dpdk-dev] [PATCH v1] doc: update bbdev guide
  2020-10-06 21:00   ` Chautru, Nicolas
@ 2020-10-06 21:10     ` Akhil Goyal
  0 siblings, 0 replies; 5+ messages in thread
From: Akhil Goyal @ 2020-10-06 21:10 UTC (permalink / raw)
  To: Chautru, Nicolas, dev; +Cc: dave.burley

Hi Nic,

> Hi Akhil,
> 
> I have no strong opinion given this is minor doc clarification update.
> Also the related feature is not really exercised until 20.11 really (with new PMD).
> Still if you think it is required the change would refer to that commit
> 3f3f608142cf48
> Let me know if I should resubmit.

I applied it as is.
Thanks.

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

end of thread, other threads:[~2020-10-06 21:10 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-20 20:58 [dpdk-dev] [PATCH v1] doc: update bbdev guide Nicolas Chautru
2020-09-15  9:56 ` Aidan Goddard
2020-10-06 20:53 ` Akhil Goyal
2020-10-06 21:00   ` Chautru, Nicolas
2020-10-06 21:10     ` Akhil Goyal

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git