From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:45586 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751270AbbHBR7l (ORCPT ); Sun, 2 Aug 2015 13:59:41 -0400 Subject: Re: [PATCH v2 1/8] iio:adc:berlin2-adc: Fix register definition To: Antoine Tenart , Hartmut Knaack References: <20150730124706.GA16416@kwain> Cc: linux-iio@vger.kernel.org, Lars-Peter Clausen , Peter Meerwald From: Jonathan Cameron Message-ID: <55BE5A8A.30900@kernel.org> Date: Sun, 2 Aug 2015 18:59:38 +0100 MIME-Version: 1.0 In-Reply-To: <20150730124706.GA16416@kwain> Content-Type: text/plain; charset=windows-1252 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 30/07/15 13:47, Antoine Tenart wrote: > On Tue, Jul 28, 2015 at 12:38:57AM +0200, Hartmut Knaack wrote: >> Active channel number is stored in BERLIN2_SM_CTRL as value, instead of a >> bit map. >> The masks for channel interrupts and data ready are a 16 bits wide bit >> map each, instead of just 4 bits. >> >> Also correct the data mask for the temperature sensor, which was >> Reported-by: Antoine Tenart >> >> Signed-off-by: Hartmut Knaack > > Acked-by: Antoine Tenart Applied to the fixes-togreg branch of iio.git Thanks, Jonathan > >> --- >> drivers/iio/adc/berlin2-adc.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/iio/adc/berlin2-adc.c b/drivers/iio/adc/berlin2-adc.c >> index aecc9ad995ad..26fa6e6cb624 100644 >> --- a/drivers/iio/adc/berlin2-adc.c >> +++ b/drivers/iio/adc/berlin2-adc.c >> @@ -26,7 +26,7 @@ >> #define BERLIN2_SM_CTRL 0x14 >> #define BERLIN2_SM_CTRL_SM_SOC_INT BIT(1) >> #define BERLIN2_SM_CTRL_SOC_SM_INT BIT(2) >> -#define BERLIN2_SM_CTRL_ADC_SEL(x) (BIT(x) << 5) /* 0-15 */ >> +#define BERLIN2_SM_CTRL_ADC_SEL(x) ((x) << 5) /* 0-15 */ >> #define BERLIN2_SM_CTRL_ADC_SEL_MASK (0xf << 5) >> #define BERLIN2_SM_CTRL_ADC_POWER BIT(9) >> #define BERLIN2_SM_CTRL_ADC_CLKSEL_DIV2 (0x0 << 10) >> @@ -53,14 +53,14 @@ >> #define BERLIN2_SM_ADC_MASK 0x3ff >> #define BERLIN2_SM_ADC_STATUS 0x1c >> #define BERLIN2_SM_ADC_STATUS_DATA_RDY(x) BIT(x) /* 0-15 */ >> -#define BERLIN2_SM_ADC_STATUS_DATA_RDY_MASK 0xf >> +#define BERLIN2_SM_ADC_STATUS_DATA_RDY_MASK GENMASK(15, 0) >> #define BERLIN2_SM_ADC_STATUS_INT_EN(x) (BIT(x) << 16) /* 0-15 */ >> -#define BERLIN2_SM_ADC_STATUS_INT_EN_MASK (0xf << 16) >> +#define BERLIN2_SM_ADC_STATUS_INT_EN_MASK GENMASK(31, 16) >> #define BERLIN2_SM_TSEN_STATUS 0x24 >> #define BERLIN2_SM_TSEN_STATUS_DATA_RDY BIT(0) >> #define BERLIN2_SM_TSEN_STATUS_INT_EN BIT(1) >> #define BERLIN2_SM_TSEN_DATA 0x28 >> -#define BERLIN2_SM_TSEN_MASK 0xfff >> +#define BERLIN2_SM_TSEN_MASK GENMASK(9, 0) >> #define BERLIN2_SM_TSEN_CTRL 0x74 >> #define BERLIN2_SM_TSEN_CTRL_START BIT(8) >> #define BERLIN2_SM_TSEN_CTRL_SETTLING_4 (0x0 << 21) /* 4 us */ >> -- >> 2.4.6 >> >