From patchwork Sat Mar 22 13:01:02 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Frank Schaefer X-Patchwork-Id: 23181 X-Patchwork-Delegate: hverkuil@xs4all.nl Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from ) id 1WRLXc-0007Ry-Kj; Sat, 22 Mar 2014 14:00:48 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.72/mailfrontend-5) with esmtp id 1WRLXa-0003cm-6o; Sat, 22 Mar 2014 14:00:48 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750974AbaCVNAl (ORCPT + 1 other); Sat, 22 Mar 2014 09:00:41 -0400 Received: from mail-ee0-f47.google.com ([74.125.83.47]:52170 "EHLO mail-ee0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750782AbaCVNAg (ORCPT ); Sat, 22 Mar 2014 09:00:36 -0400 Received: by mail-ee0-f47.google.com with SMTP id b15so2732285eek.34 for ; Sat, 22 Mar 2014 06:00:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=hSz670wQ0r+IyaskOcG+WqERlGlEyc9Nhw9p1OsJzsY=; b=qWEjf12dkPSA9W/aqnBYEDx9jMx+e23B+pIKoE7UjEoroEVoE/rll0kAEqnDhPDVmP zEWyyldLArPGV4ZeSFWVCgeWJ+xicbHbx+AqZnVmoZDI9f8pwDbmvsorC2VIpzovobmC SA56YpUFln9YU2nydFc7xmhEBmpJBMN3q1jMY3VJDb6dmLuuirDHWKw7zakSxCVeSyXR 4dsqCLdrKcmuOqOUJVtECr+t+QbwGHGfcyVvgJj/0uH/i/pQfhlmDj+YlLFMzZ4VAFUs OP+xw4V9KnBlHtnsS3uWvpv6qhrEh0uX8ydxMWeSYv1/V/p47/lFuZRLXl6IdQ2cevDg Vm/Q== X-Received: by 10.15.60.199 with SMTP id g47mr53714972eex.37.1395493235453; Sat, 22 Mar 2014 06:00:35 -0700 (PDT) Received: from Athlon64X2-5000.site (ip-81-210-162-48.unitymediagroup.de. [81.210.162.48]) by mx.google.com with ESMTPSA id h47sm10902164eey.13.2014.03.22.06.00.34 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 22 Mar 2014 06:00:35 -0700 (PDT) From: =?UTF-8?q?Frank=20Sch=C3=A4fer?= To: m.chehab@samsung.com Cc: linux-media@vger.kernel.org, =?UTF-8?q?Frank=20Sch=C3=A4fer?= Subject: [PATCH 4/5] em28xx: remove function em28xx_compression_disable() and its call Date: Sat, 22 Mar 2014 14:01:02 +0100 Message-Id: <1395493263-2158-4-git-send-email-fschaefer.oss@googlemail.com> X-Mailer: git-send-email 1.8.4.5 In-Reply-To: <1395493263-2158-1-git-send-email-fschaefer.oss@googlemail.com> References: <1395493263-2158-1-git-send-email-fschaefer.oss@googlemail.com> MIME-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2014.3.22.124814 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1800_1899 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, CT_TEXT_PLAIN_UTF8_CAPS 0, DKIM_SIGNATURE 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __CT 0, __CTE 0, __CT_TEXT_PLAIN 0, __FRAUD_BODY_WEBMAIL 0, __FRAUD_WEBMAIL 0, __FRAUD_WEBMAIL_FROM 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __MULTIPLE_RCPTS_CC_X2 0, __PHISH_SPEAR_STRUCTURE_1 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __SXL_SIG_ERROR_SERVFAIL , __SXL_URI_ERROR_SERVFAIL , __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' em28xx_compression_disable() is a single line function which is called only one time and this call also isn't needed. Register 0x26 is always configured as part of the scaler configuration, which in turn is always done when the resolution changes. And the initial resolution setting is applied at first device open. Signed-off-by: Frank Schäfer --- drivers/media/usb/em28xx/em28xx-video.c | 1 - drivers/media/usb/em28xx/em28xx.h | 6 ------ 2 files changed, 7 deletions(-) diff --git a/drivers/media/usb/em28xx/em28xx-video.c b/drivers/media/usb/em28xx/em28xx-video.c index 0856e5d..e15ac75 100644 --- a/drivers/media/usb/em28xx/em28xx-video.c +++ b/drivers/media/usb/em28xx/em28xx-video.c @@ -2376,7 +2376,6 @@ static int em28xx_v4l2_init(struct em28xx *dev) (EM28XX_XCLK_AUDIO_UNMUTE | val)); em28xx_set_outfmt(dev); - em28xx_compression_disable(dev); /* Add image controls */ /* NOTE: at this point, the subdevices are already registered, so bridge diff --git a/drivers/media/usb/em28xx/em28xx.h b/drivers/media/usb/em28xx/em28xx.h index e95f4eb..dd6190c 100644 --- a/drivers/media/usb/em28xx/em28xx.h +++ b/drivers/media/usb/em28xx/em28xx.h @@ -793,12 +793,6 @@ int em28xx_init_camera(struct em28xx *dev); printk(KERN_WARNING "%s: "fmt,\ dev->name , ##arg); } while (0) -static inline int em28xx_compression_disable(struct em28xx *dev) -{ - /* side effect of disabling scaler and mixer */ - return em28xx_write_reg(dev, EM28XX_R26_COMPR, 0x00); -} - /*FIXME: maxw should be dependent of alt mode */ static inline unsigned int norm_maxw(struct em28xx *dev) {