On 8/10/19 6:41 PM, Dmitry Torokhov wrote: > Hi Marek, Hi, > On Sat, Mar 02, 2019 at 03:17:04PM +0100, Marek Vasut wrote: >> Add support for ILI2117 touch controller. This controller is similar >> to the ILI210x and ILI251x, except for the following differences: >> - Reading out of touch data must happen at most 300 mS after the >> interrupt line was asserted. No command must be sent, the data >> are returned upon pure I2C read of 43 bytes long. >> - Supports 10 simultaneous touch inputs. >> - Touch data format is slightly different. > > So with this and also I see there is another ili2117a submission, I do > believe that we need to switch to using function pointers instead of > if/else if/else style cheking of the model. How about we add tested functionality in first and only then do bigger untested changes ? I think that would work better for everyone.
