From patchwork Thu Dec 19 15:05:04 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michal Simek X-Patchwork-Id: 303570 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 88A072C00A6 for ; Fri, 20 Dec 2013 02:06:50 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754492Ab3LSPFW (ORCPT ); Thu, 19 Dec 2013 10:05:22 -0500 Received: from mail-ee0-f47.google.com ([74.125.83.47]:54165 "EHLO mail-ee0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754481Ab3LSPFT (ORCPT ); Thu, 19 Dec 2013 10:05:19 -0500 Received: by mail-ee0-f47.google.com with SMTP id e51so446939eek.6 for ; Thu, 19 Dec 2013 07:05:18 -0800 (PST) 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 :content-type; bh=wNNcIHMgY4tTzPIs8l65XtD97XcbDtm+5zlvArLOLAY=; b=D7uK80GHqDVAT49PkovbAci9gJWLMNCrs+QZx2bo1zO8/6uugmkSvULwSMvz/VdNr1 YulJwdxrlnwAlkK5IWYUz5wwIvtyk6C286lmnq1+nfg+fpPt97aQkGbRFqFc0q/OcJTz QAz1XJKLywZ5n10kEF32k/tJDzNUBYGvgy4gEmqCHQv4FM8bpMYQ2DwzuBnOvXkuKKwg 2SUMDyg+siOEdVOWUHtu+G2AHougG19WRsvyQ6XfdLj0AEeM7ne0uzCklOsN2FhpmL0J Nd9eJQdFbM7VJ3JnWR/OEMAx0wQ4LEKfbb0MLNlEUBUUXR+EaJo/FCratzYgwI7FAL4Z CL3A== X-Gm-Message-State: ALoCoQk0LepqQQis53ZhBGrCaAnd9aiZeVlCrkkYC3TajitgoFIy0OE8mVWwLlpMhyqG//c0HBt8 X-Received: by 10.15.24.142 with SMTP id j14mr216749eeu.52.1387465518073; Thu, 19 Dec 2013 07:05:18 -0800 (PST) Received: from localhost (nat-63.starnet.cz. [178.255.168.63]) by mx.google.com with ESMTPSA id j46sm10063703eew.18.2013.12.19.07.05.16 for (version=TLSv1.1 cipher=RC4-SHA bits=128/128); Thu, 19 Dec 2013 07:05:16 -0800 (PST) From: Michal Simek To: linux-kernel@vger.kernel.org, monstr@monstr.eu, wsa@the-dreams.de, joe@perches.com Cc: Kedareswara rao Appana , Jingoo Han , Jean Delvare , Peter Korsgaard , linux-i2c@vger.kernel.org Subject: [PATCH v5 1/3] i2c: xilinx: Fix i2c checkpatch warnings Date: Thu, 19 Dec 2013 16:05:04 +0100 Message-Id: X-Mailer: git-send-email 1.8.2.3 Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org From: Kedareswara rao Appana Code changes to fix checkpatch warnings listed below. - WARNING: please, no space before tabs - WARNING: quoted string split across lines Signed-off-by: Kedareswara rao Appana Signed-off-by: Michal Simek --- Changes in v5: None Changes in v4: - Get back full dev_dbg message Changes in v3: None Changes in v2: - It is checkpatch warnings not sparse - Extend patch description with checkpatch warnings Tested by: ./scripts/checkpatch.pl -f drivers/i2c/busses/i2c-xiic.c --- drivers/i2c/busses/i2c-xiic.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) -- 1.8.2.3 diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c index fc2716a..5e5fae7 100644 --- a/drivers/i2c/busses/i2c-xiic.c +++ b/drivers/i2c/busses/i2c-xiic.c @@ -69,7 +69,7 @@ struct xiic_i2c { struct i2c_adapter adap; struct i2c_msg *tx_msg; spinlock_t lock; - unsigned int tx_pos; + unsigned int tx_pos; unsigned int nmsgs; enum xilinx_i2c_state state; struct i2c_msg *rx_msg; @@ -272,8 +272,8 @@ static void xiic_read_rx(struct xiic_i2c *i2c) bytes_in_fifo = xiic_getreg8(i2c, XIIC_RFO_REG_OFFSET) + 1; - dev_dbg(i2c->adap.dev.parent, "%s entry, bytes in fifo: %d, msg: %d" - ", SR: 0x%x, CR: 0x%x\n", + dev_dbg(i2c->adap.dev.parent, + "%s entry, bytes in fifo: %d, msg: %d, SR: 0x%x, CR: 0x%x\n", __func__, bytes_in_fifo, xiic_rx_space(i2c), xiic_getreg8(i2c, XIIC_SR_REG_OFFSET), xiic_getreg8(i2c, XIIC_CR_REG_OFFSET)); @@ -340,9 +340,10 @@ static void xiic_process(struct xiic_i2c *i2c) ier = xiic_getreg32(i2c, XIIC_IIER_OFFSET); pend = isr & ier; - dev_dbg(i2c->adap.dev.parent, "%s entry, IER: 0x%x, ISR: 0x%x, " - "pend: 0x%x, SR: 0x%x, msg: %p, nmsgs: %d\n", - __func__, ier, isr, pend, xiic_getreg8(i2c, XIIC_SR_REG_OFFSET), + dev_dbg(i2c->adap.dev.parent, "%s: IER: 0x%x, ISR: 0x%x, pend: 0x%x\n", + __func__, ier, isr, pend); + dev_dbg(i2c->adap.dev.parent, "%s: SR: 0x%x, msg: %p, nmsgs: %d\n", + __func__, xiic_getreg8(i2c, XIIC_SR_REG_OFFSET), i2c->tx_msg, i2c->nmsgs); /* Do not processes a devices interrupts if the device has no @@ -542,9 +543,10 @@ static void xiic_start_send(struct xiic_i2c *i2c) xiic_irq_clr(i2c, XIIC_INTR_TX_ERROR_MASK); - dev_dbg(i2c->adap.dev.parent, "%s entry, msg: %p, len: %d, " - "ISR: 0x%x, CR: 0x%x\n", - __func__, msg, msg->len, xiic_getreg32(i2c, XIIC_IISR_OFFSET), + dev_dbg(i2c->adap.dev.parent, "%s entry, msg: %p, len: %d", + __func__, msg, msg->len); + dev_dbg(i2c->adap.dev.parent, "%s entry, ISR: 0x%x, CR: 0x%x\n", + __func__, xiic_getreg32(i2c, XIIC_IISR_OFFSET), xiic_getreg8(i2c, XIIC_CR_REG_OFFSET)); if (!(msg->flags & I2C_M_NOSTART)) {