Hi,
On 01/13/2016 08:47 AM, Dmitry Torokhov wrote:
> Hi Mika,
>
> On Tue, Jan 12, 2016 at 9:51 PM, <mika.penttila@xxxxxxxxxxxx> wrote:
>> From: Mika Penttilä <mika.penttila@xxxxxxxxxxxx>
>>
>> ---
>> .../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@xxxxxxxxxxxxxxx
>> S: Maintained
>> F: drivers/net/ethernet/sis/sis900.*
>>
>> +SIS 9200 I2C TOUCHSCREEN DRIVER
>> +M: Mika Penttilä <mika.penttila@xxxxxxxxxxxx>
>> +L: linux-input@xxxxxxxxxxxxxxx
>> +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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
