From owner-svn-src-head@freebsd.org Sat Jan 27 20:48:45 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 6353AEC165E; Sat, 27 Jan 2018 20:48:45 +0000 (UTC) (envelope-from manu@bidouilliste.com) Received: from mail.blih.net (mail.blih.net [212.83.177.182]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mail.blih.net", Issuer "mail.blih.net" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id A710E792CD; Sat, 27 Jan 2018 20:48:44 +0000 (UTC) (envelope-from manu@bidouilliste.com) Received: from mail.blih.net (mail.blih.net [212.83.177.182]) by mail.blih.net (OpenSMTPD) with ESMTP id 18b70aef; Sat, 27 Jan 2018 21:48:42 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=bidouilliste.com; h=date :from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; s=mail; bh=UKOWarcINVngJ2j2j9/PfzUQexI=; b=qGiVWbN6rHzH8cpouWCYYTaBcRPU FHvQ4vhwTT1aJAeHDq8k8j14ENpntkfr3lv95GnqSpmrcM2sPAQooOFs0E7PIUBc 0Ows6Cxy1NpLCD6QVwtNM2HebQolm/z2ZiXmzNEBaX1B0bdfwNKRDWN9szBlLwRX m1CXycaqnS82PR4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=bidouilliste.com; h=date :from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; q=dns; s= mail; b=NyNkajQD1Kl4LsaWe5qtb82wpTmhPALwLy+bxdLWdQqz0QuxVvdzWpPx BzDJMdaxlWqXP2oDUCbMvtR4HiXT+HmQG+xapYWV/OKKPrK5D9oiLRLIchqkO+pS b/ZZKlLv4zK5MSj4voqoYOElmci2apA9jhilwN+RGFxEixNpWC4= Received: from arcadia.home.blih.net (ip-9.net-89-3-105.rev.numericable.fr [89.3.105.9]) by mail.blih.net (OpenSMTPD) with ESMTPSA id 394df92a TLS version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO; Sat, 27 Jan 2018 21:48:41 +0100 (CET) Date: Sat, 27 Jan 2018 21:48:40 +0100 From: Emmanuel Vadot To: Adrian Chadd Cc: Poul-Henning Kamp , Warner Losh , John Baldwin , Ravi Pokala , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules Message-Id: <20180127214840.dc033d582696ff7b51f6d7e6@bidouilliste.com> In-Reply-To: References: <201801220710.w0M7AUm9091853@repo.freebsd.org> <90451.1516663240@critter.freebsd.dk> <2987003.eeGRFBb6N8@ralph.baldwin.cx> <93949.1516733748@critter.freebsd.dk> <20180127210801.37b8001125dd0a2c92372f98@bidouilliste.com> X-Mailer: Sylpheed 3.6.0 (GTK+ 2.24.31; amd64-portbld-freebsd12.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 27 Jan 2018 20:48:45 -0000 On Sat, 27 Jan 2018 12:13:57 -0800 Adrian Chadd wrote: > Hi, > > Find the middle ground. Don't dissuade the developer too much. This is what happened two years ago when I started hacking on the allwinner SoCs : - I asked what should be done for bringing a new board - andrew@ told me that we first need to switch to upstream dts and update drivers. - Guess what, I did that. > Here's an example: > > Make the driver follow DTS, allow a tunable/kenv check for it to > override whether it needs to be in the DTS or not (the "keep phk happy > for now" compromise) and have it default to obeying the device tree. > > That way phk is kept happy and the defaults are the same as normal-ish > ARM /and/ you have a springboard to experiment with extending FDT > overlays at runtime for people who wish to do so. I don't care about keeping phk@ (or any other developer) happy, we have a standard, let's stick to it. > (I personally hate having to edit the dts/recompile/reboot for every > test hardware change; it makes breadboarding things up kinda > hilariously annoying.) Use overlays then. And if you don't want to reboot provide patch for loading overlays at runtime. > -adrian -- Emmanuel Vadot