From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from relay1-v.mail.gandi.net ([217.70.178.75]) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1NUHiJ-0004yb-Or for barebox@lists.infradead.org; Mon, 11 Jan 2010 10:41:40 +0000 Date: Mon, 11 Jan 2010 11:37:00 +0100 From: Matthias Kaehlcke Message-ID: <20100111103700.GH14051@darwin> References: <20100109232838.GB14051@darwin> <20100111094902.GR6923@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20100111094902.GR6923@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 0/5] Add support for EDB93xx boards To: Sascha Hauer Cc: barebox@lists.infradead.org Hi Sascha, El Mon, Jan 11, 2010 at 10:49:02AM +0100 Sascha Hauer ha dit: > Thanks for the nice work. The patches look mostly fine. I wonder that > U-Boot ep93xx support has never made it to the mainline. Some comments > about the patches inline. in late december i sent a (new) set of patches to the U-Boot mailing list. i still havn't received feedback, i suppose due to christmas, new year, ... my aim is to mainline ep93xx support, i am confident this will be accomplished soon. > On Sun, Jan 10, 2010 at 12:28:38AM +0100, Matthias Kaehlcke wrote: > > Add support for the following Cirrus Logic EDB93xx boards: > > > > EDB9301 > > EDB9302 > > EDB9302A > > EDB9307 > > EDB9307A > > EDB93012 > > EDB9315 > > EDB9315A > > > > This patchset has been tested on boards based on the EDB9301 and EDB9307A designs > > > > Matthias Kaehlcke (5): > > ARM: Add support for EP93xx SoCs > > Add EP93xx ethernet driver > > Add support for EP9xx GPIOs > > Add PrimeCell PL010 serial driver > > Add support for EDB93xx boards > > > > Documentation/boards.dox | 8 + > > MAINTAINERS | 11 + > > arch/arm/Kconfig | 6 + > > arch/arm/Makefile | 9 + > > arch/arm/configs/edb93xx_defconfig | 237 ++++++++ > > arch/arm/mach-ep93xx/Kconfig | 446 +++++++++++++++ > > arch/arm/mach-ep93xx/Makefile | 3 + > > arch/arm/mach-ep93xx/clocksource.c | 96 ++++ > > arch/arm/mach-ep93xx/gpio.c | 136 +++++ > > arch/arm/mach-ep93xx/include/mach/ep93xx-regs.h | 600 ++++++++++++++++++++ > > arch/arm/mach-ep93xx/include/mach/gpio.h | 29 + > > arch/arm/mach-ep93xx/led.c | 62 ++ > > arch/arm/mach-ep93xx/led.h | 26 + > > arch/arm/mach-ep93xx/lowlevel_init.S | 64 +++ > > Aren't the LED functions and thus the lowlevel_init board specific? > I can imagine many custom boards do not have LEDs connected to these > gpio pins. custom board could do this, but probably shouldn't. the EP93xx explicitly 'reserves' specific GPIOs for a red and a green LED. these LEDs are used by the boot algorithm to give feed back before the bootloader is started. please tell me if this convinces you to leave the LED functions and lowlevel_init in the SoC part or if you prefer moving them to the board directory. > > board/edb93xx/Makefile | 4 + > > board/edb93xx/barebox.lds.S | 79 +++ > > I think it's better to add a > > #ifdef CONFIG_ARCH_EP93XX > /* the EP93xx expects to find the pattern 'CRUS' at > * 0x1000 > */ > . = 0x1000; > LONG(0x53555243) /* 'CRUS' */ > #endif > > to the generic linker script. We might have to revert this when we're > heading to an ifdeffery here, but for now I think it's better be able to > make changes to the linker script in only one file. that's fine for me thanks for your review! -- Matthias Kaehlcke Embedded Linux Developer Barcelona If God had not intended for us to eat animals, how come He made them out of meat? (Sarah Palin) .''`. using free software / Debian GNU/Linux | http://debian.org : :' : `. `'` gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4 `- _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox