On 5/6/2020 5:09 PM, Ajit Khaparde wrote: > > > On Wed, May 6, 2020 at 1:57 AM Ferruh Yigit <ferruh.yi...@intel.com > <mailto:ferruh.yi...@intel.com>> wrote: > > On 5/1/2020 11:03 PM, Ajit Khaparde wrote: > > - Update list of supported adapters. > > - Update list of supported features. > > - Add some details to describe the features. > > - Remove obsolete limitations. > > - Fix and update links. > > > > Signed-off-by: JP Lee <jongpil....@broadcom.com > <mailto:jongpil....@broadcom.com>> > > Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com > <mailto:ajit.khapa...@broadcom.com>> > > --- > > doc/guides/nics/bnxt.rst | 1131 +++++++++++++++++++++++++++++++++----- > > 1 file changed, 1001 insertions(+), 130 deletions(-) > > > > diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst > > index 434ba9d6c..0da312557 100644 > > --- a/doc/guides/nics/bnxt.rst > > +++ b/doc/guides/nics/bnxt.rst > > @@ -1,138 +1,1009 @@ > > -.. SPDX-License-Identifier: BSD-3-Clause > > - Copyright 2016-2019 Broadcom > > +.. SPDX-License-Identifier: BSD-3-Clause > > + Copyright 2020 Broadcom Inc. > > > > BNXT Poll Mode Driver > > ===================== > > > > -The BNXT PMD (**librte_pmd_bnxt**) implements support for adapters > based on > > -Ethernet controllers and SoCs belonging to the **Broadcom BCM5730X > NetXtreme-C® > > -Family of Ethernet Network Controllers**, the **Broadcom > BCM574XX/BCM575XX > > -NetXtreme-E® Family of Ethernet Network Controllers**, the **Broadcom > BCM588XX > > -Stingray Family of SmartNIC Adapters**, and the **Broadcom StrataGX® > BCM5871X > > -Series of Communications Processors**. A complete list with links to > reference > > -material is included below. > > +The Broadcom BNXT PMD (Poll Mode Driver, librte_pmd_bnxt) implements > support for adapters based on Ethernet controllers and SoCs belonging to > the > Broadcom BCM574XX/BCM575XX NetXtreme-E® Family of Ethernet Network > Controllers, the Broadcom BCM588XX Stingray Family of Smart NIC Adapters, > and the Broadcom StrataGX® BCM5873X Series of Communications Processors. > > Hi Ajit, > > This adds single line with 358 char length, most probably the web output > is good > but it is hard to read/maintain as .rst doc. > > What is the motivation for joining the multiple lines? > > No specific reason. Mostly an oversight. We can fix it if necessary. > > > > For the content we need your ack, but for the style/formatting/language > etc.. > would you mind waiting for an ack from doc maintainers, cc'ed, before > merge? > > Sure. Not an issue. >
There was no doc review, unfortunately. At least can you send a new version by keeping the line length around ~80. And I suggest if there is not other change in the text, keep the original line length.
