[PATCH] ep93xx: Add support for Simplemachines Sim.One board
Ryan Mallon
ryan at bluewatersys.com
Wed Jan 20 16:59:51 EST 2010
H Hartley Sweeten wrote:
> On Wednesday, January 20, 2010 1:35 PM, Ryan Mallon wrote:
>
>> H Hartley Sweeten wrote:
>>
>>
>
> Two minor nitpicks below...
>
>
>> --- /dev/null
>> +++ b/arch/arm/mach-ep93xx/simone.c
>> @@ -0,0 +1,97 @@
>> +/*
>> + * arch/arm/mach-ep93xx/simone.c
>> + * Simplemachines Sim.One support.
>> + *
>> + * Copyright (C) 2009 Ryan Mallon <ryan at bluewatersys.com>
>>
>
> 2010?
>
I started writing the patch last year, hence the 2009. Can update if
necessary.
>> +static struct ep93xxfb_mach_info simone_fb_info = {
>> + .num_modes = EP93XXFB_USE_MODEDB,
>> + .bpp = 16,
>> + .flags = EP93XXFB_USE_SDCSN0 | EP93XXFB_PCLK_FALLING,
>> +};
>>
>
> Nitpick.. Please put the fb stuff above after the i2c stuff below.
>
Can do if necessary. Its pretty minor though.
>
> Did you happen to look at the patch I sent on 12/23/2009 with subject
> [PATCH] ep93xx: add missing SDRAM options for EDB93xx
>
> I think the EP93XX_SDCE0_PHYS_BASE can be changed to PHYS_OFFSET in
> all the ep93xx platform init code. I'm not sure what Uwe Kleine-König
> is going to do with the run time determined PHYS_OFFSET but I think
> that would at least get ep93xx ready for it.
>
The other boards are all still using the old way. I can change this
board now, or just leave as is and we can change all of the boards later?
>
> Other that that, looks good to me. This pretty much matches what I had
> pending. Have you at least compile tested this?
>
> It would be nice if someone could actually test this to make sure the
> kernel boots correctly on the Sim.One. But for what it's worth:
>
> Acked-by: H Hartley Sweeten <hsweeten at visionengravers.com>
>
I'll wait on the guys from Simplemachines to have their say before I put
this into the patch system.
~Ryan
--
Bluewater Systems Ltd - ARM Technology Solution Centre
Ryan Mallon 5 Amuri Park, 404 Barbadoes St
ryan at bluewatersys.com PO Box 13 889, Christchurch 8013
http://www.bluewatersys.com New Zealand
Phone: +64 3 3779127 Freecall: Australia 1800 148 751
Fax: +64 3 3779135 USA 1800 261 2934
More information about the linux-arm-kernel
mailing list