From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: "Mika Penttilä" <mika.penttila@nextfour.com>
Cc: "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
"曾婷葳 (tammy_tseng)" <tammy_tseng@sis.com>,
"游擱豪 (yuger_yu)" <yuger_yu@sis.com>
Subject: Re: [PATCH RESEND 2/2] Input: Documentation and MAINTAINERS parts.
Date: Tue, 12 Jan 2016 23:13:25 -0800 [thread overview]
Message-ID: <20160113071325.GC8906@dtor-ws> (raw)
In-Reply-To: <5695F6FE.5050407@nextfour.com>
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@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.
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.
--
Dmitry
--
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
next prev parent reply other threads:[~2016-01-13 7:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-13 5:51 [PATCH RESEND 0/2] Input: SiS 9200 family I2C touchscreen controller driver mika.penttila
2016-01-13 5:51 ` [PATCH RESEND 1/2] Input: Driver for SiS 9200 family multitouch controller mika.penttila
2016-01-13 5:51 ` mika.penttila
2016-01-13 5:51 ` [PATCH RESEND 2/2] Input: Documentation and MAINTAINERS parts mika.penttila
2016-01-13 5:51 ` mika.penttila
2016-01-13 6:47 ` Dmitry Torokhov
2016-01-13 7:04 ` Mika Penttilä
2016-01-13 7:13 ` Dmitry Torokhov [this message]
2016-01-13 8:06 ` Mika Penttilä
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160113071325.GC8906@dtor-ws \
--to=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=mika.penttila@nextfour.com \
--cc=tammy_tseng@sis.com \
--cc=yuger_yu@sis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).