From owner-svn-src-head@freebsd.org Tue Dec 12 17:05:40 2017 Return-Path: Delivered-To: svn-src-head@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 2134CE9F1A0; Tue, 12 Dec 2017 17:05:40 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id E3FA865C2A; Tue, 12 Dec 2017 17:05:39 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id vBCH5cls087174; Tue, 12 Dec 2017 09:05:38 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id vBCH5bHb087173; Tue, 12 Dec 2017 09:05:37 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201712121705.vBCH5bHb087173@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r326773 - in head/sys: conf dev/syscon In-Reply-To: <20171212165828.e9a0eb263cc4dca94d0d6ca9@bidouilliste.com> To: Emmanuel Vadot Date: Tue, 12 Dec 2017 09:05:37 -0800 (PST) CC: rgrimes@freebsd.org, Alexey Dokuchaev , Nathan Whitehorn , Kyle Evans , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII 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, 12 Dec 2017 17:05:40 -0000 > On Tue, 12 Dec 2017 07:40:21 -0800 (PST) > "Rodney W. Grimes" wrote: > > > > On Tue, 12 Dec 2017 06:18:36 +0000 > > > Alexey Dokuchaev wrote: > > > > > > > On Mon, Dec 11, 2017 at 10:14:04AM -0800, Nathan Whitehorn wrote: > > > > > I think this name might confuse people looking for "syscons". Can it be > > > > > renamed? Also, if it is ARM-specific, maybe it belongs in sys/arm? > > > > > > > > +1 for rename, it's very confusing now. > > > > ++1 rename it or expect irrate users > > So where exactly in the confusion ? > > Is it because your tab completion when doing cd sys/dev/sysc will > now ask you to type more char ? > This is the only place where "syscons" (not syscon) exists. My appologies, I did not notice that there infact is a delta of syscon vs syscons. I thought that the name was an exact match. I still not real happy with it, but my objection is retracted. > > > > > > > > ./danfe > > > > > > It's not confusing. The spec is clear it's named "syscon" so our > > > driver must be named syscon, naming it otherwise would be confusing. > > > What's confusing is the directory name for "sc" named syscons, it's > > > config options are named SC_* and the device is named sc. > > > > The spec is wrong for dictating what any implementation must > > call this, the spec does not own the implementation name space. > > > > It is called called syscons and uses the variables and driver > > names sc beacuse that is what we did back in the 90's, drivers > > typically had 2 character names. > > > > > The only valuable remark in this thread is bde's one saying it's not > > > documented etc ... and yes we do sucks in this area in the arm world. > > > > Thank you for your dismissal of input from the community, does that > > mean your gong to ignore them? > > > > > Anyhow, I caught a discussion between mmel@ and kevans@ on IRC saying > > > that we will move the driver into extres after adding some standard > > > stuff common to extres framework. > > > > FYI why is this so bad an idea to call it syscons: > > man syscons > > How do you plan to resolve that name space conflict? > > AND this man page has existed since 1.0, So 25 years of > > finger memory, your just not gona do well trying to change that. > > The new driver name is "syscon" not "syscons", there is no name space > conflict. Again, I am sorry, I did not notice this and thought we had a direct conflict. > > Again, we will move it to sys/dev/extres, now can we stop whining > about small details like that and do some actual work please ? I am glad it is moving, but that was never the reason that someone raised issue. Please stop dismissing concerns, especially if raised by more than one person, as whining, that is not helpful. -- Rod Grimes rgrimes@freebsd.org