From owner-freebsd-current@freebsd.org Wed Jul 20 04:48:17 2016 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 00F8BB9F060 for ; Wed, 20 Jul 2016 04:48:17 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-io0-x231.google.com (mail-io0-x231.google.com [IPv6:2607:f8b0:4001:c06::231]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id BF4D71D01 for ; Wed, 20 Jul 2016 04:48:16 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-io0-x231.google.com with SMTP id q83so37862804iod.1 for ; Tue, 19 Jul 2016 21:48:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=9suyaohUWBeV735L0ru2cUAnxrQp0ykuEX3icn0KfQw=; b=kMrFampAyrZ6E5nTe1mnEjvumC/sdPe6xLImxTg9LIhmg3xijE5HZE7iNHRTL2uSjb korRJvYWWMRpo50aZZHoP5Q7Z9YCBz7qnSGnV/n0PmZTFE+uI/aS1xAmHRKirAk0MsHw fAlu6+noPEEduTLOiAgqu97yWLbVqxXqX1yKQW5MPb0ITSYAjxTjZ1Q5p1BJ9O3i13OO lKHg0/COL+/vDDqUxLbbgZNoNyMJ5CGFbvVuGS8kmBxELzF026xHXAGQxjg6POKGbvGw 3MDT3EB/+zVQJLV9q74OKZ+2rGqTE+53P5uQXW+2hpzTOAe+b1DXxM47y9ajFs2lGvX+ QuOA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=9suyaohUWBeV735L0ru2cUAnxrQp0ykuEX3icn0KfQw=; b=nLQPva21ISWAWzSgjITdUYXwaMVTUh3ygamMvVCopjZ+7YGWm0g7p20rICTFuql6zf IykHC3OH3ii6DzEn3KxoiwhKExidm6GeLZa16pgEZC942jcL1LovUSYk66tZvgEZ0vPV rSSqMW6g8h/mmVxoudyR4MTIrH2D6AA3emLmcpH9BB5h5I+SAz6Ut7YOHhR2xu1XUr1g WbOg6v0fHnJgSGOJGqo+Vp64cgHB7SWoHggeyMOPpW98j7N6Jf2wRTOp7nW/in8GXPLU /qjoVLnGrBvv+1wzvgGUH4rRWyFvbQWR7X6lwpFRkgK1snpxDP0oNe8MOChT9eXaZR7h m3/Q== X-Gm-Message-State: ALyK8tLVo5RSu9V2GNBSAGB0P7aSiaOAzLG60y/A5+OYnUaqWC+YQMpor3GJkHrhMW2gBa1YqvAwhRjK9z8uSA== X-Received: by 10.107.40.133 with SMTP id o127mr22216139ioo.183.1468990096156; Tue, 19 Jul 2016 21:48:16 -0700 (PDT) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 10.79.137.131 with HTTP; Tue, 19 Jul 2016 21:48:15 -0700 (PDT) X-Originating-IP: [69.53.245.200] In-Reply-To: References: From: Warner Losh Date: Tue, 19 Jul 2016 22:48:15 -0600 X-Google-Sender-Auth: seQkqHe1kGTCx-UKLnXRDzXMIVs Message-ID: Subject: Re: Call for Testing: Switching back to our BSD licensed dtc(1) To: Ed Maste Cc: FreeBSD Current Content-Type: text/plain; charset=UTF-8 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Jul 2016 04:48:17 -0000 On Tue, Jul 19, 2016 at 1:12 PM, Ed Maste wrote: > dtc(1) is the Device Tree Compiler, used for embedded builds. We have > two versions of dtc(1) in the FreeBSD tree: a GPLv2 one from > https://git.kernel.org/cgit/utils/dtc/dtc.git and a BSD licensed one > in https://svn.freebsd.org/base/head/usr.bin/dtc. > > We switched back to the GPL one since device tree files in the most > recent import required features not available in our own at the time. > However, as of r292876 the BSD licensed dtc(1) is functional and we > should be able to switch back to it. > > I would encourage embedded users (primarily ARM boards) to test with > WITHOUT_GPL_DTC in /etc/src.conf and report their success or failure. My concern with this is overlay support. Overlay support for "shields" or "badges" is going into upstream GPL dtc shortly. The patches have been kicking around for a while and are almost ready to go in. We need this feature and I'd planned on bringing it into the tree as soon as it goes into the git repo. It's sorely needed for things like RPi and BBB that make it super easy to add boards with parts that need a bit more DTB into the FDT to describe them. There's been patches to bring this functionality into the loader and into the in-tree gpl dtc floating around for a while now. I know that the BSDL compiler doesn't support this. I know from painful experience that the BSDL compiler is quite hard to fix grammar items for since it doesn't use yacc, but uses a hand-coded parser / lexer. If support could be added for this to that, we can throw the switch. Otherwise, I don't think that it would be wise to switch. Last time I ran my tests on the BSDL compiler, it failed a few of them. I'll re-run to see if things have improved since I last checked it out and report any failures here. Warner