From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christopher Heiny Subject: Re: [PATCH 00/15] Input: synaptics-rmi4 - cleanup and add DT support Date: Fri, 24 Jan 2014 15:24:27 -0800 Message-ID: <52E2F62B.80101@synaptics.com> References: <1390521623-6491-1-git-send-email-courtney.cavin@sonymobile.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from us-mx2.synaptics.com ([192.147.44.131]:55148 "EHLO us-mx2.synaptics.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750826AbaAXXY2 (ORCPT ); Fri, 24 Jan 2014 18:24:28 -0500 In-Reply-To: <1390521623-6491-1-git-send-email-courtney.cavin@sonymobile.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Courtney Cavin , linux-input@vger.kernel.org Cc: dmitry.torokhov@gmail.com On 01/23/2014 04:00 PM, Courtney Cavin wrote: > This is an attempt to get this driver closer to being in an upstream-able form. > In the process, this drops legacy callback methods for managing power and GPIO > configuration, in favor of the already existing frameworks. Most of this > series is cleanup, but there are a few intermixed bug fixes to make it all > work. Hi Courtney! Thanks for the extensive patches. There's a great deal of cool stuff in there that we're really happy to see (as in "buy Courtney a beer! No, make it two beers!" levels of happy). We really want to get the driver upstream, and your improvements will help a lot. The device tree work looks to be particularly helpful. Unfortunately some of your cleanup is too aggressive, and winds up breaking code outside of the current patch submissions. The current patch codebase is significantly reduced in size in order to be more manageable, but there's a non-trivial amount of production code outside that codebase that depends on things like the current GPIO/IRQ handling, the debugfs infrastructure, platform suspend/resume, and so on. This code will be submitted after the core of the driver is upstreamed. Tearing out the supporting code now and then putting it all back later just makes more work for all concerned. Additionally, some of your changes are redundant to, or in conflict with, patches submitted over the past few weeks. Have you been following that series and discussions relating to them? It would probably be more efficient for you to offer comments on those patches, rather than submitting conflicting work. We'll be going over the new feature and bug fix parts of this patch series, and will send comments in the next couple of workdays. Thanks, Chris > > This patch series is based off of the synaptics-rmi4 branch merged into > Linus' 3.13. A tree is available at [1]. > > This was tested on Synaptics TM2281-001 & TM2282-001. > > [1] http://github.com/courtc/linux.git > tag for-input/synaptics-rmi4 > > Courtney Cavin (15): > Input: synaptics-rmi4 - fix checkpatch.pl, sparse and GCC warnings > Input: synaptics-rmi4 - don't kfree devm_ alloced memory > Input: synaptics-rmi4 - don't free devices directly > Input: synaptics-rmi4 - remove sensor name from platform data > Input: synaptics-rmi4 - remove gpio handling and polling > Input: synaptics-rmi4 - remove platform suspend callbacks > Input: synaptics-rmi4 - remove remaining debugfs code > Input: synaptics-rmi4 - cleanup platform data > Input: synaptics-rmi4 - remove unused defines and variables > Input: synaptics-rmi4 - add devicetree support > Input: synaptics-rmi4 - add regulator support > Input: synaptics-rmi4 - don't immediately set page on probe > Input: synaptics-rmi4 - properly set F01 container on PDT scan > Input: synaptics-rmi4 - ensure we have IRQs before reading status > Input: synaptics-rmi4 - correct RMI4 spec url > > Documentation/devicetree/bindings/input/rmi4.txt | 117 +++++ > .../devicetree/bindings/vendor-prefixes.txt | 1 + > drivers/input/rmi4/Kconfig | 1 - > drivers/input/rmi4/rmi_bus.c | 131 +----- > drivers/input/rmi4/rmi_bus.h | 18 +- > drivers/input/rmi4/rmi_driver.c | 321 +++++-------- > drivers/input/rmi4/rmi_driver.h | 33 +- > drivers/input/rmi4/rmi_f01.c | 163 ++++--- > drivers/input/rmi4/rmi_f11.c | 523 ++++++--------------- > drivers/input/rmi4/rmi_i2c.c | 55 +-- > include/linux/rmi.h | 219 ++------- > 11 files changed, 551 insertions(+), 1031 deletions(-) > create mode 100644 Documentation/devicetree/bindings/input/rmi4.txt > -- Christopher Heiny Senior Staff Firmware Engineer Synaptics Incorporated