From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Mon, 9 Jun 2014 09:14:11 +0200
Subject: [U-Boot] [PATCH] arm: fix a build error with CONFIG_USE_IRQ
In-Reply-To: <1401101030-6907-1-git-send-email-yamada.m@jp.panasonic.com>
References: <1401101030-6907-1-git-send-email-yamada.m@jp.panasonic.com>
Message-ID: <E1WttmW-0005Cb-HT@janus>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Masahiro,

On Mon, 26 May 2014 19:43:50 +0900, Masahiro Yamada
<yamada.m@jp.panasonic.com> wrote:

> Commit 41623c91 moved exception handling to arch/arm/lib/vectors.S,
> breaking CONFIG_USE_IRQ feature.
> 
> If CONFIG_USE_IRQ is enabled, undefined reference error occurs.
> 
>   arch/arm/lib/built-in.o: In function `interrupt_init':
>   arch/arm/lib/interrupts.c:37: undefined reference to `IRQ_STACK_START'
>   arch/arm/lib/interrupts.c:37: undefined reference to `FIQ_STACK_START'
>   make: *** [u-boot] Error 1
> 
> Because arch/arm/lib/vectors.S includes references to
> CONFIG_SYS_DV_NOR_BOOT_CFG and CONFIG_USE_IRQ, it must include <config.h>.

I routinely build all of ARM (32-bit) and AARCH64 (64-bit) targets and
make sure all of them build properly, and concerning CONFIG_USE_IRQ, I
even have a branch set to remove this option since a

	git grep CONFIG_USE_IRQ | grep -vE \
	'(#define|#ifdef|#ifndef|#if.*defined|#undef|#endif)'

shows that it is defined nowhere in our code.

Which source code are you using?

Amicalement,
-- 
Albert.