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

From: Dmitry Torokhov
Date: Fri Jan 21 2011 - 04:27:56 EST


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