Re: [PATCH RESEND 2/2] Input: Documentation and MAINTAINERS parts.

["Mika Penttilä" <mika.penttila@nextfour.com>]

Hi,

On 01/13/2016 08:47 AM, Dmitry Torokhov wrote:
> Hi Mika,
> 
> On Tue, Jan 12, 2016 at 9:51 PM,  <mika.penttila@nextfour.com> wrote:
>> From: Mika Penttilä <mika.penttila@nextfour.com>
>>
>> ---
>>  .../bindings/input/touchscreen/sis_i2c.txt         | 22 ++++++++++++++++++++++
>>  .../devicetree/bindings/vendor-prefixes.txt        |  1 +
>>  MAINTAINERS                                        |  7 +++++++
>>  3 files changed, 30 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/input/touchscreen/sis_i2c.txt
>>
>> diff --git a/Documentation/devicetree/bindings/input/touchscreen/sis_i2c.txt b/Documentation/devicetree/bindings/input/touchscreen/sis_i2c.txt
>> new file mode 100644
>> index 0000000..b0e9980
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/input/touchscreen/sis_i2c.txt
>> @@ -0,0 +1,22 @@
>> +* SiS I2C Multiple Touch Controller
>> +
>> +Required properties:
>> +- compatible: must be "sis,9200_ts"
>> +- reg: i2c slave address
>> +- interrupt-parent: the phandle for the interrupt controller
>> +- interrupts: touch controller interrupt
>> +- irq-gpios: the gpio pin used as irq pin
>> +- reset-gpios: the gpio pin used to reset the controller
>> +
>> +Example:
>> +
>> +       sis9255@5c  {
>> +                compatible = "sis,9200_ts";
>> +                reg = <0x5c>;
>> +                pinctrl-names = "default";
>> +                pinctrl-0 = <&pinctrl_sis>;
>> +                interrupt-parent = <&gpio3>;
>> +                interrupts = <19 2>;
>> +                irq-gpios = <&gpio3 19 0>;
>> +                reset-gpios = <&gpio2 30 0>;
>> +       };
>> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> index 55df1d4..a6ed5f9 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> @@ -207,6 +207,7 @@ simtek
>>  sii    Seiko Instruments, Inc.
>>  silergy        Silergy Corp.
>>  sirf   SiRF Technology, Inc.
>> +sis    Silicon Integrated Systems Corp.
>>  sitronix       Sitronix Technology Corporation
>>  skyworks       Skyworks Solutions, Inc.
>>  smsc   Standard Microsystems Corporation
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 233f834..fbf9470 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -9809,6 +9809,13 @@ L:       netdev@vger.kernel.org
>>  S:     Maintained
>>  F:     drivers/net/ethernet/sis/sis900.*
>>
>> +SIS 9200 I2C TOUCHSCREEN DRIVER
>> +M:     Mika Penttilä <mika.penttila@nextfour.com>
>> +L:     linux-input@vger.kernel.org
>> +S:     Maintained
>> +F:     Documentation/devicetree/bindings/input/touchscreen/sis_i2c.txt
>> +F:     drivers/input/touchscreen/sis_i2c.c
> 
> I have seen (and reviewed) the driver for the same part posted by SIS
> folks, are you working with them?
> 
> Thanks.
> 

We are an independent company from SiS and using sis controller in our
product and have communicated privately with SiS wrt driver. I didn't
see their very first posting while developing this, but later ones yes
and I think this version is right wrt multitouch protocol B handling,
doesn't keep special private state etc.

We would like to see this mainlined soon so whichever route is best is
ok for me. I have the time now and can actively work on this towards
inclusion with the support of SiS folks. If ok would be nice to get
signed-of-bys's from SiS also.

Thanks,
Mika

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html