From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753400AbbHNG4H (ORCPT ); Fri, 14 Aug 2015 02:56:07 -0400 Received: from mail-pa0-f53.google.com ([209.85.220.53]:36062 "EHLO mail-pa0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752012AbbHNG4F (ORCPT ); Fri, 14 Aug 2015 02:56:05 -0400 Date: Fri, 14 Aug 2015 12:25:42 +0530 From: Sudip Mukherjee To: Dmitry Torokhov , pali.rohar@gmail.com Cc: linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, gregkh@linuxfoundation.org Subject: Re: [PATCH v2] Input: drivers/joystick: use parallel port device model Message-ID: <20150814065541.GA25515@sudip-pc> References: <1438698351-24604-1-git-send-email-sudipm.mukherjee@gmail.com> <20150808103515.GC1301@sudip-pc> <20150813134614.GA29298@sudip-pc> <20150813162602.GA15693@dtor-ws> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150813162602.GA15693@dtor-ws> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 13, 2015 at 09:26:02AM -0700, Dmitry Torokhov wrote: > On Thu, Aug 13, 2015 at 07:16:14PM +0530, Sudip Mukherjee wrote: > > On Sat, Aug 08, 2015 at 04:05:15PM +0530, Sudip Mukherjee wrote: > > > On Tue, Aug 04, 2015 at 07:55:51PM +0530, Sudip Mukherjee wrote: > > > > Modify db9 driver to use the new Parallel Port device model. > > > > > > > > Signed-off-by: Sudip Mukherjee > > > > --- > > > > > > > > v2: an extra check was removed. db9_base could not be removed. Already > > > > mailed you the reasons, reference is at: > > > > https://lkml.org/lkml/2015/8/3/403 > > > > > > Hi Dmitry, > > > I was telling you on the other thread that to remove db9_base we will > > > need to modify the parport code and then it has to be tested by everyone > > > again and that will also change the old behaviour of parport. The v2 to > > > which I am replying now is still having that db9_base. > > > > > > Please see the following patch, this is the minimum change required to > > > update db9 driver. Personally I donot like this approach as in this > > > method if a parallel port is hotplugged after this driver is loaded then > > > it will not work. Please let me know your views on the posted v2 and > > > also on the following proposal. > > > > Hi Dmitry, > > Did you have a chance to look at this? > > Right, so I agree that we should allow for hotplug and so the v2 is in > that sense is better than v3. > > I still would prefer for the core to keep track of relationship between > devices and drivers, but if you would prefer to postpone implementing it > that woudl be fine. Thanks. Right now the code is already a mixture of the new code and the old code and its almost a mess. After all drivers are converted and the old code removed then the parport code will become much simpler than what it is now. And it will be easy to make those enhancements at that time. > > > Pali Rohár has the hardware and he is ready to test it. > > Pali, could you give the v2 version of the patch a spin > (https://patchwork.kernel.org/patch/6940321/)? Adding Pali to the cc list. Hi Pali, Are you still away from your desktop with the joystick? or can it be tested? regards sudip