Received: by 2002:a25:86ce:0:0:0:0:0 with SMTP id y14csp401246ybm; Wed, 22 May 2019 05:16:34 -0700 (PDT) X-Google-Smtp-Source: APXvYqzHcdpuW+EssIcHS7EzSM+FWUROHfjJGyIZqvKbLsPhWeryVmQ2OxFTssloGOgyix5g8OjJ X-Received: by 2002:a17:902:8a4:: with SMTP id 33mr62455635pll.19.1558527394304; Wed, 22 May 2019 05:16:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558527394; cv=none; d=google.com; s=arc-20160816; b=y5wObw7pWRSlmOk888Zh+0C9Hswbo8+3KHqG6bEKVWh7HD/7VEMR/CQOKxW+KiP346 fGEUwZPMek1+JbR0rOHlLV/APJKQNgzId5wjWldN91le+W/hyzxWy9oFkwZRo+r9iZVU wDbdsVOQ+ObDE/8sAiSUCU0QzCHwnDs02XzqQHqqlxT9h3+m4zeo12Q5N/gTmQ2tym6n K2GxxiM8iJGeI/EngrgLILCcpc9CLoAL6fO1hUZ6OwemQgZW1dRBTXLbhqZNyKRd7cYi UGtHO3Khb+Rt+D/fkc4cK5MAO+YJ+HoMs3jbA6y6kntuzzGBMboU2kHD5+gPDiyeXZL9 qCLA== 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:date:subject:cc:to:from :dkim-signature; bh=nrrFf5jPyFl/413hkwocQVvbRFpG3X2ac3ZXAgGL0fk=; b=ftpN8/sbZz0CyYvGBKoG0zr5AKJlMVLy4C4AH9N+8IpDfK8QiyG/rwAzKqg1XCiiYP zqUCQK7bjZb9aoLwc+99y2JBYuh4207rL6cFBzi4ZUZWHn8Msxy/CvRnkc+7CD1dMTrU T4iu3P7qx+f0B0ONGNzLXEyb7snBxADKwwYZ5QBN5p9AfmLRRlL65Peh1hH20kTAsCQr kdTbGCl/2tYpeuBEkbnhWuFGiLMx03Oq909BQig51k4ELhEbbLQJVyRW5AlvQEoPFkkg VxfdBAwpaw6NDsGVD6vIKLcWtl4gwrX3mDrb0aKXB+zzm9er3wb0l+hykArSVm/sAz3a 9n8w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=BHplIj21; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id i23si7724162pgh.26.2019.05.22.05.16.17; Wed, 22 May 2019 05:16:34 -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=@gmail.com header.s=20161025 header.b=BHplIj21; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729379AbfEVMO4 (ORCPT + 99 others); Wed, 22 May 2019 08:14:56 -0400 Received: from mail-qk1-f194.google.com ([209.85.222.194]:41854 "EHLO mail-qk1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729355AbfEVMOz (ORCPT ); Wed, 22 May 2019 08:14:55 -0400 Received: by mail-qk1-f194.google.com with SMTP id m18so1265470qki.8 for ; Wed, 22 May 2019 05:14:54 -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 :mime-version:content-transfer-encoding; bh=nrrFf5jPyFl/413hkwocQVvbRFpG3X2ac3ZXAgGL0fk=; b=BHplIj21ouH1CPEeBdRJET6uPmvNfutmVv3Q/qYh61PSD48ja48H0btCCwbHETZ4ln PgIS/XJf/9f/8o7l1QSJN+JLK8gfkH9zf/RiNnl3bpMKUJ9++Ng0V1J+OzDqjehbp6Sv SZ7vKRYQfh8QXsXr8ZxxR2Yr4Oupnkz2UeqvsiinISRSAtcldtcFaGGH8KOL5aevo3CF G9cLWfn8u10UVRbywhVNLSSLnGlTnWDuCsSfQZ26ocvcYZTP7CA1gP7AYhmUuN6GxHzY CDkdCIG7IA9Gk5KwIjpYu7SVDFKFFKYxCMtUe6ROgscIghzI6pmWjhhlDTp50Degh+6O O4sg== 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:mime-version:content-transfer-encoding; bh=nrrFf5jPyFl/413hkwocQVvbRFpG3X2ac3ZXAgGL0fk=; b=dGSilaoshnsx0x+JwUUwhz/nU9l1N0998sQ6wskJJSxxPS3CECVAENO8KHHSkLXWaD dNJCkGIpc1dh1f4QRgUJ3REu1lqMED5CLvjvLBiziiz4iAxxY7NbDOp9TTTz1dnUK23C 7vRRhTbzcqApB6Iq2nuHocinkSMwpxke0TyJuBfSqwaSa/FLErc280b+WtCUuM56iLOc NHXGk4FtAXINdt7wA2jQgsfcp1y3njvD+IRdL7Xk3vtSc6SPlh38e0Xx0Kdla70n3pn3 7P2mLSOmdpKklP84AOhXICy+Y7uynFfInGWUeMXChc+djx+EcSEtxz5t+m5FT/Vj2ZJp 8YMA== X-Gm-Message-State: APjAAAUV5jcGMmPF4eSjc7HaAkIuwXOkThRMPYdh9fU+i7EXdsXrUztR yBln3ra3EKrhRnvzzDyihKs= X-Received: by 2002:a05:620a:1641:: with SMTP id c1mr68784078qko.103.1558527293814; Wed, 22 May 2019 05:14:53 -0700 (PDT) Received: from arch-01.home (c-73-132-202-198.hsd1.dc.comcast.net. [73.132.202.198]) by smtp.gmail.com with ESMTPSA id w2sm8742070qto.19.2019.05.22.05.14.52 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 22 May 2019 05:14:53 -0700 (PDT) From: Geordan Neukum To: Greg Kroah-Hartman , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Cc: Matt Sickler , Dan Carpenter , Geordan Neukum Subject: [PATCH 5/6] staging: kpc2000: kpc_i2c: Remove unnecessary function tracing prints Date: Wed, 22 May 2019 12:14:01 +0000 Message-Id: <47d7da2db4c4cbb40476fb80383184525919c65b.1558526487.git.gneukum1@gmail.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Many of the functions in kpc_i2c log debug-level messages to the kernel log message buffer upon invocation. This is unnecessary, as debugging tools like kgdb, kdb, etc. or the tracing tool ftrace should be able to provide this same information. Therefore, remove these print statements. Signed-off-by: Geordan Neukum --- drivers/staging/kpc2000/kpc2000_i2c.c | 26 -------------------------- 1 file changed, 26 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_i2c.c b/drivers/staging/kpc2000/kpc2000_i2c.c index 5d98ed54c05c..f9259c06b605 100644 --- a/drivers/staging/kpc2000/kpc2000_i2c.c +++ b/drivers/staging/kpc2000/kpc2000_i2c.c @@ -139,8 +139,6 @@ static int i801_check_pre(struct i2c_device *priv) { int status; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - status = inb_p(SMBHSTSTS(priv)); if (status & SMBHSTSTS_HOST_BUSY) { dev_err(&priv->adapter.dev, "SMBus is busy, can't use it! (status=%x)\n", status); @@ -165,8 +163,6 @@ static int i801_check_post(struct i2c_device *priv, int status, int timeout) { int result = 0; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - /* If the SMBus is still busy, we give up */ if (timeout) { dev_err(&priv->adapter.dev, "Transaction timeout\n"); @@ -214,8 +210,6 @@ static int i801_transaction(struct i2c_device *priv, int xact) int result; int timeout = 0; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - result = i801_check_pre(priv); if (result < 0) return result; @@ -244,8 +238,6 @@ static void i801_wait_hwpec(struct i2c_device *priv) int timeout = 0; int status; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - do { usleep_range(250, 500); status = inb_p(SMBHSTSTS(priv)); @@ -262,8 +254,6 @@ static int i801_block_transaction_by_block(struct i2c_device *priv, union i2c_sm int i, len; int status; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - inb_p(SMBHSTCNT(priv)); /* reset the data buffer index */ /* Use 32-byte buffer to process this transaction */ @@ -298,8 +288,6 @@ static int i801_block_transaction_byte_by_byte(struct i2c_device *priv, union i2 int result; int timeout; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - result = i801_check_pre(priv); if (result < 0) return result; @@ -364,8 +352,6 @@ static int i801_block_transaction_byte_by_byte(struct i2c_device *priv, union i2 static int i801_set_block_buffer_mode(struct i2c_device *priv) { - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - outb_p(inb_p(SMBAUXCTL(priv)) | SMBAUXCTL_E32B, SMBAUXCTL(priv)); if ((inb_p(SMBAUXCTL(priv)) & SMBAUXCTL_E32B) == 0) return -EIO; @@ -378,8 +364,6 @@ static int i801_block_transaction(struct i2c_device *priv, union i2c_smbus_data int result = 0; //unsigned char hostc; - dev_dbg(&priv->adapter.dev, "%s\n", __func__); - if (command == I2C_SMBUS_I2C_BLOCK_DATA) { if (read_write == I2C_SMBUS_WRITE) { /* set I2C_EN bit in configuration register */ @@ -427,10 +411,6 @@ static s32 i801_access(struct i2c_adapter *adap, u16 addr, unsigned short flags, int ret, xact = 0; struct i2c_device *priv = i2c_get_adapdata(adap); - dev_dbg(&priv->adapter.dev, - "%s (addr=%0d) flags=%x read_write=%x command=%x size=%x", - __func__, addr, flags, read_write, command, size); - hwpec = (priv->features & FEATURE_SMBUS_PEC) && (flags & I2C_CLIENT_PEC) && size != I2C_SMBUS_QUICK && size != I2C_SMBUS_I2C_BLOCK_DATA; switch (size) { @@ -605,9 +585,6 @@ int pi2c_probe(struct platform_device *pldev) struct i2c_device *priv; struct resource *res; - dev_dbg(&pldev->dev, "%s(pldev = %p '%s')\n", __func__, pldev, - pldev->name); - priv = devm_kzalloc(&pldev->dev, sizeof(*priv), GFP_KERNEL); if (!priv) return -ENOMEM; @@ -653,9 +630,6 @@ int pi2c_remove(struct platform_device *pldev) { struct i2c_device *lddev; - dev_dbg(&pldev->dev, "%s(pldev = %p '%s')\n", __func__, pldev, - pldev->name); - lddev = (struct i2c_device *)pldev->dev.platform_data; i2c_del_adapter(&lddev->adapter); -- 2.21.0