Skip to contentSkip to conversation

Revert "jasper: Disable debugging when building for xtensa"

PATCH New Max Filippov 2 participants 2 messages Nov 28, 2016 – Nov 30, 2016

lore
Share & tools
lore

Max Filippov

author
#0 lore raw

PATCH 1/1 Revert "jasper: Disable debugging when building for xtensa"

This reverts commit 71d9b0c1f06896f113b09e941aa84d979bff5710. Now that -mauto-litpools is in TARGET_ABI when building for xtensa, -O0 builds succeed, so this workaround is no longer needed.

Signed-off-by Max Filippov <[email protected]>
---
 package/jasper/jasper.mk | 6 ------
 1 file changed, 6 deletions(-)
package/jasper/jasper.mk +0 -6
index bacb1a6..ebd7750 100644
@@ -12,10 +12,4 @@ JASPER_LICENSE = JasPer License Version 2.0
 JASPER_LICENSE_FILES = LICENSE
 JASPER_CONF_OPTS = --disable-strict
 
-# Xtensa gcc is unable to generate correct code with -O0 enabled by
-# --enable-debug. Allow package build but disable debug.
-ifeq ($(BR2_xtensa)$(BR2_ENABLE_DEBUG),yy)
-JASPER_CONF_OPTS += --disable-debug
-endif
-
 $(eval $(autotools-package))
Signature · 1 line
2.1.4
Applied
#1 lore raw

PATCH 1/1 Revert "jasper: Disable debugging when building for xtensa"

Hello,

Quoted 8 lines · Max Filippov

This reverts commit 71d9b0c1f06896f113b09e941aa84d979bff5710. Now that -mauto-litpools is in TARGET_ABI when building for xtensa, -O0 builds succeed, so this workaround is no longer needed.

Signed-off-by Max Filippov <[email protected]>
---
 package/jasper/jasper.mk | 6 ------
 1 file changed, 6 deletions(-)

Applied to next, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com