From patchwork Thu May 24 20:37:58 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: William Breathitt Gray X-Patchwork-Id: 10425519 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5C3A160327 for ; Thu, 24 May 2018 20:38:27 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4823328DCC for ; Thu, 24 May 2018 20:38:27 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3BEA028E2A; Thu, 24 May 2018 20:38:27 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A214128DCC for ; Thu, 24 May 2018 20:38:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S971454AbeEXUiL (ORCPT ); Thu, 24 May 2018 16:38:11 -0400 Received: from mail-yb0-f193.google.com ([209.85.213.193]:37651 "EHLO mail-yb0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161286AbeEXUiG (ORCPT ); Thu, 24 May 2018 16:38:06 -0400 Received: by mail-yb0-f193.google.com with SMTP id i13-v6so1088844ybl.4; Thu, 24 May 2018 13:38:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=dEza728hgXZvF5AnHRfDYdQoTqONevJkIdqFkaVR42k=; b=O3b+2isCNTm0xGKdc8dI5ZIehPn8At6T+1t68fHEijezKcn8f4tSu2jjUsBskzkDKi t9PFsIeTlzbYi9Vr/okDi+ZIQYh0VgyRssoUTZR3xf7swn97PJ5kZba0ERdX+3/7cbQQ o0eyJnsD8bBurMyoT+AOv6MEw7Z1iJkvSiQxuqwh3V5srPGTsFUdg0trR60K7XvzsXfY 8ozBae8C9kkLXXKDafH47N0QBRl1IWOcHcYc/Q+AabvNpmsJy2gGg9Ms2D+IBETRV7LU 4Lai/nTslnRLmImMgclyBJtjsGRTvvbFiVWf+OdEXzp7xGHS8ObYaNOURzXymMIai/55 ml9A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=dEza728hgXZvF5AnHRfDYdQoTqONevJkIdqFkaVR42k=; b=DuNzbikYge6X8jfe/EtPvCsf5wuzoFJWsusH0Wvk7DhhCM2bnEpjR6WDFY2qg5lY+3 Q4a3acpY5FMIp6Y2pcE4SnCQBSdWimK+Mw720bM53gLSyos6QRkxzjEpZSV1fXOrbQHm Zz/0ez0jpayatQPEd2B/psfNsm5x9zxdQbRsPaxFK26MQV+AkNYumeDpvBikT6auCNc1 fFb4iHWwa2uxEvVO3zNqMdwMlfu7Szsgs/+B8GuQBwCD/9ob5ZbR5AXrGHqN5gEFZ48V XnIdbT6L0KkaalAhzH5Oc+3sSbLmGgy6XNphokPT7HoKMSUWIzjH1eKKN3XFhYtZ9Z4G SxKw== X-Gm-Message-State: ALKqPwermo63RKkRYSMwVeJfmhvuy8iQzdsblDJVo4yQcvtmaaQGy5ek aNPIB00vt7hWmwv3ZxRyAGE= X-Google-Smtp-Source: AB8JxZro1NWvW1/AxwEFgiVzdVxFnVc5U0B4GJ8BLk2dPGLkcK7n2KohHjvAGDcNilsTE8k/apb/cQ== X-Received: by 2002:a25:3485:: with SMTP id b127-v6mr5332347yba.275.1527194285504; Thu, 24 May 2018 13:38:05 -0700 (PDT) Received: from localhost ([72.188.97.40]) by smtp.gmail.com with ESMTPSA id k2-v6sm8066030ywi.18.2018.05.24.13.38.04 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 24 May 2018 13:38:05 -0700 (PDT) From: William Breathitt Gray To: jic23@kernel.org Cc: knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, William Breathitt Gray Subject: [PATCH 2/2] iio: 104-quad-8: Provide defines for magic numbers Date: Thu, 24 May 2018 16:37:58 -0400 Message-Id: <3b4452fd58136d15e4188f465657db5911a01dc6.1527193661.git.vilhelm.gray@gmail.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: References: Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This patch adds several register and bit defines to help improve the clarity of the code by cleaning up magic numbers throughout the driver source code. Signed-off-by: William Breathitt Gray --- drivers/iio/counter/104-quad-8.c | 86 ++++++++++++++++++++++---------- 1 file changed, 60 insertions(+), 26 deletions(-) diff --git a/drivers/iio/counter/104-quad-8.c b/drivers/iio/counter/104-quad-8.c index 4be85ec54af4..7d605d783051 100644 --- a/drivers/iio/counter/104-quad-8.c +++ b/drivers/iio/counter/104-quad-8.c @@ -59,6 +59,39 @@ struct quad8_iio { unsigned int base; }; +#define REG_CHANNEL_OPERATION 0x11 +#define REG_INDEX_INPUT_LEVELS 0x16 +/* Borrow Toggle flip-flop */ +#define FLAG_BT BIT(0) +/* Carry Toggle flip-flop */ +#define FLAG_CT BIT(1) +/* Error flag */ +#define FLAG_E BIT(4) +/* Up/Down flag */ +#define FLAG_UD BIT(5) +/* Reset and Load Signal Decoders */ +#define CTR_RLD 0x00 +/* Counter Mode Register */ +#define CTR_CMR 0x20 +/* Input / Output Control Register */ +#define CTR_IOR 0x40 +/* Index Control Register */ +#define CTR_IDR 0x60 +/* Reset Byte Pointer (three byte data pointer) */ +#define RLD_RESET_BP 0x01 +/* Reset Counter */ +#define RLD_RESET_CNTR 0x02 +/* Reset Borrow Toggle, Carry Toggle, Compare Toggle, and Sign flags */ +#define RLD_RESET_FLAGS 0x04 +/* Reset Error flag */ +#define RLD_RESET_E 0x06 +/* Preset Register to Counter */ +#define RLD_PRESET_CNTR 0x08 +/* Transfer Counter to Output Latch */ +#define RLD_CNTR_OUT 0x10 +#define CHAN_OP_ENABLE_COUNTERS 0x00 +#define CHAN_OP_RESET_COUNTERS 0x01 + static int quad8_read_raw(struct iio_dev *indio_dev, struct iio_chan_spec const *chan, int *val, int *val2, long mask) { @@ -72,19 +105,20 @@ static int quad8_read_raw(struct iio_dev *indio_dev, switch (mask) { case IIO_CHAN_INFO_RAW: if (chan->type == IIO_INDEX) { - *val = !!(inb(priv->base + 0x16) & BIT(chan->channel)); + *val = !!(inb(priv->base + REG_INDEX_INPUT_LEVELS) + & BIT(chan->channel)); return IIO_VAL_INT; } flags = inb(base_offset + 1); - borrow = flags & BIT(0); - carry = !!(flags & BIT(1)); + borrow = flags & FLAG_BT; + carry = !!(flags & FLAG_CT); /* Borrow XOR Carry effectively doubles count range */ *val = (borrow ^ carry) << 24; /* Reset Byte Pointer; transfer Counter to Output Latch */ - outb(0x11, base_offset + 1); + outb(CTR_RLD | RLD_RESET_BP | RLD_CNTR_OUT, base_offset + 1); for (i = 0; i < 3; i++) *val |= (unsigned int)inb(base_offset) << (8 * i); @@ -120,17 +154,17 @@ static int quad8_write_raw(struct iio_dev *indio_dev, return -EINVAL; /* Reset Byte Pointer */ - outb(0x01, base_offset + 1); + outb(CTR_RLD | RLD_RESET_BP, base_offset + 1); /* Counter can only be set via Preset Register */ for (i = 0; i < 3; i++) outb(val >> (8 * i), base_offset); /* Transfer Preset Register to Counter */ - outb(0x08, base_offset + 1); + outb(CTR_RLD | RLD_PRESET_CNTR, base_offset + 1); /* Reset Byte Pointer */ - outb(0x01, base_offset + 1); + outb(CTR_RLD | RLD_RESET_BP, base_offset + 1); /* Set Preset Register back to original value */ val = priv->preset[chan->channel]; @@ -138,9 +172,9 @@ static int quad8_write_raw(struct iio_dev *indio_dev, outb(val >> (8 * i), base_offset); /* Reset Borrow, Carry, Compare, and Sign flags */ - outb(0x04, base_offset + 1); + outb(CTR_RLD | RLD_RESET_FLAGS, base_offset + 1); /* Reset Error flag */ - outb(0x06, base_offset + 1); + outb(CTR_RLD | RLD_RESET_E, base_offset + 1); return 0; case IIO_CHAN_INFO_ENABLE: @@ -153,7 +187,7 @@ static int quad8_write_raw(struct iio_dev *indio_dev, ior_cfg = val | priv->preset_enable[chan->channel] << 1; /* Load I/O control configuration */ - outb(0x40 | ior_cfg, base_offset + 1); + outb(CTR_IOR | ior_cfg, base_offset + 1); return 0; case IIO_CHAN_INFO_SCALE: @@ -217,7 +251,7 @@ static ssize_t quad8_write_preset(struct iio_dev *indio_dev, uintptr_t private, priv->preset[chan->channel] = preset; /* Reset Byte Pointer */ - outb(0x01, base_offset + 1); + outb(CTR_RLD | RLD_RESET_BP, base_offset + 1); /* Set Preset Register */ for (i = 0; i < 3; i++) @@ -258,7 +292,7 @@ static ssize_t quad8_write_set_to_preset_on_index(struct iio_dev *indio_dev, (unsigned int)preset_enable << 1; /* Load I/O control configuration to Input / Output Control Register */ - outb(0x40 | ior_cfg, base_offset); + outb(CTR_IOR | ior_cfg, base_offset); return len; } @@ -274,7 +308,7 @@ static int quad8_get_noise_error(struct iio_dev *indio_dev, struct quad8_iio *const priv = iio_priv(indio_dev); const int base_offset = priv->base + 2 * chan->channel + 1; - return !!(inb(base_offset) & BIT(4)); + return !!(inb(base_offset) & FLAG_E); } static const struct iio_enum quad8_noise_error_enum = { @@ -294,7 +328,7 @@ static int quad8_get_count_direction(struct iio_dev *indio_dev, struct quad8_iio *const priv = iio_priv(indio_dev); const int base_offset = priv->base + 2 * chan->channel + 1; - return !!(inb(base_offset) & BIT(5)); + return !!(inb(base_offset) & FLAG_UD); } static const struct iio_enum quad8_count_direction_enum = { @@ -324,7 +358,7 @@ static int quad8_set_count_mode(struct iio_dev *indio_dev, mode_cfg |= (priv->quadrature_scale[chan->channel] + 1) << 3; /* Load mode configuration to Counter Mode Register */ - outb(0x20 | mode_cfg, base_offset); + outb(CTR_CMR | mode_cfg, base_offset); return 0; } @@ -364,7 +398,7 @@ static int quad8_set_synchronous_mode(struct iio_dev *indio_dev, priv->synchronous_mode[chan->channel] = synchronous_mode; /* Load Index Control configuration to Index Control Register */ - outb(0x60 | idr_cfg, base_offset); + outb(CTR_IDR | idr_cfg, base_offset); return 0; } @@ -410,7 +444,7 @@ static int quad8_set_quadrature_mode(struct iio_dev *indio_dev, priv->quadrature_mode[chan->channel] = quadrature_mode; /* Load mode configuration to Counter Mode Register */ - outb(0x20 | mode_cfg, base_offset); + outb(CTR_CMR | mode_cfg, base_offset); return 0; } @@ -446,7 +480,7 @@ static int quad8_set_index_polarity(struct iio_dev *indio_dev, priv->index_polarity[chan->channel] = index_polarity; /* Load Index Control configuration to Index Control Register */ - outb(0x60 | idr_cfg, base_offset); + outb(CTR_IDR | idr_cfg, base_offset); return 0; } @@ -556,28 +590,28 @@ static int quad8_probe(struct device *dev, unsigned int id) priv->base = base[id]; /* Reset all counters and disable interrupt function */ - outb(0x01, base[id] + 0x11); + outb(CHAN_OP_RESET_COUNTERS, base[id] + REG_CHANNEL_OPERATION); /* Set initial configuration for all counters */ for (i = 0; i < QUAD8_NUM_COUNTERS; i++) { base_offset = base[id] + 2 * i; /* Reset Byte Pointer */ - outb(0x01, base_offset + 1); + outb(CTR_RLD | RLD_RESET_BP, base_offset + 1); /* Reset Preset Register */ for (j = 0; j < 3; j++) outb(0x00, base_offset); /* Reset Borrow, Carry, Compare, and Sign flags */ - outb(0x04, base_offset + 1); + outb(CTR_RLD | RLD_RESET_FLAGS, base_offset + 1); /* Reset Error flag */ - outb(0x06, base_offset + 1); + outb(CTR_RLD | RLD_RESET_E, base_offset + 1); /* Binary encoding; Normal count; non-quadrature mode */ - outb(0x20, base_offset + 1); + outb(CTR_CMR, base_offset + 1); /* Disable A and B inputs; preset on index; FLG1 as Carry */ - outb(0x40, base_offset + 1); + outb(CTR_IOR, base_offset + 1); /* Disable index function; negative index polarity */ - outb(0x60, base_offset + 1); + outb(CTR_IDR, base_offset + 1); } /* Enable all counters */ - outb(0x00, base[id] + 0x11); + outb(CHAN_OP_ENABLE_COUNTERS, base[id] + REG_CHANNEL_OPERATION); return devm_iio_device_register(dev, indio_dev); }