Re: [PATCH] au1550-spi: proper platform driver

[Jan Nikitenko <jan.nikitenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

Manuel Lauss wrote:
> Hello Jan,
> 
> On Tue, Jun 10, 2008 at 11:46:02AM +0200, Jan Nikitenko wrote:
>> From: Manuel Lauss <mano-nEyxjcs6f3Vin2gBucwGBecsttgLyre6@public.gmane.org>
>> Date: Sat, 7 Jun 2008 12:14:37 +0200
>> Subject: [PATCH] au1550_spi: proper platform device
>>
>> Remove the Au1550 resource table and instead extract MMIO/IRQ/DMA
>> resources from platform resource information like any well-behaved
>> platform driver.
>>
>> Signed-off-by: Manuel Lauss <mano-nEyxjcs6f3Vin2gBucwGBecsttgLyre6@public.gmane.org>
>> [jan.nikitenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org: fixed request_mem_region() call arg,
>>  removed ioremap of memio (kseg1) resources,
> 
> Please leave it in.  Although it may seem pointless (now), ioremap()ing
> the physical base address will yield the KSEG1 address,  and as a nice
> side-effect /proc/iomem will also display the correct base address of the
> PSC.  (And to me the whole thing looks "cleaner" with it ;-) )

Having the ioremap() there, I got 0xc41da000 for 0xb1b00000 address of psc1 and 
that did not work on au1550 unfortunately. So we can't have it there...

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php