linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Vishnu Patekar <vishnupatekar0510-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
Cc: dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org,
	linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
	leafy.myeh-Q9AEpCAkrSgqDJ6do+/SaQ@public.gmane.org,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	pawel.moll-5wv7dgnIgG8@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org,
	galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	VishnuPatekar0510-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCHv2 0/3] ARM:sunxi:ps2 Added support for A10/A20 ps2 controller.
Date: Fri, 12 Dec 2014 05:55:50 -0800 (PST)	[thread overview]
Message-ID: <e46c1d5e-835f-466c-a26d-a94d23ea30c2@googlegroups.com> (raw)
In-Reply-To: <548AEA8E.6040100-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>


[-- Attachment #1.1: Type: text/plain, Size: 3156 bytes --]

Hello Hans,
Thank you for testing it on A10 board. now we have PS2 working on A10 as 
well. :)

I'll include these in next version which I'm was about to send. 


On Friday, December 12, 2014 6:46:25 PM UTC+5:30, Hans de Goede wrote:
>
> Hi, 
>
> On 07-12-14 00:15, vishnupatekar wrote: 
> > Here is v2 of SUNXI PS2 controller support patch-set as with v1. 
>
> So as promised I've run this on an A10 board, with a small fix to 
> the interrupt property in the sun4i.dtsi file this works as is. 
>
> This means that the sun7i compat should be dropped, and the sun4i 
> compat used everywhere. 
>
> I've created 3 fixup patches to do this, note one also renames / moves 
> the bindings file as it is in the wrong place. 
>
> I've attached the 3 fixup patches, pleasesquash these into your 
> existing patches before posting the next version. 
>
> Thanks & Regards, 
>
> Hans 
>
>
> > 
> > Changes in v2: 
> > 1. added default n depends on ARCH_SUNXI || COMPILE_TEST in Kconfig. 
> > 2. handled errors and free resources on errors. 
> > 3. used BIT(x), DIV_ROUND_UP macros. 
> > 4. corrected style errors. 
> > 5. added support for A10 also, A10 and A2 have same properties of PS2 
> controller. 
> > 6. by default commented ps20 and ps21 nodes,as ps20 pins conflict with 
> HDMI 
> >     connector on Lime2 Board. 
> > 7. added compatible as allwinner,sun4i-a10-ps2. 
> > 8. corrected the possible race condition. 
> > 
> > Patch 0 Summary: Allwinner A10/A20 PS2 controller. These modifications 
> are 
> >         for PS2 host controller. IBM compliant IBM PS2 and AT-compatible 
> >         keyboard and mouse can be connected. 
> > 
> > Patch 1 device tree bindings. 
> > 
> > Patch 2 adds support for sun7i ps2 driver. 
> > 
> > Patch 3 device tree support for PS2 controller. 
> >   1) Added A10/A20 ps2 nodes to the dtsi 
> >   2) Added A10/A20 ps2 pinmux to the dtsi 
> >   3) Added ps2 nodes to the lime2 dts file 
> > 
> > vishnupatekar (3): 
> >    sunxi:dts-bindings:input:ps2 bindings for A10/A20 ps2. 
> >    sunxi:drivers:input:ps2 Added sunxi A10/A20 ps2 driver 
> >    ARM:dts:sunxi:ps2 dt nodes for A10/A20 PS2 controller. 
> > 
> >   .../bindings/input/allwinner,sunxi-ps2.txt         |   23 ++ 
> >   arch/arm/boot/dts/sun4i-a10.dtsi                   |   27 ++ 
> >   arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts    |   13 + 
> >   arch/arm/boot/dts/sun7i-a20.dtsi                   |   29 ++ 
> >   drivers/input/serio/Kconfig                        |   10 + 
> >   drivers/input/serio/Makefile                       |    1 + 
> >   drivers/input/serio/sunxi-ps2.c                    |  364 
> ++++++++++++++++++++ 
> >   7 files changed, 467 insertions(+) 
> >   create mode 100644 
> Documentation/devicetree/bindings/input/allwinner,sunxi-ps2.txt 
> >   create mode 100644 drivers/input/serio/sunxi-ps2.c 
> > 
>

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

[-- Attachment #1.2: Type: text/html, Size: 4352 bytes --]

      parent reply	other threads:[~2014-12-12 13:55 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-06 23:15 [PATCHv2 0/3] ARM:sunxi:ps2 Added support for A10/A20 ps2 controller vishnupatekar
     [not found] ` <1417907719-26775-1-git-send-email-VishnuPatekar0510-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-06 23:15   ` [PATCHv2 1/3] sunxi:dts-bindings:input:ps2 bindings for A10/A20 ps2 vishnupatekar
2014-12-08 22:15     ` Maxime Ripard
2014-12-09 10:21       ` Vishnu Patekar
2014-12-06 23:15   ` [PATCHv2 2/3] sunxi:drivers:input:ps2 Added sunxi A10/A20 ps2 driver vishnupatekar
     [not found]     ` <1417907719-26775-3-git-send-email-VishnuPatekar0510-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-08  4:43       ` Vishnu Patekar
2014-12-08 22:41       ` Maxime Ripard
     [not found]         ` <CAEzqOZu9TWmYyBOP8t+b7mepqdbKjhf-3P-uNJS_Od05XiBLZw@mail.gmail.com>
     [not found]           ` <CAEzqOZu9TWmYyBOP8t+b7mepqdbKjhf-3P-uNJS_Od05XiBLZw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-12-11 21:09             ` Maxime Ripard
2014-12-06 23:15   ` [PATCHv2 3/3] ARM:dts:sunxi:ps2 dt nodes for A10/A20 PS2 controller vishnupatekar
     [not found]     ` <1417907719-26775-4-git-send-email-VishnuPatekar0510-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-12-08 22:22       ` Maxime Ripard
2014-12-12 13:15   ` [PATCHv2 0/3] ARM:sunxi:ps2 Added support for A10/A20 ps2 controller Hans de Goede
     [not found]     ` <548AEA8E.6040100-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-12-12 13:55       ` Vishnu Patekar [this message]

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=e46c1d5e-835f-466c-a26d-a94d23ea30c2@googlegroups.com \
    --to=vishnupatekar0510-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=leafy.myeh-Q9AEpCAkrSgqDJ6do+/SaQ@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    /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).