All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] crypto: vmx - VMX crypto should depend on CONFIG_VSX
@ 2015-09-09  6:15 Michael Ellerman
  2015-09-09  6:16 ` Herbert Xu
  0 siblings, 1 reply; 2+ messages in thread
From: Michael Ellerman @ 2015-09-09  6:15 UTC (permalink / raw)
  To: linuxppc-dev; +Cc: leosilva, herbert, Paul Mackerras, tim.gardner

This code uses FP (floating point), Altivec and VSX. It can just depend on
CONFIG_VSX though, because that already depends on FP and Altivec.

Otherwise we get lots of link errors such as:

  drivers/built-in.o: In function `.p8_aes_setkey':
  aes.c:(.text+0x2d325c): undefined reference to `.enable_kernel_altivec'
  aes.c:(.text+0x2d326c): undefined reference to `.enable_kernel_vsx'

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
---
 drivers/crypto/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/Kconfig b/drivers/crypto/Kconfig
index 07bc7aa6b224..d234719065a5 100644
--- a/drivers/crypto/Kconfig
+++ b/drivers/crypto/Kconfig
@@ -461,7 +461,7 @@ config CRYPTO_DEV_QCE
 
 config CRYPTO_DEV_VMX
 	bool "Support for VMX cryptographic acceleration instructions"
-	depends on PPC64
+	depends on PPC64 && VSX
 	help
 	  Support for VMX cryptographic acceleration instructions.
 
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] crypto: vmx - VMX crypto should depend on CONFIG_VSX
  2015-09-09  6:15 [PATCH] crypto: vmx - VMX crypto should depend on CONFIG_VSX Michael Ellerman
@ 2015-09-09  6:16 ` Herbert Xu
  0 siblings, 0 replies; 2+ messages in thread
From: Herbert Xu @ 2015-09-09  6:16 UTC (permalink / raw)
  To: Michael Ellerman; +Cc: linuxppc-dev, leosilva, Paul Mackerras, tim.gardner

On Wed, Sep 09, 2015 at 04:15:23PM +1000, Michael Ellerman wrote:
> This code uses FP (floating point), Altivec and VSX. It can just depend on
> CONFIG_VSX though, because that already depends on FP and Altivec.
> 
> Otherwise we get lots of link errors such as:
> 
>   drivers/built-in.o: In function `.p8_aes_setkey':
>   aes.c:(.text+0x2d325c): undefined reference to `.enable_kernel_altivec'
>   aes.c:(.text+0x2d326c): undefined reference to `.enable_kernel_vsx'
> 
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>

Could you please cc linux-crypto@vger.kernel.org?

Thanks,
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-09  6:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-09  6:15 [PATCH] crypto: vmx - VMX crypto should depend on CONFIG_VSX Michael Ellerman
2015-09-09  6:16 ` Herbert Xu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.