From owner-freebsd-current@FreeBSD.ORG Sun Sep 12 16:57:45 2010 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 204E41065672; Sun, 12 Sep 2010 16:57:45 +0000 (UTC) (envelope-from marius@alchemy.franken.de) Received: from alchemy.franken.de (alchemy.franken.de [194.94.249.214]) by mx1.freebsd.org (Postfix) with ESMTP id 804888FC1C; Sun, 12 Sep 2010 16:57:43 +0000 (UTC) Received: from alchemy.franken.de (localhost [127.0.0.1]) by alchemy.franken.de (8.14.3/8.14.3/ALCHEMY.FRANKEN.DE) with ESMTP id o8CGNqG8089754; Sun, 12 Sep 2010 18:23:53 +0200 (CEST) (envelope-from marius@alchemy.franken.de) Received: (from marius@localhost) by alchemy.franken.de (8.14.3/8.14.3/Submit) id o8CGNqJ6089753; Sun, 12 Sep 2010 18:23:52 +0200 (CEST) (envelope-from marius) Date: Sun, 12 Sep 2010 18:23:52 +0200 From: Marius Strobl To: Alexander Best Message-ID: <20100912162352.GA89643@alchemy.franken.de> References: <20100909191750.GA58228@freebsd.org> <20100909194109.GA64914@freebsd.org> <20100909195045.GA68352@freebsd.org> <201009100747.39964.jhb@freebsd.org> <20100912024049.GA32655@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100912024049.GA32655@freebsd.org> User-Agent: Mutt/1.4.2.3i Cc: freebsd-current@freebsd.org Subject: Re: {arch}/conf/DEFAULTS and uart X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 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: Sun, 12 Sep 2010 16:57:45 -0000 On Sun, Sep 12, 2010 at 02:40:49AM +0000, Alexander Best wrote: > On Fri Sep 10 10, John Baldwin wrote: > > On Thursday, September 09, 2010 3:50:45 pm Alexander Best wrote: > > > On Thu Sep 9 10, Alexander Best wrote: > > > > On Thu Sep 9 10, Alexander Best wrote: > > > > > hi there, > > > > > > > > > > except for arm most archs seem to enforce uart support in conf/DEFAULTS. is > > > > > this really necessary? shouldn't DEFAULTS only contain vital devices/options > > > > > without a kernel on a specific arch won't function at all? > > > > > > > > jhb just explained to me, that the uart entry in DEFAULTS is not a controller > > > > or something like that, but the uart backend to use *if* uart gets defined in > > > > the kernel config. > > > > > > > > sorry for the noise folks. > > > > > > however i found some missing comments and incorrect syntax which i fixed. > > > > > > see the attached patch. > > > > I think the ia64 ordering for 'io and mem' is probably more correct > > (alphabetically sorted), so I would fix i386 and amd64 and leave ia64 alone. > > > > The powerpc 'machine' changes are wrong I think as it would break GENERIC64 > > and powerpc64 kernel configs in general. Nathan purposefully removed > > 'machine' from the powerpc DEFAULTS. > > here's try #2. ;) > > diff --git a/sys/sparc64/conf/DEFAULTS b/sys/sparc64/conf/DEFAULTS > index 38b2408..2e60c94 100644 > --- a/sys/sparc64/conf/DEFAULTS > +++ b/sys/sparc64/conf/DEFAULTS > @@ -5,7 +5,7 @@ > > machine sparc64 > > -# Pseudo devices. > +# Pseudo devices > device mem # Memory and kernel memory devices > > # UART chips on this platform > @@ -17,5 +17,5 @@ device uart_z8530 > options GEOM_PART_BSD > options GEOM_PART_VTOC8 > > -# Let sunkbd emulate an AT keyboard by default. > +# Let sunkbd emulate an AT keyboard by default IMO this is a complete sentence and thus the period should stay. Marius