From owner-freebsd-arm@FreeBSD.ORG Mon Jun 21 12:08:50 2010 Return-Path: Delivered-To: freebsd-arm@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D62051065690; Mon, 21 Jun 2010 12:08:50 +0000 (UTC) (envelope-from raj@semihalf.com) Received: from smtp.semihalf.com (smtp.semihalf.com [213.17.239.109]) by mx1.freebsd.org (Postfix) with ESMTP id 1EE3D8FC15; Mon, 21 Jun 2010 12:08:49 +0000 (UTC) Received: from localhost (unknown [213.17.239.109]) by smtp.semihalf.com (Postfix) with ESMTP id 9DCD0C42DA; Mon, 21 Jun 2010 14:14:18 +0200 (CEST) X-Virus-Scanned: by amavisd-new at semihalf.com Received: from smtp.semihalf.com ([213.17.239.109]) by localhost (smtp.semihalf.com [213.17.239.109]) (amavisd-new, port 10024) with ESMTP id D0rYzjmZWf4W; Mon, 21 Jun 2010 14:14:17 +0200 (CEST) Received: from [10.0.0.79] (cardhu.semihalf.com [213.17.239.108]) by smtp.semihalf.com (Postfix) with ESMTPSA id C182BC42D5; Mon, 21 Jun 2010 14:14:17 +0200 (CEST) Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii From: Rafal Jaworowski In-Reply-To: <20100620220302.8eb84547.nork@FreeBSD.org> Date: Mon, 21 Jun 2010 14:08:47 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: References: <20100620220302.8eb84547.nork@FreeBSD.org> To: Norikatsu Shigemura X-Mailer: Apple Mail (2.1081) Cc: Alexander Motin , Warner Losh , freebsd-arm@FreeBSD.org Subject: Re: OpenRD-Client/Ultimate support 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: Mon, 21 Jun 2010 12:08:51 -0000 On 2010-06-20, at 15:03, Norikatsu Shigemura wrote: > Hi raj, imp and mav. >=20 > I made some patches for OpenRD Client and Ultimate. By my = patches, > I confirmed that I can use OpenRD Ultimate (Sorry, I don't have > OpenRD Client). If you OKed, I'll commit attached patches. Great to see OpenRD converted to FDT, thanks a lot for your work! > My modifications: >=20 > mge(4): > 1. Change how to get PHY numbers if not defined `phy-handle' > on a dts file. OpenRD already set it on FreeBSD booting. > Suggested by: hrs@ Why would you want to handle a scenario when there's no phy-handle? It's = mandatory if the Ethernet controller is connected to a physical layer = device. > 2. Change how to set PHY numbers. > Suggested by: Kristof Provost Please elaborate on what is being fixed with PHY numbering here. > 3. Remove waiting link-up codes. > Reported by: nyan@ >=20 > 4. Don't count-up watchdog timer if link downed. > Pointed out by: yongari@ There's also a related thread on link problems with mge(4), do you think = your changes will help with these issues as well? = http://docs.freebsd.org/cgi/getmsg.cgi?fetch=3D118358+0+archive/2010/freeb= sd-net/20100620.freebsd-net > mvs(4): > 5. FDT-ish >=20 > Kernel Configuration: > 6. Generated by DB-88F6XXX, and added some my flavour, mvs(4). >=20 > Device Tree Source: > 7. Generated by db88f6281.dts. > 8. Add second NIC. > 9. Setup MPP for OpenRD Client/Ultimate. > Obtained from: u-boot-1.1.4_2010.5.5.tar.bz2 on = http://code.google.com/p/openrd/downloads/list In general the DTS looks good, only the PHY info seems missing. There's = a lot in common between DTS contents for systems based on the same SOC, = and one day we should have the common piece factored out, although it's = a more general problem how to make such sharing optimal, and not easy to = resolve. > 10. Change PCI address range. >=20 > TODO: > a. OpenRD base support (some different MPP) > b. uart(4) is storange, maybe s/1066/115200/: > uart0: console (1066,n,8,1) > c. PCI mem address range 0xf4000000 to 0xe8000000 (max 128MB) > In this time, I can get following results, so I set following > parameter to openrd-cl.dts. > If PCIe device require 128MB, maybe, can't allocate. > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - = - - - - - - - > map[10]: type Prefetchable Memory, range 32, base 0xf4000000, = size 26, enabled > map[14]: type Memory, range 32, base 0xf8000000, size 18, = enabled > map[18]: type I/O Port, range 32, base 0xf1100000, size 7, = enabled > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - = - - - - - - - > But I can't understand how to describe pci0: related > parameters. So I don't have any idea to change address > window. Note the PCI/E driver for Marvell as is in SVN has some problems = (doesn't respect/force alignment of windows, which is required by = hardware etc.), which might be affecting you here. We have some rework = and improvements in this regard in a local tree, but I need to = reintegrate them against the FDT-driven code. Rafal