Is it a fix or a new feature?
It’s a new feature
If it is a fix then only it should be sent to stable and the patch title should
include "crypto/ccp: fix ...".
Ok got it.. next time will keep in mind.
If it is a feature(which is more likely the case IMO), the title is fine or I
would change it to "crypto/ccp: enable IOMMU"
This title well be fine ("crypto/ccp: enable IOMMU") do let me know should I
resubmit ??
Regards,
S.Amarnath
-----Original Message-----
From: Akhil Goyal <akhil.go...@nxp.com>
Sent: Wednesday, February 12, 2020 12:41 PM
To: Somalapuram, Amaranath <amaranath.somalapu...@amd.com>; dev@dpdk.org;
Kumar, Ravi1 <ravi1.ku...@amd.com>
Cc: sta...@dpdk.org
Subject: RE: [dpdk-dev] [PATCH v2] crypto/ccp: enable IOMMU for CCP
[CAUTION: External Email]
>
> CCP use vdev framework, and vdev framework don’t support IOMMU.
> Adding custom IOMMU support for AMD CCP drives.
> Cc: sta...@dpdk.org
>
> Signed-off-by: Amaranath Somalapuram <asoma...@amd.com>
> ---
Is it a fix or a new feature?
If it is a fix then only it should be sent to stable and the patch title should
include "crypto/ccp: fix ...".
If it is a feature(which is more likely the case IMO), the title is fine or I
would change it to "crypto/ccp: enable IOMMU"
@Ravi: could you please review this patch.
