linux-kernel - RE: [PATCH v1 1/9] crypto/ycc: Add YCC (Yitian Cryptography Complex) accelerator driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 24 Aug 2022 14:57:49 +0000
From:   "Elliott, Robert (Servers)" <elliott@....com>
To:     'Guanjun' <guanjun@...ux.alibaba.com>,
        "herbert@...dor.apana.org.au" <herbert@...dor.apana.org.au>
CC:     "zelin.deng@...ux.alibaba.com" <zelin.deng@...ux.alibaba.com>,
        "xuchun.shang@...ux.alibaba.com" <xuchun.shang@...ux.alibaba.com>,
        "artie.ding@...ux.alibaba.com" <artie.ding@...ux.alibaba.com>,
        "linux-crypto@...r.kernel.org" <linux-crypto@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v1 1/9] crypto/ycc: Add YCC (Yitian Cryptography Complex)
 accelerator driver



> -----Original Message-----
> From: 'Guanjun' <guanjun@...ux.alibaba.com>
> Sent: Wednesday, August 24, 2022 4:50 AM
> To: herbert@...dor.apana.org.au
> Subject: [PATCH v1 1/9] crypto/ycc: Add YCC (Yitian Cryptography Complex)
> accelerator driver
> 
...
> diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
> index 3e6aa31..d739354 100644
> --- a/drivers/crypto/Kconfig
> +++ b/drivers/crypto/Kconfig
> @@ -799,6 +799,8 @@ source "drivers/crypto/hisilicon/Kconfig"
> 
>  source "drivers/crypto/amlogic/Kconfig"
> 
> +source "drivers/crypto/ycc/Kconfig"
> +
>  config CRYPTO_DEV_SA2UL
>  	tristate "Support for TI security accelerator"
>  	depends on ARCH_K3 || COMPILE_TEST

This menu isn't perfectly sorted, but since the new entry is for
"Alibaba YCC", putting this at the top of the file would be
better positioned for eventual sorting.

Naming the directory alibaba/ rather than ycc/ might be more
welcoming for future drivers, too.

...
> diff --git a/drivers/crypto/ycc/Kconfig b/drivers/crypto/ycc/Kconfig
> +++ b/drivers/crypto/ycc/Kconfig
> @@ -0,0 +1,8 @@
> +config CRYPTO_DEV_YCC
> +	tristate "Support for Alibaba YCC cryptographic accelerator"
> +	depends on CRYPTO && CRYPTO_HW && PCI
> +	default n
> +	help
> +	  Enables the driver for the on-chip cryptographic accelerator of
> +	  Alibaba Yitian SoCs which is based on ARMv9 architecture.
> +	  If unsure say N.