From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753659AbcLJRvL (ORCPT ); Sat, 10 Dec 2016 12:51:11 -0500 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:48672 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752418AbcLJRvK (ORCPT ); Sat, 10 Dec 2016 12:51:10 -0500 Subject: Re: [PATCH] Staging: iio: impedance-analyzer: ad5933: fix wrong comments To: Nizam Haider , pmeerw@pmeerw.net References: <20161207174447.24128-1-nizamhaider786@gmail.com> Cc: lars@metafoo.de, Michael.Hennerich@analog.com, knaack.h@gmx.de, gregkh@linuxfoundation.org, linux-iio@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Nizam Haider From: Jonathan Cameron Message-ID: <1eda9a06-063e-e865-9032-2883b4d26584@kernel.org> Date: Sat, 10 Dec 2016 17:51:07 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161207174447.24128-1-nizamhaider786@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/12/16 17:44, Nizam Haider wrote: > according to datasheet complete control register is of 2 bytes. > http://www.analog.com/media/en/technical-documentation/data-sheets/AD5933.pdf > > Signed-off-by: Nizam Haider Applied to the togreg branch of iio.git - initially pushed out as testing so the autobuilders can completely ignore this patch ;) Jonathan > --- > drivers/staging/iio/impedance-analyzer/ad5933.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance-analyzer/ad5933.c > index 3892a74..f39e03a 100644 > --- a/drivers/staging/iio/impedance-analyzer/ad5933.c > +++ b/drivers/staging/iio/impedance-analyzer/ad5933.c > @@ -23,8 +23,8 @@ > #include > > /* AD5933/AD5934 Registers */ > -#define AD5933_REG_CONTROL_HB 0x80 /* R/W, 2 bytes */ > -#define AD5933_REG_CONTROL_LB 0x81 /* R/W, 2 bytes */ > +#define AD5933_REG_CONTROL_HB 0x80 /* R/W, 1 byte */ > +#define AD5933_REG_CONTROL_LB 0x81 /* R/W, 1 byte */ > #define AD5933_REG_FREQ_START 0x82 /* R/W, 3 bytes */ > #define AD5933_REG_FREQ_INC 0x85 /* R/W, 3 bytes */ > #define AD5933_REG_INC_NUM 0x88 /* R/W, 2 bytes, 9 bit */ >