Re: [v4 3/3] 3/3 spi: Cypress TTSP G3 SPI Device Driver

On Thu, Jan 20, 2011 at 04:40:35PM +0530, Trilok Soni wrote:

Hi Dmitry,

On 1/13/2011 12:32 AM, Kevin McNeely wrote:

quoted

Hi Dmitry,

Thank you for your review.

Will the existing patches get into the current merge?

Can I revisit this discussion as part of our plan to provide a Protocol
B driver?

Thanks and best regards,
Kevin

Any updates on Kevin's question here?

Hi Trilok, Kevin,

Ah, sorry, was not feeling well last week and lost track of this driver.

No, I am sorry, bit the driver is not quite ready to be merged. Aside of
SPI abort/timeout issues we also have cyttsp_close() implementation
busted (once you close device you lose irq) and bunch of other things I
need to look over. I also have concern about having protocol A now and
upgrading to protocol B: clients that only understand protocol A will
get broken at A->B transition. I think we should consider implementing B
right away for mainline (no, I do not want A/B config options).

Sorry for being slow with doing full review...

Thanks.

-- 
Dmitry