On 01/13/2016 09:13 AM, Dmitry Torokhov wrote:
> On Wed, Jan 13, 2016 at 09:04:30AM +0200, Mika Penttilä wrote:
>> 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.
>
> As long as you are not butting heads with them over the same codebase I
> am fine with either option. I'll take a look at your version in a day or
> so, and I will expect you work with SiS folks on the next version.
>
> Ultimately it would be the best if SiS maintains the code once it is
> in mainline so they can keep adding support for newer hardware.
>
> Thanks.
>
Yes, sounds ok to me,
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
