Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966469AbaLMLMD (ORCPT ); Sat, 13 Dec 2014 06:12:03 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57053 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966450AbaLMLMA (ORCPT ); Sat, 13 Dec 2014 06:12:00 -0500 Message-ID: <548C1E7D.1000005@redhat.com> Date: Sat, 13 Dec 2014 12:09:49 +0100 From: Hans de Goede User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: VishnuPatekar , linux-input@vger.kernel.org, maxime.ripard@free-electrons.com, dmitry.torokhov@gmail.com CC: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux@arm.linux.org.uk, grant.likely@linaro.org, benh@kernel.crashing.org, msalter@redhat.com, ralf@linux-mips.org, jdelvare@suse.de Subject: Re: [PATCHv3 0/5] ARM:sunxi:ps2 Added support for A10/A20 ps2 controller. References: <1418408748-9797-1-git-send-email-vishnupatekar0510@gmail.com> In-Reply-To: <1418408748-9797-1-git-send-email-vishnupatekar0510@gmail.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi VishnuPatekar, The patch mangling for this set seems to have gone a bit wrong I'm afraid, a lot of the patches have my fixup commit messages (which should have disappeared when squashing in the patches), please replace those by proper commit messages describing what the patch actually does. Also the adding of the commented nodes for the lime2 seems to be gone entirely from the set, instead now only a comment about the conflict with the hdmi pins is added, but it is above the i2c node instead of above a ps2 node. Regards, Hans On 12-12-14 19:25, VishnuPatekar wrote: > v2 --> v3 > 1. changed config to SERIO_SUN4I_PS2 from SERIO_SUNXI_PS2 > 2. changed driver name to sun4i-ps2 from sunxi-ps2. > 3. changed the function names to sun4i_ps2_*. > 4. added locking in sun4i_ps2_open. > 5. kept compatible "sun4i-a10-ps2" for A10 and A20, as A10 is earlier SOC. > 6. corrected the style errors. > 7. separated the dts patches. > 8. removed commented ps2 notes from lime2 dts. > 9. added note that ps2 pins confilt with hdmi. > 10. corrected the interrupt property for A10. > 11. moved dt-bindings to Documentation/devicetree/bindings/serio > > v1 --> 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. > 7. added compatible as allwinner,sun4i-a10-ps2. > 8. corrected the possible race condition. > > > VishnuPatekar (5): > sunxi:dts-bindings:input:ps2 bindings for A10/A20 ps2. > ARM:sunxi:drivers:input Add support for A10/A20 PS2 > ARM: sunxi: dts: Add PS2 nodes to dtsi for A10 and A20 > ARM: sunxi: dts: Add A10/A20 PS2 pin muxing options > ARM: sunxi: dts: Add note ps2 pins conflict with hdmi > > .../bindings/serio/allwinner,sun4i-ps2.txt | 23 ++ > arch/arm/boot/dts/sun4i-a10.dtsi | 31 ++ > arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts | 6 +- > arch/arm/boot/dts/sun7i-a20.dtsi | 32 ++ > drivers/input/serio/Kconfig | 11 + > drivers/input/serio/Makefile | 1 + > drivers/input/serio/sun4i-ps2.c | 362 ++++++++++++++++++++ > 7 files changed, 465 insertions(+), 1 deletion(-) > create mode 100644 Documentation/devicetree/bindings/serio/allwinner,sun4i-ps2.txt > create mode 100644 drivers/input/serio/sun4i-ps2.c > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/