Re: [PATCH v4 0/5] Introduce AMD Secure Processor device

2017-07-18 Thread Herbert Xu
On Thu, Jul 06, 2017 at 09:59:12AM -0500, Brijesh Singh wrote:
> CCP device (drivers/crypto/ccp/ccp.ko) is part of AMD Secure Processor,
> which is not dedicated solely to crypto. The AMD Secure Processor includes
> CCP and PSP (Platform Secure Processor) devices.
> 
> This patch series adds a framework that allows functional component of the
> AMD Secure Processor to be initialized and handled appropriately. The series
> does not makes any logic modification into CCP - it refactors the code to
> integerate CCP into AMD secure processor framework.
> 
> ---
> 
> Changes since v3:
>  - guard sp_dev_resume and sp_dev_suspend with CONFIG_PM
>  - update Kconfig description for AMD SP device
> 
> Changes since v2:
>  - move the ccp->io_regs initialization before device setup().
>  - maintain the original Kconfig hierarchy
>  - rename ccp-{pci,platform}.c -> sp-{pci,platform}.c
>  - do not fail the module_init() when ccp device is not found
> 
> Changes since v1:
>  - remove unused function [sp_get_device()]
> 
> Brijesh Singh (5):
>   crypto: ccp - Use devres interface to allocate PCI/iomap and cleanup
>   crypto: ccp - Introduce the AMD Secure Processor device
>   crypto: cpp - Abstract interrupt registeration
>   crypto: ccp - rename ccp driver initialize files as sp device
>   crypto: ccp - remove ccp_present() check from device initialize

All patches applied.  Thanks.
-- 
Email: Herbert Xu 
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt


[PATCH v4 0/5] Introduce AMD Secure Processor device

2017-07-06 Thread Brijesh Singh
CCP device (drivers/crypto/ccp/ccp.ko) is part of AMD Secure Processor,
which is not dedicated solely to crypto. The AMD Secure Processor includes
CCP and PSP (Platform Secure Processor) devices.

This patch series adds a framework that allows functional component of the
AMD Secure Processor to be initialized and handled appropriately. The series
does not makes any logic modification into CCP - it refactors the code to
integerate CCP into AMD secure processor framework.

---

Changes since v3:
 - guard sp_dev_resume and sp_dev_suspend with CONFIG_PM
 - update Kconfig description for AMD SP device

Changes since v2:
 - move the ccp->io_regs initialization before device setup().
 - maintain the original Kconfig hierarchy
 - rename ccp-{pci,platform}.c -> sp-{pci,platform}.c
 - do not fail the module_init() when ccp device is not found

Changes since v1:
 - remove unused function [sp_get_device()]

Brijesh Singh (5):
  crypto: ccp - Use devres interface to allocate PCI/iomap and cleanup
  crypto: ccp - Introduce the AMD Secure Processor device
  crypto: cpp - Abstract interrupt registeration
  crypto: ccp - rename ccp driver initialize files as sp device
  crypto: ccp - remove ccp_present() check from device initialize

 drivers/crypto/Kconfig|   6 +-
 drivers/crypto/ccp/Kconfig|  21 ++-
 drivers/crypto/ccp/Makefile   |   6 +-
 drivers/crypto/ccp/ccp-dev-v3.c   |  17 +-
 drivers/crypto/ccp/ccp-dev-v5.c   |  12 +-
 drivers/crypto/ccp/ccp-dev.c  | 124 -
 drivers/crypto/ccp/ccp-dev.h  |  25 +--
 drivers/crypto/ccp/ccp-pci.c  | 356 --
 drivers/crypto/ccp/ccp-platform.c | 293 ---
 drivers/crypto/ccp/sp-dev.c   | 277 +
 drivers/crypto/ccp/sp-dev.h   | 132 ++
 drivers/crypto/ccp/sp-pci.c   | 276 +
 drivers/crypto/ccp/sp-platform.c  | 252 +++
 include/linux/ccp.h   |   7 +-
 14 files changed, 1062 insertions(+), 742 deletions(-)
 delete mode 100644 drivers/crypto/ccp/ccp-pci.c
 delete mode 100644 drivers/crypto/ccp/ccp-platform.c
 create mode 100644 drivers/crypto/ccp/sp-dev.c
 create mode 100644 drivers/crypto/ccp/sp-dev.h
 create mode 100644 drivers/crypto/ccp/sp-pci.c
 create mode 100644 drivers/crypto/ccp/sp-platform.c

-- 
2.9.4