From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:41532 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750803AbbJKOD2 (ORCPT ); Sun, 11 Oct 2015 10:03:28 -0400 Subject: Re: [PATCH] iio: mxs-lradc: Fix temperature offset To: Marek Vasut , Alexandre Belloni References: <1444141347-15301-1-git-send-email-alexandre.belloni@free-electrons.com> <561455AC.60504@denx.de> Cc: Lars-Peter Clausen , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org From: Jonathan Cameron Message-ID: <561A6C2E.4080800@kernel.org> Date: Sun, 11 Oct 2015 15:03:26 +0100 MIME-Version: 1.0 In-Reply-To: <561455AC.60504@denx.de> Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 07/10/15 00:13, Marek Vasut wrote: > On 10/06/2015 16:22, Alexandre Belloni wrote: >> 0° Kelvin is actually −273.15°C, not -272.15°C. Fix the temperature offset. >> >> Reported-by: Janusz Użycki >> Signed-off-by: Alexandre Belloni > > Nice find :-) > > Acked-by: Marek Vasut Ah, missed this when applying. Have added your Ack to the V2 and will push it out later. J > > (I am using different MUA, please pardon the possible issues) > >> --- >> drivers/staging/iio/adc/mxs-lradc.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c >> index 3f7715c9968b..67bfa54c4fb1 100644 >> --- a/drivers/staging/iio/adc/mxs-lradc.c >> +++ b/drivers/staging/iio/adc/mxs-lradc.c >> @@ -916,10 +916,10 @@ static int mxs_lradc_read_raw(struct iio_dev *iio_dev, >> if (chan->type == IIO_TEMP) { >> /* The calculated value from the ADC is in Kelvin, we >> * want Celsius for hwmon so the offset is >> - * -272.15 * scale >> + * -273.15 * scale >> */ >> - *val = -1075; >> - *val2 = 691699; >> + *val = -1079; >> + *val2 = 644268; >> >> return IIO_VAL_INT_PLUS_MICRO; >> } >> > > -- > 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