From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Thu, 19 Jun 2014 06:35:43 +0200
Subject: [U-Boot] [PATCH v6 1/2] lib,
 fdt: move fdtdec_get_int() out of lib/fdtdec.c
In-Reply-To: <20140618210936.GI26243@bill-the-cat>
References: <1402905520-15847-1-git-send-email-hs@denx.de>
	<1402905520-15847-2-git-send-email-hs@denx.de>
	<20140618210936.GI26243@bill-the-cat>
Message-ID: <53A2689F.6070408@denx.de>
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

Hello Tom,

Am 18.06.2014 23:09, schrieb Tom Rini:
> On Mon, Jun 16, 2014 at 09:58:39AM +0200, Heiko Schocher wrote:
>
>> move fdtdec_get_int() out of lib/fdtdec.c into lib/fdtdec_common.c
>> as this function is also used, if CONFIG_OF_CONTROL is not
>> used. Poped up on the ids8313 board using signed FIT images,
>> and activating CONFIG_SYS_GENERIC_BOARD. Without this patch
>> it shows on boot:
> [snip]
>> diff --git a/lib/Makefile b/lib/Makefile
>> index 377ab13..dd87ee2 100644
>> --- a/lib/Makefile
>> +++ b/lib/Makefile
>> @@ -23,6 +23,13 @@ obj-$(CONFIG_USB_TTY) += circbuf.o
>>   obj-y += crc7.o
>>   obj-y += crc8.o
>>   obj-y += crc16.o
>> +ifdef CONFIG_FIT
>> +obj-y += fdtdec_common.o
>> +else
>> +ifdef CONFIG_OF_CONTROL
>> +obj-y += fdtdec_common.o
>> +endif
>> +endif
>
> OK, but why not:
> obj-$(CONFIG_FIT) += fdtdec_common.o
> obj-$(CONFIG_OF_CONTROL) += fdtdec_common.o
>
> Since we filter out duplicates I would swear.

If so, I change this.

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany