From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christopher Heiny Subject: Re: [PATCH 01/15] Input: synaptics-rmi4 - fix checkpatch.pl, sparse and GCC warnings Date: Thu, 13 Feb 2014 11:25:33 -0800 Message-ID: <52FD1C2D.8050901@synaptics.com> References: <1390521623-6491-1-git-send-email-courtney.cavin@sonymobile.com> <20140213063611.GB15260@core.coreip.homeip.net> <52FD1559.2040101@synaptics.com> <2278620.abrl2TfnmQ@dtor-d630.eng.vmware.com> <20140213191216.GA3916@core.coreip.homeip.net> 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]:58245 "EHLO us-mx2.synaptics.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751008AbaBMTZe (ORCPT ); Thu, 13 Feb 2014 14:25:34 -0500 In-Reply-To: <20140213191216.GA3916@core.coreip.homeip.net> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Dmitry Torokhov Cc: Courtney Cavin , linux-input@vger.kernel.org On 02/13/2014 11:12 AM, Dmitry Torokhov wrote: > On Thu, Feb 13, 2014 at 11:10:29AM -0800, Dmitry Torokhov wrote: >> On Thursday, February 13, 2014 10:56:25 AM Christopher Heiny wrote: >>> On 02/12/2014 10:36 PM, Dmitry Torokhov wrote: >>>> On Wed, Feb 05, 2014 at 05:36:09PM -0800, Christopher Heiny wrote: >>>>>> On 02/05/2014 05:09 PM, Dmitry Torokhov wrote: >>>>>>>> On Tue, Feb 04, 2014 at 03:08:12PM -0800, Christopher Heiny wrote: >>>>>>>>>>>> On 01/23/2014 04:00 PM, Courtney Cavin wrote: >>>>>>>>>>>>>>>> Cc: Christopher Heiny >>>>>>>>>>>>>>>> Cc: Dmitry Torokhov >>>>>>>>>>>>>>>> Signed-off-by: Courtney Cavin >>>>>>>>>>>>>>>> --- >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> drivers/input/rmi4/rmi_bus.c | 4 ++-- >>>>>>>>>>>>>>>> drivers/input/rmi4/rmi_bus.h | 2 +- >>>>>>>>>>>>>>>> drivers/input/rmi4/rmi_driver.c | 17 ++++++++++++----- >>>>>>>>>>>>>>>> drivers/input/rmi4/rmi_f11.c | 4 +++- >>>>>>>>>>>>>>>> 4 files changed, 18 insertions(+), 9 deletions(-) >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> diff --git a/drivers/input/rmi4/rmi_bus.c >>>>>>>>>>>>>>>> b/drivers/input/rmi4/rmi_bus.c >>>>>>>>>>>>>>>> index 96a76e7..8a939f3 100644 >>>>>>>>>>>>>>>> --- a/drivers/input/rmi4/rmi_bus.c >>>>>>>>>>>>>>>> +++ b/drivers/input/rmi4/rmi_bus.c >>>>>>>>>>>>>>>> @@ -37,7 +37,7 @@ static void rmi_release_device(struct >>>>>>>>>>>>>>>> device *dev) >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> kfree(rmi_dev); >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> } >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -struct device_type rmi_device_type = { >>>>>>>>>>>>>>>> +static struct device_type rmi_device_type = { >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> .name = "rmi_sensor", >>>>>>>>>>>>>>>> .release = rmi_release_device, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> }; >>>>>>>>>>>> >>>>>>>>>>>> This struct is used by diagnostic modules to identify sensor >>>>>>>>>>>> devices, so it cannot be static. >>>>>>>> >>>>>>>> Then we need to declare it somewhere or provide an accessor function. >>>>>> >>>>>> Currently it's in a header not included in the patches. We'll move >>>>>> it to rmi_bus.h. >>>> >>>> Hmm, we do have rmi_is_physical_device() to identify whether it is a >>>> sensor or a function, so I believe we should mark all structures static >>>> to avoid anyone poking at them. >>> >>> I was poking around in the dependent code late last night and came to >>> the same conclusion. I'll send a micropatch later today to get it out >>> of the way. >> >> No need, I untangled relevant bits from the one Courtney sent. Thanks! >> >> Thanks. > > Here it is by the way. > > Input: synaptics-rmi4 - make device types module-private > > From: Courtney Cavin > > Nobody outside of RMI bus core should be accessing rmi_device_type, > rmi_function_type or rmi_physical_driver structures so let's limit their > scope. > > We provide rmi_is_physical_device() and rmi_is_function_device() to allow > determining whether one is dealing with sensor or function device. > > Signed-off-by: Courtney Cavin > Signed-off-by: Dmitry Torokhov Signed-off-by: Christopher Heiny > --- > drivers/input/rmi4/rmi_bus.c | 4 ++-- > drivers/input/rmi4/rmi_driver.c | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/input/rmi4/rmi_bus.c b/drivers/input/rmi4/rmi_bus.c > index 7efe7ed..6e0454a 100644 > --- a/drivers/input/rmi4/rmi_bus.c > +++ b/drivers/input/rmi4/rmi_bus.c > @@ -38,7 +38,7 @@ static void rmi_release_device(struct device *dev) > kfree(rmi_dev); > } > > -struct device_type rmi_device_type = { > +static struct device_type rmi_device_type = { > .name = "rmi_sensor", > .release = rmi_release_device, > }; > @@ -154,7 +154,7 @@ static void rmi_release_function(struct device *dev) > kfree(fn); > } > > -struct device_type rmi_function_type = { > +static struct device_type rmi_function_type = { > .name = "rmi_function", > .release = rmi_release_function, > }; > diff --git a/drivers/input/rmi4/rmi_driver.c b/drivers/input/rmi4/rmi_driver.c > index 788343a..4406a7f 100644 > --- a/drivers/input/rmi4/rmi_driver.c > +++ b/drivers/input/rmi4/rmi_driver.c > @@ -946,7 +946,7 @@ err_free_mem: > return retval < 0 ? retval : 0; > } > > -struct rmi_driver rmi_physical_driver = { > +static struct rmi_driver rmi_physical_driver = { > .driver = { > .owner = THIS_MODULE, > .name = "rmi_physical", > -- Christopher Heiny Senior Staff Firmware Engineer Synaptics Incorporated