lkml.org 
[lkml]   [2006]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] spi: Updated PXA2xx SSP SPI Driver
From
Date
On Fri, 2006-02-10 at 17:45 -0500, Nicolas Pitre wrote:
>
> Actually, it will do almost the same as you originally did if you look
> at its definition. However, because a solution needs to be implemented
> to support multiple ports then using __raw_readl() at the driver level
> is more familiar to other kernel people.
>
Yea, I did notice that also. I just wanted to make sure I understood
what is going on so that I do not make the same mistake for the third
time.

> In fact, if there was only one SSP port, I'd have asked you to use
> SSPCR0, SSPDR, etc. directly. But the fact that the driver can handle
> multiple ports makes it rather messy (and the SSP*_P() macros in
> pxa-regs.h are an abomination IMHO).
>
I did not use the SSP*_P functions because of the mess. Actually at one
point the driver implemented register accesses in similar manner as
SSP*_P but David Brownell got me straightened out. Thus the original
question.

Stephen



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-11 00:25    [W:0.038 / U:0.300 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site