From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH ARM v6 05/14] mini-os: import libfdt
Date: Wed, 16 Jul 2014 14:02:12 +0100
Message-ID: <53C677D4.2060109@citrix.com>
References: <1405508874-3921-1-git-send-email-talex5@gmail.com>	
	<1405508874-3921-6-git-send-email-talex5@gmail.com>	
	<53C665BA.2010305@citrix.com>
	<1405513794.14331.2.camel@kazak.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta3.messagelabs.com ([195.245.230.39])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <Andrew.Cooper3@citrix.com>) id 1X7Oqy-0004kJ-MB
	for xen-devel@lists.xenproject.org; Wed, 16 Jul 2014 13:02:36 +0000
In-Reply-To: <1405513794.14331.2.camel@kazak.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Thomas Leonard <talex5@gmail.com>, Dave.Scott@eu.citrix.com, anil@recoil.org, stefano.stabellini@eu.citrix.com, samuel.thibault@ens-lyon.org, xen-devel@lists.xenproject.org
List-Id: xen-devel@lists.xenproject.org

On 16/07/14 13:29, Ian Campbell wrote:
> On Wed, 2014-07-16 at 12:44 +0100, Andrew Cooper wrote:
>> On 16/07/14 12:07, Thomas Leonard wrote:
>>> From: Karim Raslan <karim.allah.ahmed@gmail.com>
>>>
>>> Looks like this is revision v1.3.0-47-gbe60268 from
>>> http://git.jdl.com/gitweb/?p=dtc.git
>>>
>>> The memmove implementation is from FreeBSD's
>>> contrib/ldns/compat/memmove.c (r246827).
>>>
>>> Signed-off-by: Karim Allah Ahmed <karim.allah.ahmed@gmail.com>
>>> [talex5@gmail.com: split out FDT support into a separate patch]
>>> [talex5@gmail.com: fixed "make clean" for FDT]
>>> [talex5@gmail.com: replaced GPL memmove with BSD one]
>>> Signed-off-by: Thomas Leonard <talex5@gmail.com>
>> Xen already has a copy of libfdt in xen/common/libfdt.
> Please review the thread on this from the previous postings.
>
> Ian.
>

Which appears to be unconcluded.

libfdt is a 3rd party library which, from what I can tell in the
history, is unmodified.  Therefore, my suggestion of moving it outside
of the xen/ tree and having both minios and Xen VPATH themselves access
to it helps with the end goal of preventing minios becoming dependent on
Xen code.  It further prevents both Xen and minios from accidentally
gaining localisation hacks in the library itself, which makes it easier
to updates to the base libdft if/when necessary.

~Andrew