26/02/2022 10:37, Jerin Jacob:
> On Sat, Feb 26, 2022 at 2:52 PM Thomas Monjalon <tho...@monjalon.net> wrote:
> >
> > 22/02/2022 20:35, Nithin Dabilpuram:
> > > Add table for environment variables used by cnxk drivers.
> > >
> > > Signed-off-by: Nithin Dabilpuram <ndabilpu...@marvell.com>
> > > ---
> > > +.. table:: cnxk environment variables
> > > +
> > > +   
> > > +---+-----------------------------+-----------------------------------------+
> > > +   | # | Variable name               | Usage                             
> > >       |
> > > +   
> > > +===+=============================+=========================================+
> > > +   | 1 | BPHY_INTR_MLOCK_DISABLE     | When defined disables memory 
> > > locking in |
> > > +   |   |                             | BPHY environment.                 
> > >       |
> > > +   
> > > +---+-----------------------------+-----------------------------------------+
> > > +   | 2 | ROC_CN10K_MBOX_TIMEOUT      | When set, overrides MBOX timeout 
> > > by     |
> > > +   |   | ROC_MBOX_TIMEOUT            | value in milli seconds.           
> > >       |
> > > +   
> > > +---+-----------------------------+-----------------------------------------+
> > > +   | 3 | CN10K_ETH_SEC_IV_OVR        | When set, overrides outbound 
> > > inline SA  |
> > > +   |   |                             | IV in CN10K. By default IV is 
> > > generated |
> > > +   |   |                             | by HW. Format of variable is 
> > > string     |
> > > +   |   |                             | of comma separated one byte 
> > > values as   |
> > > +   |   |                             | for ex: "0x0, 0x10, 0x20, ..."    
> > >       |
> > > +   
> > > +---+-----------------------------+-----------------------------------------+
> >
> > Using tables for such list is a bad idea.
> > The source code is constrained in a small column,
> > and the HTML rendering is constrained by the page width.
> > I recommend switching to the definition list syntax
> > which has a nice rendering for such definitions.
> >
> > Example:
> > ``BPHY_INTR_MLOCK_DISABLE``
> >    Disable memory locking in BPHY environment.
> 
> Sure, You can remove this doc update while merging to the main tree.
> We will submit a separate patch for updating this document
> based on your suggestion.

It has been squashed in a commit adding the third variable,
so yes it can be dropped and submitted again for the 3 variables.


Reply via email to