]> git.baikalelectronics.ru Git - kernel.git/commit
crypto: ux500 - do not build with -O0
authorArnd Bergmann <arnd@arndb.de>
Mon, 27 Jun 2016 09:17:40 +0000 (11:17 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Tue, 28 Jun 2016 08:06:33 +0000 (16:06 +0800)
commitf2a719560f7e2ae838ac350ee5c1958ae3c8ae17
tree4a6ce09019681a00689026452e2a03c6082f963b
parent77d28c84658236d74520aa3c9d86df94c5dcd16d
crypto: ux500 - do not build with -O0

The ARM allmodconfig build currently warngs because of the
ux500 crypto driver not working well with the jump label
implementation that we started using for dynamic debug, which
breaks building with 'gcc -O0':

In file included from /git/arm-soc/include/linux/jump_label.h:105:0,
                 from /git/arm-soc/include/linux/dynamic_debug.h:5,
                 from /git/arm-soc/include/linux/printk.h:289,
                 from /git/arm-soc/include/linux/kernel.h:13,
                 from /git/arm-soc/include/linux/clk.h:16,
                 from /git/arm-soc/drivers/crypto/ux500/hash/hash_core.c:16:
/git/arm-soc/arch/arm/include/asm/jump_label.h: In function 'hash_set_dma_transfer':
/git/arm-soc/arch/arm/include/asm/jump_label.h:13:7: error: asm operand 0 probably doesn't match constraints [-Werror]
  asm_volatile_goto("1:\n\t"

Turning off compiler optimizations has never really been supported
here, and it's only used when debugging the driver. I have not found
a good reason for doing this here, other than a misguided attempt
to produce more readable assembly output. Also, the driver is only
used in obsolete hardware that almost certainly nobody will spend
time debugging any more.

This just removes the -O0 flag from the compiler options.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/ux500/cryp/Makefile
drivers/crypto/ux500/hash/Makefile