From owner-freebsd-arm@FreeBSD.ORG Tue Nov 25 19:06:18 2008 Return-Path: Delivered-To: arm@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id EC0AC1065670; Tue, 25 Nov 2008 19:06:18 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id AE94E8FC19; Tue, 25 Nov 2008 19:06:18 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.2/8.14.1) with ESMTP id mAPJ3r55061842; Tue, 25 Nov 2008 12:03:53 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Tue, 25 Nov 2008 12:05:23 -0700 (MST) Message-Id: <20081125.120523.-201316873.imp@bsdimp.com> To: stas@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <20081125220040.c8996e0b.stas@FreeBSD.org> References: <20081125212409.3dab8178.stas@FreeBSD.org> <20081125.113647.1346821827.imp@bsdimp.com> <20081125220040.c8996e0b.stas@FreeBSD.org> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: arm@FreeBSD.org Subject: Re: Code review request: boards on AT91 X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the StrongARM Processor List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 25 Nov 2008 19:06:19 -0000 In message: <20081125220040.c8996e0b.stas@FreeBSD.org> Stanislav Sedov writes: : -----BEGIN PGP SIGNED MESSAGE----- : Hash: SHA1 : : On Tue, 25 Nov 2008 11:36:47 -0700 (MST) : "M. Warner Losh" mentioned: : : > : : > : I think we could pass the board type via a special kenv variable : > : for now. I think it will work fine and applicable to all supported : > : architectures. We could probably reuse board type constants that Linux : > : kernel uses. : > : > Right now that's passed in from uboot and other loaders in I think r3, : > but I'd have to go look at some other code to be sure. : > : : Well, I don't think we want to follow The Linux kernel argument passing : paradigm... But using a simple kenv variable would be just fine. : We have a patch ready for kenv/hints support in u-boot and in : our arm port, and our loader could be used on arm with Rafal's : u-boot API. You are assuming that /boot/loader is always used. In the cases where it isn't, we need that value from r3. when it is, a kenv could be used. This switching likely also needs to be configurable more easily than it is now. Warner