On 06/28/2017 02:53 PM, Tom Lendacky wrote:

In this I am leaving the top level config as-is and adding
CONFIG_CRYPTO_DEV_SP_CCP to enable the CCP device support inside the SP device 
driver.

[*] Support for AMD Secure Processor
<M>  Secure Processor device driver
<M>    Encryption and hashing offload support
-*-    Cryptographic Coprocessor device

I think the "Encryption and hashing offload support" should be indented
under "Cryptographic Coprocessor device" since it is a function of the
CCP and not the SP. Not sure if we can remove a level of menu somehow,
something to explore.

Good point, the updated Kconfig looks like this and let me know it works.

[*] Support for AMD Secure Processor
<M>   Secure Processor device driver
[*]     Cryptographic Coprocessor device
<M>       Encryption and hashing offload support

# cat drivers/crypto/ccp/Kconfig

config CRYPTO_DEV_CCP_DD
         tristate "Secure Processor device driver"
         default m
         help
           Provides AMD Secure Processor device driver.
           If you choose 'M' here, this module will be called ccp.
config CRYPTO_DEV_SP_CCP
         bool "Cryptographic Coprocessor device"
         default y
         depends on CRYPTO_DEV_CCP_DD
         select HW_RANDOM
         select DMA_ENGINE
         select DMADEVICES
         select CRYPTO_SHA1
         select CRYPTO_SHA256
         help
           Provides the support for AMD Cryptographic Coprocessor (CCP) device
           which can be used to offload encryption operations such as SHA, AES
           and more.
config CRYPTO_DEV_CCP_CRYPTO
         tristate "Encryption and hashing offload support"
         default m
         depends on CRYPTO_DEV_CCP_DD
         depends on CRYPTO_DEV_SP_CCP
         select CRYPTO_HASH
         select CRYPTO_BLKCIPHER
         select CRYPTO_AUTHENC
         help
           Support for using the cryptographic API with the AMD Cryptographic
           Coprocessor. This module supports offload of SHA and AES algorithms.
           If you choose 'M' here, this module will be called ccp_crypto.

Reply via email to