From patchwork Mon May 30 01:09:01 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Corey Minyard X-Patchwork-Id: 68815 Delivered-To: patch@linaro.org Received: by 10.140.92.199 with SMTP id b65csp1159607qge; Sun, 29 May 2016 18:09:57 -0700 (PDT) X-Received: by 10.66.185.68 with SMTP id fa4mr42176745pac.41.1464570596558; Sun, 29 May 2016 18:09:56 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h192si32779846pfc.70.2016.05.29.18.09.56; Sun, 29 May 2016 18:09:56 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-i2c-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com; spf=pass (google.com: best guess record for domain of linux-i2c-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-i2c-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753399AbcE3BJc (ORCPT + 1 other); Sun, 29 May 2016 21:09:32 -0400 Received: from mail-pa0-f66.google.com ([209.85.220.66]:34990 "EHLO mail-pa0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753390AbcE3BJR (ORCPT ); Sun, 29 May 2016 21:09:17 -0400 Received: by mail-pa0-f66.google.com with SMTP id gp3so12876935pac.2; Sun, 29 May 2016 18:09:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=lrOJdqNOMSuNmpk0NIUE1o6mofG7H1ULtSDsgYYjmj0=; b=hc7ptIRa2FMgQJHKLNXL4IMAcyIB9bTDl2FuwpiCq5DvoA29EJV994Khj58fF36d2W ijUJzgK9QGchLm+O6tpKdgw3U4kioGl5JRe0O+1rOXfO0D3Ytu5ZMUL3ADWxjCk5VpGK nFe1iFsJFbQbmWoWNW9WPNf6lbiUCyxhKEV3602HpzZFo002fF+BiqH4jc71BUvxIclW mXbv3YRHgssaEMaEj+ojSvmS3u+z2r4OU/i9FlqKxK7nRcFmtehx5N/Up4uAzqjJpZcp Rm5Id7lZQ4Y/7udX6tp3F2XisgJSd3b7bFqKtmzT6URZEpAsizscsBO89i5K+dOHLVwk p8tQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=lrOJdqNOMSuNmpk0NIUE1o6mofG7H1ULtSDsgYYjmj0=; b=Ek/qVEY/vMQW+UMvwob+SuF//m0u6gvPGodWpxZFp0q2BCk+JG2N6tWj0+X+3UjLoU ymRYyzkLTzw2p0d5cCFJHRs+CDnmtIFxKfAI/b3sRWSfmpfVLAk7qamI/LVUu0Td4J5Z dpndw6EspdEMAJBA1zQKc9c5j6wYBWytJhgPh77ZnorWJzPaSGPVMWmBkrN7O09939Yq KQhXIGTsOhsIMtK8ZM3TLo2LyPIAkhYy2bG/hP5UGAYhzrJau6/ZIxmZwQYD1fUndDi7 jcAbdhL84qjodZC8sRTt7c5kQXp94kgaj8Aa76JXzYxILzwfFhRw9pgw97rXWpjXZ6O2 axIg== X-Gm-Message-State: ALyK8tKPGs7VbsXKLoE+aynolGlGz+8qhbTEMgkwJN1J/po7PxiXEDG4FXzi+ZSbRaVU3w== X-Received: by 10.66.63.104 with SMTP id f8mr42283808pas.109.1464570556884; Sun, 29 May 2016 18:09:16 -0700 (PDT) Received: from serve.minyard.net ([108.19.215.157]) by smtp.gmail.com with ESMTPSA id ry2sm42769070pac.9.2016.05.29.18.09.15 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 29 May 2016 18:09:15 -0700 (PDT) Received: from t430.minyard.net (t430m.minyard.net [192.168.27.3]) by serve.minyard.net (Postfix) with ESMTPA id 825979D8; Sun, 29 May 2016 20:09:12 -0500 (CDT) Received: by t430.minyard.net (Postfix, from userid 1000) id 3A78F300520; Sun, 29 May 2016 20:09:08 -0500 (CDT) From: minyard@acm.org To: Jean Delvare , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, minyard@acm.org Cc: Corey Minyard Subject: [PATCH v2 07/10] i2c-i801: Fix some inconsistent variable names Date: Sun, 29 May 2016 20:09:01 -0500 Message-Id: <1464570544-975-8-git-send-email-minyard@acm.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1464570544-975-1-git-send-email-minyard@acm.org> References: <1464570544-975-1-git-send-email-minyard@acm.org> Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org From: Corey Minyard The priv->cmd is called subcmd elsewhere, and that's a more appropriate name for it, so rename it. The "size" parameter passed in to i801_access is passed to other functions and those name is "command". This is confusing with the "command" parameter passed in to i801_access. Rename it to "size" everywhere. Signed-off-by: Corey Minyard --- drivers/i2c/busses/i2c-i801.c | 29 ++++++++++++++--------------- 1 file changed, 14 insertions(+), 15 deletions(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c index ae1e60a..b415948 100644 --- a/drivers/i2c/busses/i2c-i801.c +++ b/drivers/i2c/busses/i2c-i801.c @@ -233,7 +233,7 @@ struct i801_priv { u8 status; /* Command state used by isr for byte-by-byte block transactions */ - u8 cmd; + u8 subcmd; bool is_read; int count; int len; @@ -468,7 +468,7 @@ static void i801_isr_byte_done(struct i801_priv *priv) { if (priv->is_read) { /* For SMBus block reads, length is received with first byte */ - if (((priv->cmd & 0x1c) == I801_BLOCK_DATA) && + if (((priv->subcmd & 0x1c) == I801_BLOCK_DATA) && (priv->count == 0)) { priv->len = inb_p(SMBHSTDAT0(priv)); if (priv->len < 1 || priv->len > I2C_SMBUS_BLOCK_MAX) { @@ -494,7 +494,7 @@ static void i801_isr_byte_done(struct i801_priv *priv) /* Set LAST_BYTE for last byte of read transaction */ if (priv->count == priv->len - 1) - outb_p(priv->cmd | SMBHSTCNT_LAST_BYTE, + outb_p(priv->subcmd | SMBHSTCNT_LAST_BYTE, SMBHSTCNT(priv)); } else if (priv->count < priv->len - 1) { /* Write next byte, except for IRQ after last byte */ @@ -555,7 +555,7 @@ static irqreturn_t i801_isr(int irq, void *dev_id) */ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, union i2c_smbus_data *data, - bool is_read, int command) + bool is_read, int size) { int i, len; int smbcmd; @@ -570,7 +570,7 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, outb_p(data->block[1], SMBBLKDAT(priv)); } - if (command == I2C_SMBUS_I2C_BLOCK_DATA && is_read) + if (size == I2C_SMBUS_I2C_BLOCK_DATA && is_read) smbcmd = I801_I2C_BLOCK_DATA; else smbcmd = I801_BLOCK_DATA; @@ -579,12 +579,12 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, priv->is_read = is_read; if (len == 1 && is_read) smbcmd |= SMBHSTCNT_LAST_BYTE; - priv->cmd = smbcmd | SMBHSTCNT_INTREN; + priv->subcmd = smbcmd | SMBHSTCNT_INTREN; priv->len = len; priv->count = 0; priv->data = &data->block[1]; - outb_p(priv->cmd | SMBHSTCNT_START, SMBHSTCNT(priv)); + outb_p(priv->subcmd | SMBHSTCNT_START, SMBHSTCNT(priv)); result = wait_event_timeout(priv->waitq, (status = priv->status), adap->timeout); @@ -610,8 +610,7 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, if (status) return status; - if (i == 1 && is_read - && command != I2C_SMBUS_I2C_BLOCK_DATA) { + if (i == 1 && is_read && size != I2C_SMBUS_I2C_BLOCK_DATA) { priv->len = inb_p(SMBHSTDAT0(priv)); if (priv->len < 1 || priv->len > I2C_SMBUS_BLOCK_MAX) return -EPROTO; @@ -642,13 +641,13 @@ static int i801_set_block_buffer_mode(struct i801_priv *priv) /* Block transaction function */ static int i801_block_transaction(struct i801_priv *priv, unsigned short flags, union i2c_smbus_data *data, bool is_read, - int command) + int size) { int result = 0; int hwpec = (priv->features & FEATURE_SMBUS_PEC) && (flags & I2C_CLIENT_PEC) - && command != I2C_SMBUS_QUICK - && command != I2C_SMBUS_I2C_BLOCK_DATA; + && size != I2C_SMBUS_QUICK + && size != I2C_SMBUS_I2C_BLOCK_DATA; if (hwpec) /* enable/disable hardware PEC */ outb_p(inb_p(SMBAUXCTL(priv)) | SMBAUXCTL_CRC, @@ -657,7 +656,7 @@ static int i801_block_transaction(struct i801_priv *priv, unsigned short flags, outb_p(inb_p(SMBAUXCTL(priv)) & (~SMBAUXCTL_CRC), SMBAUXCTL(priv)); - if (!is_read || command == I2C_SMBUS_I2C_BLOCK_DATA) { + if (!is_read || size == I2C_SMBUS_I2C_BLOCK_DATA) { if (data->block[0] < 1) data->block[0] = 1; if (data->block[0] > I2C_SMBUS_BLOCK_MAX) @@ -670,13 +669,13 @@ static int i801_block_transaction(struct i801_priv *priv, unsigned short flags, SMBus (not I2C) block transactions, even though the datasheet doesn't mention this limitation. */ if ((priv->features & FEATURE_BLOCK_BUFFER) - && command != I2C_SMBUS_I2C_BLOCK_DATA + && size != I2C_SMBUS_I2C_BLOCK_DATA && i801_set_block_buffer_mode(priv) == 0) result = i801_block_transaction_by_block(priv, data, is_read, hwpec); else result = i801_block_transaction_byte_by_byte(priv, data, - is_read, command); + is_read, size); /* * Some BIOSes don't like it when PEC is enabled at reboot or