From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759029AbdENPPm (ORCPT ); Sun, 14 May 2017 11:15:42 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:46808 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758573AbdENPPk (ORCPT ); Sun, 14 May 2017 11:15:40 -0400 Subject: Re: [PATCH v2 3/4] iio: accel: adxl345: Setup DATA_READY trigger To: Andy Shevchenko , linux-iio@vger.kernel.org, Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , Dmitry Torokhov , Michael Hennerich , Daniel Baluta , Alison Schofield , Florian Vaussard , "linux-kernel@vger.kernel.org" References: <20170502121554.GD3030@Socrates-UM> <20170510132448.GB5456@Socrates-UM> From: Jonathan Cameron Message-ID: <683cefe7-77f8-1671-85b0-286122b72c9d@kernel.org> Date: Sun, 14 May 2017 16:15:37 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <20170510132448.GB5456@Socrates-UM> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GH Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/05/17 14:24, Eva Rachel Retuya wrote: > On Wed, May 03, 2017 at 12:05:00AM +0300, Andy Shevchenko wrote: >> On Tue, May 2, 2017 at 3:15 PM, Eva Rachel Retuya wrote: >>> On Mon, May 01, 2017 at 02:31:00PM +0300, Andy Shevchenko wrote: >>> [...] >>>>> -int adxl345_core_probe(struct device *dev, struct regmap *regmap, >>>>> +int adxl345_core_probe(struct device *dev, struct regmap *regmap, int irq, >>>>> const char *name); >>>> >>>> I think I commented this once. Instead of increasing parameters, >>>> please introduce a new struct (as separate preparatory patch) which >>>> will hold current parameters. Let's call it >>>> strut adxl345_chip { >>>> struct device *dev; >>>> struct regmap *regmap; >>>> const char *name; >>>> }; >>>> >>>> I insisnt in this chage. >>> >>> I'm not sure if what you want is more simpler, is it something like what >>> this driver does? >> >> Nope. The driver you were referring to does the same you did. >> >> I'm proposing the above struct to be introduced along with changing >> prototype like: >> >> -int adxl345_core_probe(struct device *dev, struct regmap *regmap, >> const char *name); >> +int adxl345_core_probe(struct adxl345_chip *chip); >> >> In next patch adding interrupt would not touch prototypes at all! >> > > OK, got it. Thanks for clarifying. > >>> >>> http://lxr.free-electrons.com/source/drivers/iio/gyro/mpu3050.h#L41 >>> http://lxr.free-electrons.com/source/drivers/iio/gyro/mpu3050-i2c.c#L34 >> >>>>> +#include >>>> >>>> Can we get rid of gnostic resource providers? >>>> >>> >>> I'm uninformed and still learning. Please let me know how to approach >>> this in some other way. >> >> I suppose something like platform_get_irq(); to use. >> But it would be nice to you to investigate more. > > I had a look and it seems I have to convert to platform_driver in order > to make use of that function. Is this correct? I believe Andy was suggesting a function 'similar to' that one rather than actually using platform_get_irq. It's not an area I know all that much about either, but there are moves to try and move the boiler plat needed to get the same parameters from devicetree and acpi into core library code so that a single function can be called to get the parameter in either case. Jonathan > > Eva > >> >> -- >> With Best Regards, >> Andy Shevchenko >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >