Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3215576pxj; Mon, 24 May 2021 01:10:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzx0gqX++p5TmpXvx6Q/R6QX8HeGMcwGLBXQOO6OJZfSlPoEDgAPWDBnEbqFWgHVAfhWteg X-Received: by 2002:a17:906:1f83:: with SMTP id t3mr4577278ejr.112.1621843852088; Mon, 24 May 2021 01:10:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621843852; cv=none; d=google.com; s=arc-20160816; b=Z9KO6FJ3x5oEiY8+pjzehciuXISy2o9jns8Q6YwIFLWJlTEAepVaSn/fjIR5f3m+oF racFy6iuOwPyOurEZPsA44xlCC4h4P8ZSFtJ8WGPh5VfOqiDz7JZLsEts2MHK6QLY7ob 8sZAq8AyoaU/wR7wvNv+ANVV5A36Seb59bantrKOxhN37dXMI9oxVTiNXTifDagGE3ls 7Kwj6wmCr5GHw8LbzpPX4C2j2Hr7eqWPANbxD1H0xArGdhfSEL+9yw6681O7Ir3Rbeat I+feIvmjAdKpDOTynJaC4dZ/Y3df3pUC4kvar9u1a5xk20qD8qtX4XGhNc70zLN7Eo5M jdPw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=mnD4LZr5vh+XK9rOEnNIl7niFIb137sw/yseGObE7Qo=; b=QSnbjxtnT1cZWBKJ/FCV978AXYVPlHxgAxOyqdNEiz2nhoNLFwclpPcTRJUG1rt1c/ B9CvyyYG52Bdoe4GC/sw8yvpJUylUhMedJjHHeNMAzjx6B3ftyE+zlWHLnMMGTgEXNTm lYQtKLx+V2e7Su8e5V11Z/mPvmcxRUjkw1JUq8/vUdCSSPgrEhBhhPbXGDMxemrYqI9x JCPoDgLjadwxU2+/0WAvjRA0WnE63wk1iBsneGjqMqNJ6FIxvnTcza4l1qGgQbvHOd3g rtLaL/TuL6nAI8SIRZX20x2mKnKAV3YU9LMrj/+r8JX3CtZbRaD7YguHVNMEFlhOzZMH 344g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=HOQOAvWt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v9si8276283ejh.703.2021.05.24.01.10.29; Mon, 24 May 2021 01:10:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=HOQOAvWt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232279AbhEXILH (ORCPT + 99 others); Mon, 24 May 2021 04:11:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59194 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232266AbhEXILH (ORCPT ); Mon, 24 May 2021 04:11:07 -0400 Received: from mail-pj1-x1032.google.com (mail-pj1-x1032.google.com [IPv6:2607:f8b0:4864:20::1032]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 781B6C061574; Mon, 24 May 2021 01:09:33 -0700 (PDT) Received: by mail-pj1-x1032.google.com with SMTP id ml1-20020a17090b3601b029015f9b1ebce0so1485124pjb.5; Mon, 24 May 2021 01:09:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=mnD4LZr5vh+XK9rOEnNIl7niFIb137sw/yseGObE7Qo=; b=HOQOAvWtX5lC19tOGny/nwjg4Fbx5t7ovxswRTbeG/BljGJt23qiDkrBtUrrm0ej0R njjEm+CWtj7BARm1yVyr4Lf6eYYaLeb9f1oVVERu9TFdSB9VfGq9PoqqUoVDYaHvNJde OUavGU1jOowQA0gD/JneY8A9msxwytL1XhNU7JLyeCAMK3icf7wtsNhDWxGJM75weVIi n0r10MV1LGI7BSU5UhJs3M7K5yluUHrWwgRvk5gwqhzg+Ubqsq8nJM8ybdCJN0TZLEvy Q4LUGESNxq+rIs8AJq71lvFQvXBRmm99Z4OPUj1Y8s9ZQBY6kt859NLc8d+rHc/8Iwnt KWLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=mnD4LZr5vh+XK9rOEnNIl7niFIb137sw/yseGObE7Qo=; b=Z3dN26eqC1FvDQoUiOWBYx1m+svBqcx8QdPPy/dkX7MYRqlrmMAGz98mOvgCvg31a6 9LKUNR72MwQBXzJxxt6a88ZASCsz9Dmilxr7tTGKSp2139kz7mdcokRHmc49q3wKx6PE 3b5Q3W8pAV334EEyd6V7PuZTMF6PlwvlynklAMdxveXi51NYJryzguQAtkqb97MjCWhc v8n1kmo7buQh/9Bu3tfE3Bo4r2+pxcivyg2pRAtNEmOj679Pnl7f8rHw6apJNHc1a+B0 0H5zzZiFsL+OOoBAHNlC1C12WB0Ul5pNWeVdS3g94br8Q0UQyuOD0EaXKsDu5j9ITnyn /aqA== X-Gm-Message-State: AOAM531YMX+ZPykDuJ4E3G/5B7OWS1TNiKD1hcgapxfTFr2l+9dG37zV 9lZ8bR+jazY0vAJnjCMuPAMq6m2ffXeERTe06jg= X-Received: by 2002:a17:902:e00e:b029:ef:5f1c:18a8 with SMTP id o14-20020a170902e00eb02900ef5f1c18a8mr24599956plo.38.1621843772829; Mon, 24 May 2021 01:09:32 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Alexandru Ardelean Date: Mon, 24 May 2021 11:09:21 +0300 Message-ID: Subject: Re: [PATCH v2 3/3] staging: iio: cdc: ad7746: extract capac setup to own function To: Lucas Stankus Cc: Lars-Peter Clausen , "Hennerich, Michael" , Jonathan Cameron , Greg Kroah-Hartman , linux-iio , linux-staging@lists.linux.dev, LKML Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, May 23, 2021 at 8:13 PM Lucas Stankus wrote: > > Refactor the capac register write logic to own function. Minor typo 'capac' -> 'capdac' Other than that: Reviewed-by: Alexandru Ardelean > > Also fixes the following checkpatch warning: > CHECK: Alignment should match open parenthesis > > Signed-off-by: Lucas Stankus > --- > drivers/staging/iio/cdc/ad7746.c | 36 ++++++++++++++++---------------- > 1 file changed, 18 insertions(+), 18 deletions(-) > > diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c > index 367a5990ae35..4221312f0a32 100644 > --- a/drivers/staging/iio/cdc/ad7746.c > +++ b/drivers/staging/iio/cdc/ad7746.c > @@ -209,6 +209,19 @@ static const unsigned char ad7746_cap_filter_rate_table[][2] = { > {16, 62 + 1}, {13, 77 + 1}, {11, 92 + 1}, {9, 110 + 1}, > }; > > +static int ad7746_set_capdac(struct ad7746_chip_info *chip, int channel) > +{ > + int ret = i2c_smbus_write_byte_data(chip->client, > + AD7746_REG_CAPDACA, > + chip->capdac[channel][0]); > + if (ret < 0) > + return ret; > + > + return i2c_smbus_write_byte_data(chip->client, > + AD7746_REG_CAPDACB, > + chip->capdac[channel][1]); > +} > + > static int ad7746_select_channel(struct iio_dev *indio_dev, > struct iio_chan_spec const *chan) > { > @@ -224,17 +237,11 @@ static int ad7746_select_channel(struct iio_dev *indio_dev, > AD7746_CONF_CAPFS_SHIFT; > delay = ad7746_cap_filter_rate_table[idx][1]; > > + ret = ad7746_set_capdac(chip, chan->channel); > + if (ret < 0) > + return ret; > + > if (chip->capdac_set != chan->channel) { > - ret = i2c_smbus_write_byte_data(chip->client, > - AD7746_REG_CAPDACA, > - chip->capdac[chan->channel][0]); > - if (ret < 0) > - return ret; > - ret = i2c_smbus_write_byte_data(chip->client, > - AD7746_REG_CAPDACB, > - chip->capdac[chan->channel][1]); > - if (ret < 0) > - return ret; > > chip->capdac_set = chan->channel; > } > @@ -478,14 +485,7 @@ static int ad7746_write_raw(struct iio_dev *indio_dev, > chip->capdac[chan->channel][chan->differential] = val > 0 ? > AD7746_CAPDAC_DACP(val) | AD7746_CAPDAC_DACEN : 0; > > - ret = i2c_smbus_write_byte_data(chip->client, > - AD7746_REG_CAPDACA, > - chip->capdac[chan->channel][0]); > - if (ret < 0) > - goto out; > - ret = i2c_smbus_write_byte_data(chip->client, > - AD7746_REG_CAPDACB, > - chip->capdac[chan->channel][1]); > + ret = ad7746_set_capdac(chip, chan->channel); > if (ret < 0) > goto out; > > -- > 2.31.1 >