Hi Akhil,

On 17-09-2018 16:07, Akhil Goyal wrote:
External Email


I think it would be better to squash the makefile related changes in
the 3/33 patch as the code

is actually added in that and here the code is not getting compiled
here.
So the changes in the following files has to be moved to patch 3/33?
  drivers/crypto/Makefile
  drivers/crypto/meson.build
  drivers/crypto/octeontx/Makefile
  drivers/crypto/octeontx/meson.build
  mk/rte.app.mk

I think this patch will just have MAINTAINER edit (even that might be
required to be moved to 3/33?) & changes to config/common_base, after
that. Is that fine?
In my opinion, you do not need this patch as separate one.
config/common_base can also be added in the 3/33.
In that case 02/33 patch would become the first patch right? The same problem would be there too, I guess. The macros added in that patch gets used only in 03/33 patch. Is that fine?

The first patch would be a shell patch for most PMD additions. That's the reason we started this way. If you want it changed, will do so. Please do let me know what will be the right approach.

Anoob

Reply via email to