> -----Original Message----- > From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com] > Sent: Tuesday, May 8, 2018 6:19 PM > To: Gujjar, Abhinandan S <abhinandan.guj...@intel.com> > Cc: Akhil Goyal <akhil.go...@nxp.com>; hemant.agra...@nxp.com; > dev@dpdk.org; Vangati, Narender <narender.vang...@intel.com>; Rao, Nikhil > <nikhil....@intel.com>; Eads, Gage <gage.e...@intel.com> > Subject: Re: [dpdk-dev] [v3,1/5] eventdev: introduce event crypto adapter > > -----Original Message----- > > Date: Tue, 8 May 2018 07:34:52 +0000 > > From: "Gujjar, Abhinandan S" <abhinandan.guj...@intel.com> > > To: Akhil Goyal <akhil.go...@nxp.com>, Jerin Jacob > > <jerin.ja...@caviumnetworks.com> > > CC: "hemant.agra...@nxp.com" <hemant.agra...@nxp.com>, > "dev@dpdk.org" > > <dev@dpdk.org>, "Vangati, Narender" <narender.vang...@intel.com>, > > "Rao, Nikhil" <nikhil....@intel.com>, "Eads, Gage" > > <gage.e...@intel.com> > > Subject: RE: [dpdk-dev] [v3,1/5] eventdev: introduce event crypto > > adapter > > > > Hi Akhil, > > > > Thanks for the feedback on the diagram. My thoughts are also in line with > diagram. > > In fact, my diagram also depicts same. The only difference is that, you have > shown application in a separate block. > > The "crypto stage + enqueue to cryptodev" block itself is part of an > application. > > So, for more clarity, I will add "application" as additional text to the > > block. > > With the limited time, changing diagram needs change in SVG file as well. > > In case, if you still feel there a change required we can take it up later. > > IMO, At least in the header file you can copy the proposed and agreed diagram. > if you don't have time then documentation patch we can fix it just after the > initial patch and worst case apply the documentation patch after RC3. Sure Jerin.
Regards Abhinandan
