Hi Feng,

On Mon, Jun 15, 2009 at 05:38:45PM +0800, Feng Tang wrote:
> This patch series adds driver for SPI controller of Intel Moorestown 
> platform.
> 
> This driver has been tested with Maxim's Max3110 UART device and
> Option's 3G modem GTM501L, both PIO and DMA works fine, and these
> 2 slave devcies can work simultaneously
> 
> User can use "dw_apb_ssi_db.pdf" from Synopsys as HW datasheet

Wow. This is a big surprise for me. I was working on a driver of the Synopsys 
Designware SPI controller myself during the last year. I posted my work on 
this list and got reviews from Linus Walleij. No word from David Brownell yet.  
I'll look into your work and see whether we can merge the drivers.

My first comment is regarding the name of the driver. This should be 
designware_spi, as this IP block appears in other chips as well, and is not 
Intel Moorestown specific.

> please help to review them.
> 
> I haven't seen your mail on this list for a while, so I cc' Andrew and Alan
> for their reviews as well.

baruch

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il -

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to