>-----Original Message-----
>From: David Brownell [mailto:[email protected]]
>Sent: 2009年6月24日 10:10
>To: Tang, Feng; Baruch Siach
>Cc: [email protected]; [email protected];
>[email protected]
>Subject: Re: [spi-devel-general] [PATCH 0/2] spi: add driver for SPI 
>controller of Intel
>Moorestown platform
>
>On Monday 15 June 2009, Tang, Feng wrote:
>> >>
>> >> 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.
>>
>> Good idea, we can work together to make the driver better.
>>
>> Regarding the driver name, it is implemented as a PCI device driver, also it 
>> has
>> some Moorestown specific stuff like DMA operation, DMA part of the driver 
>> only
>> works with the DMA controller on Moorestown, that's why I used such a name.
>> Anyway, we can discuss more about it.
>
>Then the way to structure it is the classic "bus glue plus core".
>
>Core would be a "designware_spi" (or, "dw_spi" by analogy to the
>"dw_dmac" DMA driver).  It would either expose private calls to
>bus glue -- PCI or APB, I guess -- or would be a platform_bus thing
>with the PCI glue creating a platform_bus child of a PCI node.

Thanks for the suggestion, will take a look at it

- Feng.

>
>- Dave

------------------------------------------------------------------------------
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to