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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, Jonathan

On 8/15/2013 6:27 PM, Jonathan Cameron wrote:

On 08/12/13 18:24, Dmitry Torokhov wrote:
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
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux