Hi Thomas, Please see inline.
Thanks, Anoob > -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Tuesday, September 10, 2019 12:00 PM > To: Jerin Jacob Kollanukkaran <[email protected]>; Anoob Joseph > <[email protected]> > Cc: Akhil Goyal <[email protected]>; Pablo de Lara > <[email protected]>; Narayana Prasad Raju Athreya > <[email protected]>; Fiona Trahe <[email protected]>; Kanaka > Durga Kotamarthy <[email protected]>; Shally Verma > <[email protected]>; Sunila Sahu <[email protected]>; [email protected] > Subject: Re: [PATCH 6/6] doc: add documentation for OCTEON TX2 crypto > asym support > > 10/09/2019 06:13, Jerin Jacob Kollanukkaran: > > From: Anoob Joseph <[email protected]> > > > Adding documentation for asymmetric features supported by > > > crypto_octeontx2 PMD. > > > > > > Signed-off-by: Anoob Joseph <[email protected]> > > > Signed-off-by: Kanaka Durga Kotamarthy <[email protected]> > > > Signed-off-by: Sunila Sahu <[email protected]> > > > --- > > > doc/guides/cryptodevs/features/octeontx2.ini | 9 +++++++++ > > > doc/guides/cryptodevs/octeontx2.rst | 13 +++++++++++++ > > > > Please update the release notes. > > Please avoid doing a separate patch for documentation. > It could be updated while adding the feature, in the same patch. [Anoob] To which patch should I squash this change? Here are my options, 1. The patch which adds ASYM in capabilities. Issue: the doc update updates about autotest, which won't be available until the final patch is merged. 2. Test patch which adds the autotest for the PMD. Issue: Additions to the doc would be done by a patch which adds test. Which one should I use?
