Hi Ulrich,

comments inline

On 17/02/2015 16:54, Ulrich Weber wrote:
> Signed-off-by: Ulrich Weber <[email protected]>
> ---
>  package/kernel/linux/modules/crypto.mk | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/package/kernel/linux/modules/crypto.mk 
> b/package/kernel/linux/modules/crypto.mk
> index 7f8b9f2..1ae962f 100644
> --- a/package/kernel/linux/modules/crypto.mk
> +++ b/package/kernel/linux/modules/crypto.mk
> @@ -267,15 +267,18 @@ $(eval $(call KernelPackage,crypto-hw-omap))
>  
>  define KernelPackage/crypto-aes
>    TITLE:=AES cipher CryptoAPI module
> -  KCONFIG:=CONFIG_CRYPTO_AES CONFIG_CRYPTO_AES_586
> +  KCONFIG:=CONFIG_CRYPTO_AES CONFIG_CRYPTO_AES_586 CONFIG_CRYPTO_AES_NI_INTEL

instead of doing this

>    FILES:=$(LINUX_DIR)/crypto/aes_generic.ko
>    AUTOLOAD:=$(call AutoLoad,09,aes_generic)
> -  $(call AddDepends/crypto)
> +  $(call AddDepends/crypto,+TARGET_x86:kmod-crypto-cryptd 
> +TARGET_x86:kmod-crypto-gf128 +TARGET_x86:kmod-crypto-xts)

and this we could do

>  endef
>  
>  define KernelPackage/crypto-aes/x86
> -  FILES+=$(LINUX_DIR)/arch/x86/crypto/aes-i586.ko
> -  AUTOLOAD:=$(call AutoLoad,09,aes-i586)
> +  FILES+=$(LINUX_DIR)/arch/x86/crypto/aes-i586.ko \
> +     $(LINUX_DIR)/arch/x86/crypto/aesni-intel.ko \
> +     $(LINUX_DIR)/crypto/ablk_helper.ko \
> +     $(LINUX_DIR)/crypto/lrw.ko
> +  AUTOLOAD:=$(call AutoLoad,09,ablk_helper lrw aes_generic aes-i586 
> aesni-intel)

DEPENDS+=+kmod-crypto-cryptd +kmod-crypto-gf128 +kmod-crypto-xts
KCONFIG+=CONFIG_CRYPTO_AES_586 CONFIG_CRYPTO_AES_NI_INTEL

same goes fro 3/5  or am i missing something ?

        John



>  endef
>  
>  $(eval $(call KernelPackage,crypto-aes))
> 
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to