From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Ni Subject: Re: [PATCH v4 2/3] hwmon: (lm90) add support to handle IRQ Date: Mon, 4 Nov 2013 18:05:59 +0800 Message-ID: <52777187.5010202@nvidia.com> References: <1375856306-14415-1-git-send-email-wni@nvidia.com> <1375856306-14415-3-git-send-email-wni@nvidia.com> <20131104103434.5a085e27@endymion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20131104103434.5a085e27-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jean Delvare Cc: "linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org" , "lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-tegra@vger.kernel.org On 11/04/2013 05:34 PM, Jean Delvare wrote: > Hi Wei, > > I'm hitting a problem with that patch now... > > On Wed, 7 Aug 2013 14:18:25 +0800, Wei Ni wrote: >> When the temperature exceed the limit range value, >> the driver can handle the interrupt. >> >> Signed-off-by: Wei Ni >> --- >> drivers/hwmon/lm90.c | 23 +++++++++++++++++++++++ >> 1 file changed, 23 insertions(+) >> >> diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c >> index 1da2eff..2e68773 100644 >> --- a/drivers/hwmon/lm90.c >> +++ b/drivers/hwmon/lm90.c >> @@ -89,6 +89,7 @@ >> #include >> #include >> #include >> +#include >> >> /* >> * Addresses to scan >> @@ -1433,6 +1434,16 @@ static bool lm90_is_tripped(struct i2c_client *client) >> return true; >> } >> >> +static irqreturn_t lm90_irq_thread(int irq, void *dev_id) >> +{ >> + struct i2c_client *client = dev_id; >> + >> + if (lm90_is_tripped(client)) > > Did you really send the latest version of the patch? lm90_is_tripped() > takes two arguments now, but here you only pass one. > > If you have a more up-to-date version of this patch, please send it > over quickly. Hi, Jean It's so sorry. As you know I resend the "[PATCH RESEND v4 1/3] hwmon: (lm90) Define status bits" last week, but didn't rebase this patch. I will rebase it and send it right now. Wei. Thanks. > >> + return IRQ_HANDLED; >> + else >> + return IRQ_NONE; >> +} >> + >> static int lm90_probe(struct i2c_client *client, >> const struct i2c_device_id *id) >> { >> @@ -1509,6 +1520,18 @@ static int lm90_probe(struct i2c_client *client, >> goto exit_remove_files; >> } >> >> + if (client->irq) { >> + dev_dbg(dev, "IRQ: %d\n", client->irq); >> + err = devm_request_threaded_irq(dev, client->irq, >> + NULL, lm90_irq_thread, >> + IRQF_TRIGGER_LOW | IRQF_ONESHOT, >> + "lm90", client); >> + if (err < 0) { >> + dev_err(dev, "cannot request IRQ: %d\n", client->irq); >> + goto exit_remove_files; >> + } >> + } >> + >> return 0; >> >> exit_remove_files: > >