From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v4 28/33] tools/libxl: Check if fdt_{first, next}_subnode are present in libfdt Date: Tue, 31 Mar 2015 12:35:55 +0100 Message-ID: <1427801755.2115.90.camel@citrix.com> References: <1426793399-6283-1-git-send-email-julien.grall@linaro.org> <1426793399-6283-29-git-send-email-julien.grall@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YcuSd-0003Ep-Vx for xen-devel@lists.xenproject.org; Tue, 31 Mar 2015 11:36:00 +0000 In-Reply-To: <1426793399-6283-29-git-send-email-julien.grall@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall Cc: xen-devel@lists.xenproject.org, tim@xen.org, Ian Jackson , stefano.stabellini@citrix.com, Wei Liu List-Id: xen-devel@lists.xenproject.org On Thu, 2015-03-19 at 19:29 +0000, Julien Grall wrote: > The functions fdt_{fisrt,next}_subnode may not be available because: "first" > diff --git a/tools/libxl/libxl_fdt.c b/tools/libxl/libxl_fdt.c > new file mode 100644 > index 0000000..f88e9f1 > --- /dev/null > +++ b/tools/libxl/libxl_fdt.c Since this is effectively shims for missing libfdt functionality how about libxl_libfdt_compat.c or some such? If wee wanted any fdt specific helpers as part of libxl itself then those would want to use the libxl_fdt.c name. > @@ -0,0 +1,84 @@ > +/* > + * libfdt - Flat Device Tree manipulation > + * Copyright (C) 2006 David Gibson, IBM Corporation. > + * > + * libfdt is dual licensed: you can use it either under the terms of > + * the GPL, or the BSD license, at your option. Since this is libxl, which should be LGPL I think we must therefore be taking the BSD option. Perhaps we should make that clear? I'm not sure. Ian.