Thanks Akhil, A couple of things I miss in term of guidelines for my benefit and that I don't see in documentation: - May I ask what rule we should use for documentation line breaking? I am unclear of the reason for some of the changes you made whose origin version looked legit to me. Were you trying to fit into 80 chars (not always the case) or was this to split the sentence by phrase as opposed to breaking simply just before 100 chars? Can you clarify the recommendation? I see in the doc it suggests 80 chars but unsure whether this is because this was not updated when moving to 100 chars. - Do we really need a dot at the end of every text in a comment even when not an actual sentence? I don't personally see the point nor do I believe this is followed in much of DPDK existing code (it depends at best), nor was this case in the file where new structure was added (keeping consistency). Is that really a recommendation for DPDK guideline for all new code to add a dot at the end of all text?
Thanks for kindly clarifying, I can then update the other serie based on such guidelines. Thanks for the other formatting issues and commit message changes. Nic > -----Original Message----- > From: Akhil Goyal <gak...@marvell.com> > Sent: Thursday, October 6, 2022 10:32 AM > To: Chautru, Nicolas <nicolas.chau...@intel.com>; dev@dpdk.org; > tho...@monjalon.net > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson, Bruce <bruce.richard...@intel.com>; > david.march...@redhat.com; step...@networkplumber.org; Zhang, > Mingshan <mingshan.zh...@intel.com>; hemant.agra...@nxp.com > Subject: RE: [EXT] [PATCH v12 0/7] bbdev changes for 22.11 > > > Hi Akhil, Thomas, > > > > v12: minor change to fix misaligned comment on patch 6 raised by Thomas. > > Thanks. > > v11: updated based on Thomas review notably on comments through the > > serie and ordering. Thanks. I have also updated rel_notes and > > deprecation through the serie this time. > > v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on > > suggestion from Ferruh/Maxime/Akhil. Thanks > > v9: removing code snippet from documentation in 5/7 requested by Akhil. > > Thanks. > > v8: edit based on review by Akhil : typos, coding guidelines. No > > functional change. Thanks > > v7: couple of typos in documentation spotted by Maxime. Thanks. > > v6: added one comment in commit 2/7 suggested by Maxime. > > v5: update base on review from Tom Rix. Number of typos reported and > > resolved, removed the commit related to rw_lock for now, added a > > commit for code clean up from review, resolved one rebase issue > > between 2 commits, used size of array for some bound check > > implementation. Thanks. > > v4: update to the last 2 commits to include function to print the > > queue status and a fix to the rte_lock within the wrong structure > > v3: update to device status info to also use padded size for the related > array. > > Adding also 2 additionals commits to allow the API struc to expose > > more information related to queues corner cases/warning as well as an > > optional rw lock. > > Hemant, Maxime, this is planned for DPDK 21.11 but would like > > review/ack early is possible to get this applied earlier and due to time off > this summer. > > Thanks > > Nic > > > > Nicolas Chautru (7): > > bbdev: allow operation type enum for growth > > bbdev: add device status info > > bbdev: add device info on queue topology > > drivers/baseband: update PMDs to expose queue per operation > > bbdev: add new operation for FFT processing > > bbdev: add queue related warning and status information > > bbdev: remove unnecessary if-check > > > > app/test-bbdev/test_bbdev.c | 2 +- > > app/test-bbdev/test_bbdev_perf.c | 6 +- > > doc/guides/prog_guide/bbdev.rst | 103 +++++++++++ > > doc/guides/rel_notes/deprecation.rst | 13 -- > > doc/guides/rel_notes/release_22_11.rst | 14 ++ > > drivers/baseband/acc100/rte_acc100_pmd.c | 30 ++-- > > .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 9 + > > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 9 + > > drivers/baseband/la12xx/bbdev_la12xx.c | 10 +- > > drivers/baseband/null/bbdev_null.c | 1 + > > .../baseband/turbo_sw/bbdev_turbo_software.c | 13 ++ > > examples/bbdev_app/main.c | 2 +- > > lib/bbdev/rte_bbdev.c | 57 +++++- > > lib/bbdev/rte_bbdev.h | 158 +++++++++++++++- > > lib/bbdev/rte_bbdev_op.h | 169 ++++++++++++++++-- > > lib/bbdev/version.map | 12 ++ > > 16 files changed, 560 insertions(+), 48 deletions(-) > > > Hi Nicolas, > > There were many formatting issues in this patch. This has been a practice in > whole of bbdev. > Please take a diff of what is applied on the tree and what was submitted to > take care of the formatting in future patches and please plan to fix the rest > of bbdev documentation (API + prog_guide). > > Series applied to dpdk-next-crypto > > Thanks.