Thread (21 messages) flat view 21 messages, 6 authors, 2013-08-27
STALE4726d

[PATCH v2 4/4] iio: at91: introduce touch screen support in iio adc driver

From: Josh Wu <hidden>
Date: 2013-08-22 10:02:59
Also in: linux-iio

Hi, Jonathan

On 8/15/2013 6:27 PM, Jonathan Cameron wrote:
On 08/12/13 18:24, Dmitry Torokhov wrote:
quoted
Hi Josh,
Hi Dmitry,

As you haven't raised it here, I am guessing you are fine with having this
chunk of input related code in an IIO driver?  I wonder if it would be
better to break this into an mfd core and child input and iio drivers.
The two parts look fairly separable to me.
Clearly that means adding some overhead, but the gain is that the input
code ends up in input and the IIO code in IIO.

What are your views on this?  I'm fairly flexible if you think it simply
isn't worth the hassle of breaking this up.  I'm more worried about setting
a precident than anything else.

Jonathan
Since Dmitry has no feedback yet. I am not sure is it a right time to 
send out the next version for the IIO ad91 touch patch?

[snip...]

Best Regards,
Josh Wu
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help