Hi Jan, > -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Tuesday, October 10, 2017 1:50 PM > To: Jan Blunck <jblu...@infradead.org> > Cc: dev@dpdk.org; Doherty, Declan <declan.dohe...@intel.com> > Subject: RE: [PATCH v3 3/4] cryptodev: rework dependency on vdev header > > > > > -----Original Message----- > > From: jblu...@gmail.com [mailto:jblu...@gmail.com] On Behalf Of Jan > > Blunck > > Sent: Tuesday, October 10, 2017 12:30 PM > > To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com> > > Cc: dev@dpdk.org; Doherty, Declan <declan.dohe...@intel.com> > > Subject: Re: [PATCH v3 3/4] cryptodev: rework dependency on vdev > > header > > > > On Tue, Oct 10, 2017 at 10:47 AM, De Lara Guarch, Pablo > > <pablo.de.lara.gua...@intel.com> wrote: > > > > > > > > >> -----Original Message----- > > >> From: Jan Blunck [mailto:jblu...@gmail.com] On Behalf Of Jan Blunck > > >> Sent: Friday, October 6, 2017 9:40 AM > > >> To: dev@dpdk.org > > >> Cc: Doherty, Declan <declan.dohe...@intel.com>; De Lara Guarch, > > >> Pablo <pablo.de.lara.gua...@intel.com> > > >> Subject: [PATCH v3 3/4] cryptodev: rework dependency on vdev > header > > >> > > >> The rte_cryptodev_vdev_pmd_init() is a helper for vdev-based drivers. > > >> By moving the helper to the header we don't require rte_vdev.h at > > >> build- time of the librte_cryptodev library. This is a preparation > > >> to move the vdev bus into a standalone library. > > >> > > >> Signed-off-by: Jan Blunck <jblu...@infradead.org> > > > > > > This looks good, but could you also remove the deprecation notice > > > from > > the previous release? > > > Apart from that: > > > > > > > I wondered about that too. The message said that it will be static > > from > > 17.11 on and I wonder if we should actually keep it into the notes for > > this release. > > Well, actually this change should be added in release notes (under API > changes, I think), and removed from deprecation.rst.
Are you going to send a new patchset soon, so it can make RC1? Thanks! Pablo > > Thanks, > Pablo
