Hi Akhil, > -----Original Message----- > From: Akhil Goyal <[email protected]> > Sent: Tuesday, September 10, 2019 5:10 PM > To: Anoob Joseph <[email protected]>; Thomas Monjalon > <[email protected]>; Jerin Jacob Kollanukkaran <[email protected]> > Cc: 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 > > Hi Anoob, > > > > > 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? > > As per the convention followed these days, documentation should be the > part of the Patch which support it. > If you have supported many features in your patchset, you should add that > piece of documentation which is supported in each of the patches in the > patchset. > > Release notes should be added in the patch which completes the whole > feature. > > -Akhil
Will do so when sending v2. Thanks, Anoob
