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 <[email protected]>
Sent: Wednesday, February 12, 2020 12:41 PM
To: Somalapuram, Amaranath <[email protected]>; [email protected];
Kumar, Ravi1 <[email protected]>
Cc: [email protected]
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: [email protected]
>
> Signed-off-by: Amaranath Somalapuram <[email protected]>
> ---
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.
