DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: simplify baseband features matrix
@ 2022-01-05 17:40 Thomas Monjalon
  2022-01-05 20:12 ` Morten Brørup
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2022-01-05 17:40 UTC (permalink / raw)
  To: dev; +Cc: nipun.gupta, hemant.agrawal, gakhil, nicolas.chautru

The "feature" BBDEV API is useless as all baseband drivers
must implement it by definition.

The non-implemented features should not be marked with "N".
Keeping them blank is clearer to read in the resulting matrix.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 doc/guides/bbdevs/features/acc100.ini        | 1 -
 doc/guides/bbdevs/features/default.ini       | 1 -
 doc/guides/bbdevs/features/fpga_5gnr_fec.ini | 1 -
 doc/guides/bbdevs/features/fpga_lte_fec.ini  | 1 -
 doc/guides/bbdevs/features/la12xx.ini        | 4 ----
 doc/guides/bbdevs/features/null.ini          | 1 -
 doc/guides/bbdevs/features/turbo_sw.ini      | 1 -
 7 files changed, 10 deletions(-)

diff --git a/doc/guides/bbdevs/features/acc100.ini b/doc/guides/bbdevs/features/acc100.ini
index 642cd48818..61a31065e1 100644
--- a/doc/guides/bbdevs/features/acc100.ini
+++ b/doc/guides/bbdevs/features/acc100.ini
@@ -11,4 +11,3 @@ LDPC Encoder (5G)      = Y
 LLR/HARQ Compression   = Y
 External DDR Access    = Y
 HW Accelerated         = Y
-BBDEV API              = Y
diff --git a/doc/guides/bbdevs/features/default.ini b/doc/guides/bbdevs/features/default.ini
index 5fe267a625..494be5e400 100644
--- a/doc/guides/bbdevs/features/default.ini
+++ b/doc/guides/bbdevs/features/default.ini
@@ -13,4 +13,3 @@ LDPC Encoder (5G)      =
 LLR/HARQ Compression   =
 External DDR Access    =
 HW Accelerated         =
-BBDEV API              =
diff --git a/doc/guides/bbdevs/features/fpga_5gnr_fec.ini b/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
index 7a0b8d4e75..66c9139409 100644
--- a/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
+++ b/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
@@ -8,4 +8,3 @@ LDPC Decoder (5G)      = Y
 LDPC Encoder (5G)      = Y
 External DDR Access    = Y
 HW Accelerated         = Y
-BBDEV API              = Y
diff --git a/doc/guides/bbdevs/features/fpga_lte_fec.ini b/doc/guides/bbdevs/features/fpga_lte_fec.ini
index f1cfb924a2..c3c398312b 100644
--- a/doc/guides/bbdevs/features/fpga_lte_fec.ini
+++ b/doc/guides/bbdevs/features/fpga_lte_fec.ini
@@ -7,4 +7,3 @@
 Turbo Decoder (4G)     = Y
 Turbo Encoder (4G)     = Y
 HW Accelerated         = Y
-BBDEV API              = Y
diff --git a/doc/guides/bbdevs/features/la12xx.ini b/doc/guides/bbdevs/features/la12xx.ini
index 0aec5eecb6..70c11990b7 100644
--- a/doc/guides/bbdevs/features/la12xx.ini
+++ b/doc/guides/bbdevs/features/la12xx.ini
@@ -4,10 +4,6 @@
 ; Refer to default.ini for the full list of available PMD features.
 ;
 [Features]
-Turbo Decoder (4G)     = N
-Turbo Encoder (4G)     = N
 LDPC Decoder (5G)      = Y
 LDPC Encoder (5G)      = Y
-LLR/HARQ Compression   = N
 HW Accelerated         = Y
-BBDEV API              = Y
diff --git a/doc/guides/bbdevs/features/null.ini b/doc/guides/bbdevs/features/null.ini
index d9bbda9cf0..50648cdecb 100644
--- a/doc/guides/bbdevs/features/null.ini
+++ b/doc/guides/bbdevs/features/null.ini
@@ -4,4 +4,3 @@
 ; Refer to default.ini for the full list of available PMD features.
 ;
 [Features]
-BBDEV API              = Y
diff --git a/doc/guides/bbdevs/features/turbo_sw.ini b/doc/guides/bbdevs/features/turbo_sw.ini
index 2c7075e212..1d908d3ceb 100644
--- a/doc/guides/bbdevs/features/turbo_sw.ini
+++ b/doc/guides/bbdevs/features/turbo_sw.ini
@@ -8,4 +8,3 @@ Turbo Decoder (4G)     = Y
 Turbo Encoder (4G)     = Y
 LDPC Decoder (5G)      = Y
 LDPC Encoder (5G)      = Y
-BBDEV API              = Y
-- 
2.34.1


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

* RE: [PATCH] doc: simplify baseband features matrix
  2022-01-05 17:40 [PATCH] doc: simplify baseband features matrix Thomas Monjalon
@ 2022-01-05 20:12 ` Morten Brørup
  2022-02-08 21:02   ` Akhil Goyal
  0 siblings, 1 reply; 3+ messages in thread
From: Morten Brørup @ 2022-01-05 20:12 UTC (permalink / raw)
  To: Thomas Monjalon, dev; +Cc: nipun.gupta, hemant.agrawal, gakhil, nicolas.chautru

> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Wednesday, 5 January 2022 18.40
> 
> The "feature" BBDEV API is useless as all baseband drivers
> must implement it by definition.
> 
> The non-implemented features should not be marked with "N".
> Keeping them blank is clearer to read in the resulting matrix.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  doc/guides/bbdevs/features/acc100.ini        | 1 -
>  doc/guides/bbdevs/features/default.ini       | 1 -
>  doc/guides/bbdevs/features/fpga_5gnr_fec.ini | 1 -
>  doc/guides/bbdevs/features/fpga_lte_fec.ini  | 1 -
>  doc/guides/bbdevs/features/la12xx.ini        | 4 ----
>  doc/guides/bbdevs/features/null.ini          | 1 -
>  doc/guides/bbdevs/features/turbo_sw.ini      | 1 -
>  7 files changed, 10 deletions(-)
> 
> diff --git a/doc/guides/bbdevs/features/acc100.ini
> b/doc/guides/bbdevs/features/acc100.ini
> index 642cd48818..61a31065e1 100644
> --- a/doc/guides/bbdevs/features/acc100.ini
> +++ b/doc/guides/bbdevs/features/acc100.ini
> @@ -11,4 +11,3 @@ LDPC Encoder (5G)      = Y
>  LLR/HARQ Compression   = Y
>  External DDR Access    = Y
>  HW Accelerated         = Y
> -BBDEV API              = Y
> diff --git a/doc/guides/bbdevs/features/default.ini
> b/doc/guides/bbdevs/features/default.ini
> index 5fe267a625..494be5e400 100644
> --- a/doc/guides/bbdevs/features/default.ini
> +++ b/doc/guides/bbdevs/features/default.ini
> @@ -13,4 +13,3 @@ LDPC Encoder (5G)      =
>  LLR/HARQ Compression   =
>  External DDR Access    =
>  HW Accelerated         =
> -BBDEV API              =
> diff --git a/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
> b/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
> index 7a0b8d4e75..66c9139409 100644
> --- a/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
> +++ b/doc/guides/bbdevs/features/fpga_5gnr_fec.ini
> @@ -8,4 +8,3 @@ LDPC Decoder (5G)      = Y
>  LDPC Encoder (5G)      = Y
>  External DDR Access    = Y
>  HW Accelerated         = Y
> -BBDEV API              = Y
> diff --git a/doc/guides/bbdevs/features/fpga_lte_fec.ini
> b/doc/guides/bbdevs/features/fpga_lte_fec.ini
> index f1cfb924a2..c3c398312b 100644
> --- a/doc/guides/bbdevs/features/fpga_lte_fec.ini
> +++ b/doc/guides/bbdevs/features/fpga_lte_fec.ini
> @@ -7,4 +7,3 @@
>  Turbo Decoder (4G)     = Y
>  Turbo Encoder (4G)     = Y
>  HW Accelerated         = Y
> -BBDEV API              = Y
> diff --git a/doc/guides/bbdevs/features/la12xx.ini
> b/doc/guides/bbdevs/features/la12xx.ini
> index 0aec5eecb6..70c11990b7 100644
> --- a/doc/guides/bbdevs/features/la12xx.ini
> +++ b/doc/guides/bbdevs/features/la12xx.ini
> @@ -4,10 +4,6 @@
>  ; Refer to default.ini for the full list of available PMD features.
>  ;
>  [Features]
> -Turbo Decoder (4G)     = N
> -Turbo Encoder (4G)     = N
>  LDPC Decoder (5G)      = Y
>  LDPC Encoder (5G)      = Y
> -LLR/HARQ Compression   = N
>  HW Accelerated         = Y
> -BBDEV API              = Y
> diff --git a/doc/guides/bbdevs/features/null.ini
> b/doc/guides/bbdevs/features/null.ini
> index d9bbda9cf0..50648cdecb 100644
> --- a/doc/guides/bbdevs/features/null.ini
> +++ b/doc/guides/bbdevs/features/null.ini
> @@ -4,4 +4,3 @@
>  ; Refer to default.ini for the full list of available PMD features.
>  ;
>  [Features]
> -BBDEV API              = Y
> diff --git a/doc/guides/bbdevs/features/turbo_sw.ini
> b/doc/guides/bbdevs/features/turbo_sw.ini
> index 2c7075e212..1d908d3ceb 100644
> --- a/doc/guides/bbdevs/features/turbo_sw.ini
> +++ b/doc/guides/bbdevs/features/turbo_sw.ini
> @@ -8,4 +8,3 @@ Turbo Decoder (4G)     = Y
>  Turbo Encoder (4G)     = Y
>  LDPC Decoder (5G)      = Y
>  LDPC Encoder (5G)      = Y
> -BBDEV API              = Y
> --
> 2.34.1
> 

Acked-by: Morten Brørup <mb@smartsharesystems.com>


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

* RE: [PATCH] doc: simplify baseband features matrix
  2022-01-05 20:12 ` Morten Brørup
@ 2022-02-08 21:02   ` Akhil Goyal
  0 siblings, 0 replies; 3+ messages in thread
From: Akhil Goyal @ 2022-02-08 21:02 UTC (permalink / raw)
  To: Morten Brørup, Thomas Monjalon, dev
  Cc: nipun.gupta, hemant.agrawal, nicolas.chautru

> > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > Sent: Wednesday, 5 January 2022 18.40
> >
> > The "feature" BBDEV API is useless as all baseband drivers
> > must implement it by definition.
> >
> > The non-implemented features should not be marked with "N".
> > Keeping them blank is clearer to read in the resulting matrix.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: Morten Brørup <mb@smartsharesystems.com>
Applied to dpdk-next-crypto

Thanks.

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

end of thread, other threads:[~2022-02-08 21:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-05 17:40 [PATCH] doc: simplify baseband features matrix Thomas Monjalon
2022-01-05 20:12 ` Morten Brørup
2022-02-08 21:02   ` Akhil Goyal

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