Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp1808634ybl; Sun, 18 Aug 2019 10:56:59 -0700 (PDT) X-Google-Smtp-Source: APXvYqzkH5l2a3gK54Yk/dcK6sHPKSowJmFW1XP69P6W7RhJlwqg6zNvcV4A4VXDb7e9m+vM42NX X-Received: by 2002:a17:902:848c:: with SMTP id c12mr16892354plo.47.1566151018949; Sun, 18 Aug 2019 10:56:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1566151018; cv=none; d=google.com; s=arc-20160816; b=xBScdbIPLiNNarZsNnLnCHP84WuHbuILocXIVWOFseDLAOSry3t9RpaOtII4U56ctV Mp1JvekCm0NlIxHsSzN/vxOq/C8cPaC+oxJbjy7djqtQbwIoL+BJ+OHbytvTJ6crv8Gj QIGjtOPqh9MzMTMXZ3GkOd4iCV7DTBfZxAP7O7Rm/clprMhfHQRItlWPH50s4wZnuCrm UthHe8VpAVkmm+1IEfH3MUHrk0HVCrrsf7HwcFbnFc/vDUysUTpeBbU9d5Gc6Y1e8vjk 21lMz9M6GOldckipdEQrI2GDuNYJ6p6QO9ZK81H2p/Xt6yzJjzaT0PyV64GdqxzdpR4j KXoA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=H5BRyFeJ4RTs6PR8gd96MQtVC+qfdj3M4avIBJekvV8=; b=K2PL6NTn8n2W4oiX//vy5FtlM54h644TFv9+/Phcn2uC6k8CSf4nHb9FW0B0uK58us gOr73kPATXgGlvYiSaPGDgz7FMS+Nzq7jKxFKYR12SYSeCJjLfBA1ku/vCrrb6vuHUhu m2duHl5iqm1v4KRephdBnT0TbFvh2H3npYPdUthJ0FLqGWQjFIUT06v9ihsCMkhiCLRb Hi2Whqr2Z+sB0H/Bcnsql1SmfaakEyW8xdldEG0fc5ooOXS2gRVDikIZidjw5wASkZoR vTg/Sdu1Ngg+NWLfBj4d06QTHm0lkVtrD3o49efgwJL5rJxFkUEvTmPXAFJjNuPAHxUD bXSA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=baXrZ81N; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s20si8153630pgi.499.2019.08.18.10.56.42; Sun, 18 Aug 2019 10:56:58 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=baXrZ81N; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726950AbfHRRzu (ORCPT + 99 others); Sun, 18 Aug 2019 13:55:50 -0400 Received: from mail.kernel.org ([198.145.29.99]:48682 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726747AbfHRRzu (ORCPT ); Sun, 18 Aug 2019 13:55:50 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4B6C220B7C; Sun, 18 Aug 2019 17:55:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1566150949; bh=81ed9RBM09C9vmEgmyh5QD5J7tn2TnHkzuCZiVnKQNw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=baXrZ81Nj/oC6AstnQfLU4ymhv9m478hZqX7mJHqFJ8Q1UzpdggNWlQaRpitHULuU riLpw5ycrtPNQnUjXpubYNkzquyVSISXhZAD66pgMg+Pt/vEt5sC/0eac7P256n+qc IA4+d5q4eNMTnVWWtIQ5TXzFIOmSjpCR44Uin3YU= Date: Sun, 18 Aug 2019 18:55:44 +0100 From: Jonathan Cameron To: "Ardelean, Alexandru" Cc: "Popa, Stefan Serban" , "linux-iio@vger.kernel.org" , "colin.king@canonical.com" , "Hennerich, Michael" , "lars@metafoo.de" , "knaack.h@gmx.de" , "pmeerw@pmeerw.net" , "kernel-janitors@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] iio: dac: ad5380: fix incorrect assignment to val Message-ID: <20190818185544.66c32add@archlinux> In-Reply-To: <876a4d5190828619f75365863cc6bf2cfea4ffea.camel@analog.com> References: <20190815115846.21800-1-colin.king@canonical.com> <876a4d5190828619f75365863cc6bf2cfea4ffea.camel@analog.com> X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 16 Aug 2019 06:16:26 +0000 "Ardelean, Alexandru" wrote: > On Thu, 2019-08-15 at 12:58 +0100, Colin King wrote: > > [External] > > > > Reviewed-by: Alexandru Ardelean Applied to the fixes-togreg branch of iio.git. Thanks, Jonathan > > > From: Colin Ian King > > > > Currently the pointer val is being incorrectly incremented > > instead of the value pointed to by val. Fix this by adding > > in the missing * indirection operator. > > > > Addresses-Coverity: ("Unused value") > > Fixes: c03f2c536818 ("staging:iio:dac: Add AD5380 driver") > > Signed-off-by: Colin Ian King > > --- > > drivers/iio/dac/ad5380.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c > > index 4335214800d2..2ebe08326048 100644 > > --- a/drivers/iio/dac/ad5380.c > > +++ b/drivers/iio/dac/ad5380.c > > @@ -220,7 +220,7 @@ static int ad5380_read_raw(struct iio_dev *indio_dev, > > if (ret) > > return ret; > > *val >>= chan->scan_type.shift; > > - val -= (1 << chan->scan_type.realbits) / 2; > > + *val -= (1 << chan->scan_type.realbits) / 2; > > return IIO_VAL_INT; > > case IIO_CHAN_INFO_SCALE: > > *val = 2 * st->vref;