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

["Elliott, Robert (Servers)" <elliott@hpe.com>]

> -----Original Message-----
> From: 'Guanjun' <guanjun@linux.alibaba.com>
> Sent: Wednesday, August 24, 2022 4:50 AM
> To: herbert@gondor.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.