Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932392Ab0BDOGa (ORCPT ); Thu, 4 Feb 2010 09:06:30 -0500 Received: from mx1.redhat.com ([209.132.183.28]:2750 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932222Ab0BDOG2 (ORCPT ); Thu, 4 Feb 2010 09:06:28 -0500 Message-ID: <4B6AD440.9060300@redhat.com> Date: Thu, 04 Feb 2010 12:05:52 -0200 From: Mauro Carvalho Chehab User-Agent: Thunderbird 2.0.0.22 (X11/20090609) MIME-Version: 1.0 To: Jiri Slaby CC: Jiri Kosina , Antti Palosaari , mchehab@infradead.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, Pekka Sarnila , linux-input@vger.kernel.org Subject: Re: [PATCH 1/1] media: dvb-usb/af9015, fix disconnection crashes References: <1264007972-6261-1-git-send-email-jslaby@suse.cz> <4B5CDB53.6030009@iki.fi> <4B5D6098.7010700@gmail.com> <4B5DDDFB.5020907@iki.fi> <4B6AA211.1060707@gmail.com> <4B6AB7E9.40607@redhat.com> <4B6AC333.6030308@gmail.com> <4B6ACEA3.3080900@redhat.com> <4B6AD0EE.9060801@gmail.com> In-Reply-To: <4B6AD0EE.9060801@gmail.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 999 Lines: 25 Jiri Slaby wrote: > On 02/04/2010 02:41 PM, Mauro Carvalho Chehab wrote: >> The point is that it is better to name the function right since the beginning. > > Sorry, I misunderstood you for the first time. It's .event member of > hid_driver. Hence I named it dvb_event (or now rc_event or whatever). > > The function may contain decisions on what to do with the event based > for example on quirks set up in .probe. And if the function grows later, > it may be factored out to rc_nokeyup_event. But rc_event is a root for > the decision tree and it should be there forever. Does it make sense now? Ah, ok. Due to the comments at the function, I misunderstood that you were planning to have separate functions for separate quirks. -- Cheers, Mauro -- 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/