Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sun, 16 Dec 2018 11:19:11 +0000
From:   Jonathan Cameron <jic23@kernel.org>
To:     Jeremy Fertic <jeremyfertic@gmail.com>
Cc:     Lars-Peter Clausen <lars@metafoo.de>,
        Michael Hennerich <Michael.Hennerich@analog.com>,
        Hartmut Knaack <knaack.h@gmx.de>,
        Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-iio@vger.kernel.org, devel@driverdev.osuosl.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 01/11] staging: iio: adt7316: fix register and bit
 definitions
Message-ID: <20181216111911.149d466b@archlinux>
In-Reply-To: <20181212005503.28054-2-jeremyfertic@gmail.com>
References: <20181212005503.28054-1-jeremyfertic@gmail.com>
        <20181212005503.28054-2-jeremyfertic@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, 11 Dec 2018 17:54:53 -0700
Jeremy Fertic <jeremyfertic@gmail.com> wrote:

> Change two register addresses and one bit definition to match the
> datasheet.
> 
> Signed-off-by: Jeremy Fertic <jeremyfertic@gmail.com>
One comment inline.  I added a fixes tag but also a note saying I would
not suggest backporting to stable.

There are too many things wrong with this driver for any backports
to really be worthwhile.  It didn't work at all for i2c until last cycle
for example!

Jonathan

> ---
>  drivers/staging/iio/addac/adt7316.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/addac/adt7316.c
> index dc93e85808e0..1fa4a4c2b4f3 100644
> --- a/drivers/staging/iio/addac/adt7316.c
> +++ b/drivers/staging/iio/addac/adt7316.c
> @@ -59,8 +59,8 @@
>  #define ADT7316_CONFIG1			0x18
>  #define ADT7316_CONFIG2			0x19
>  #define ADT7316_CONFIG3			0x1A
> -#define ADT7316_LDAC_CONFIG		0x1B
> -#define ADT7316_DAC_CONFIG		0x1C
> +#define ADT7316_DAC_CONFIG		0x1B
> +#define ADT7316_LDAC_CONFIG		0x1C
>  #define ADT7316_INT_MASK1		0x1D
>  #define ADT7316_INT_MASK2		0x1E
>  #define ADT7316_IN_TEMP_OFFSET		0x1F
> @@ -117,7 +117,7 @@
>   */
>  #define ADT7316_ADCLK_22_5		0x1
>  #define ADT7316_DA_HIGH_RESOLUTION	0x2
> -#define ADT7316_DA_EN_VIA_DAC_LDCA	0x4
> +#define ADT7316_DA_EN_VIA_DAC_LDCA	0x8
This looks to be called LDAC not LDCA?  

Different error so I don't mind that being fixed later!

Jonathan
>  #define ADT7516_AIN_IN_VREF		0x10
>  #define ADT7316_EN_IN_TEMP_PROP_DACA	0x20
>  #define ADT7316_EN_EX_TEMP_PROP_DACB	0x40