From owner-svn-src-head@freebsd.org Tue Jan 30 09:42:58 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 1842FEE5074; Tue, 30 Jan 2018 09:42:58 +0000 (UTC) (envelope-from gonzo@bluezbox.com) Received: from id.bluezbox.com (id.bluezbox.com [45.55.20.155]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id A84C48448E; Tue, 30 Jan 2018 09:42:57 +0000 (UTC) (envelope-from gonzo@bluezbox.com) Received: from localhost ([127.0.0.1] helo=id.bluezbox.com) by id.bluezbox.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.89 (FreeBSD)) (envelope-from ) id 1egSRJ-000MEt-B6; Tue, 30 Jan 2018 01:42:54 -0800 Received: (from gonzo@localhost) by id.bluezbox.com (8.15.2/8.15.2/Submit) id w0U9gpLC085490; Tue, 30 Jan 2018 01:42:51 -0800 (PST) (envelope-from gonzo@bluezbox.com) X-Authentication-Warning: id.bluezbox.com: gonzo set sender to gonzo@bluezbox.com using -f Date: Tue, 30 Jan 2018 01:42:51 -0800 From: Oleksandr Tymoshenko To: Poul-Henning Kamp Cc: Emmanuel Vadot , Warner Losh , John Baldwin , Ravi Pokala , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org, owner-src-committers@freebsd.org Subject: Re: svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules Message-ID: <20180130094251.GA85100@bluezbox.com> References: <20180127210801.37b8001125dd0a2c92372f98@bidouilliste.com> <72042.1517094867@critter.freebsd.dk> <8d8ae9d10058fd72ce3ec467181c9f22@megadrive.org> <13025.1517179897@critter.freebsd.dk> <20180129063950.GA59901@bluezbox.com> <32793.1517221534@critter.freebsd.dk> <20180129132736.GA66330@bluezbox.com> <33452.1517233531@critter.freebsd.dk> <20180130025048.GA76676@bluezbox.com> <90452.1517301074@critter.freebsd.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <90452.1517301074@critter.freebsd.dk> X-Operating-System: FreeBSD/11.1-RELEASE-p4 (amd64) User-Agent: Mutt/1.9.1 (2017-09-22) X-Spam-Level: -- X-Spam-Report: Spam detection software, running on the system "id.bluezbox.com", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see The administrator of that system for details. Content preview: Poul-Henning Kamp (phk@phk.freebsd.dk) wrote: > -------- > In message <20180130025048.GA76676@bluezbox.com>, Oleksandr Tymoshenko writes: > > >> You seem to have skipped the bit about "without reboot" [...] Content analysis details: (-2.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] 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: Tue, 30 Jan 2018 09:42:58 -0000 Poul-Henning Kamp (phk@phk.freebsd.dk) wrote: > -------- > In message <20180130025048.GA76676@bluezbox.com>, Oleksandr Tymoshenko writes: > > >> You seem to have skipped the bit about "without reboot" ? > > > >No I haven't. config.txt is parsed by VC firmware on boot time > >and pwm.dtbo is applied even before boot loader kicks in. When > >kernel starts pwm node has pinctl information ready and status > >property is set to "okay". So if you download next available image, > >flash and boot it and run kldload bcm2835_pwm it will give you > >working PWM device out of the box without reboot. > > But only if you add pwm.dtbo to config.txt and reboot, because > I see nothing indicating that we will put it there in the > release images we (may start to) produce ? No. We're going to add them in our release image. That's what I meant by "working with Glen and Brad". Glen as release engineer built all those scripts that prepare SoC images. He works closely with FreeBSD/ARM community and although I can't speak for him I hope he won't object to the change that introduces new feature and doesn't violate POLA. We already install two overlays that are required to get reasonable boot-time defaults. PI3 release image config is here: http://svn.freebsd.org/base/head/release/arm64/RPI3.conf Adding pwm.dtbo is one-line change. Adding line to config.txt is one more line. RaspBSD is volunteer project by Brad Davis so it's completely up to him whether to enable PWM by default or not. He knows his userbase best. All one can do is to make a case and present it to Brad. -- gonzo