From patchwork Sun Sep 18 14:02:15 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Baoyou Xie X-Patchwork-Id: 76474 Delivered-To: patch@linaro.org Received: by 10.140.106.72 with SMTP id d66csp538642qgf; Sun, 18 Sep 2016 07:03:43 -0700 (PDT) X-Received: by 10.98.56.207 with SMTP id f198mr38833084pfa.83.1474207423487; Sun, 18 Sep 2016 07:03:43 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d63si20201078pfc.34.2016.09.18.07.03.43; Sun, 18 Sep 2016 07:03:43 -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=@linaro.org; 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 dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935460AbcIRODk (ORCPT + 27 others); Sun, 18 Sep 2016 10:03:40 -0400 Received: from mail-pf0-f178.google.com ([209.85.192.178]:33368 "EHLO mail-pf0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757020AbcIRODa (ORCPT ); Sun, 18 Sep 2016 10:03:30 -0400 Received: by mail-pf0-f178.google.com with SMTP id 21so31274953pfy.0 for ; Sun, 18 Sep 2016 07:03:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=qGINNA9trQCGMMHZ/ojEHR4RLQgMZui7eBY3PAT7c2U=; b=E2+uOG4+szrmhj6ahjSbyxa+Geo30XHKB7MnHgk/SWHbDBM5PR5tQGXcK2ZY7RzI1W 61O92VNrDwFUatLXIknDN+3ev4hL0JyHxN40M0mACRPITB/4vEboEkrSj+nWp1PsAl6B FkFD/2/PA7lqudHnJgFNnnsl0zMUFAZdTTgSU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=qGINNA9trQCGMMHZ/ojEHR4RLQgMZui7eBY3PAT7c2U=; b=GP7kTwORB9a/VvVL10nEl2t4z1Hm2aYNZfTh7fCuF1HaCZ5khvHibP0s5yJB2NXDyc PsiwBp5jrt6iBJV0qskUDd/cpVrYjQ7kY75IjSuvCgptXICcT5wNRFEH+C8KOzSiyvM/ E7etVVTzi8NezZd6X/8qBFHHWzRgR4B+k0dHFb3ZdP31gPZrSQNydDvoIh4MbQYeCjNU PdosxKnlHZ/E9gUCZLDSTP2UNTxsSz1fxomp2aTxl4/6y4EPTr11aIW474Vqjjw4EqkX 1+EdjuvDNrzaV3bxluuNZAdJYf8K/OwuwFReHSqfhmmURcwu6DLMV8x2PxPAwSJwINL+ sm0A== X-Gm-Message-State: AE9vXwOlm2DHFa46Aa+LdgHYeFo5US8vRrX7PZjRmtibJ8qlPHFH7aBpzr8tpfnhU5VTFGjc X-Received: by 10.98.159.194 with SMTP id v63mr38969483pfk.91.1474207409730; Sun, 18 Sep 2016 07:03:29 -0700 (PDT) Received: from localhost.localdomain ([104.237.91.172]) by smtp.gmail.com with ESMTPSA id e71sm38963793pfc.75.2016.09.18.07.03.24 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sun, 18 Sep 2016 07:03:29 -0700 (PDT) From: Baoyou Xie To: ck.hu@mediatek.com, p.zabel@pengutronix.de, airlied@linux.ie, matthias.bgg@gmail.com Cc: dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, arnd@arndb.de, baoyou.xie@linaro.org, xie.baoyou@zte.com.cn Subject: [PATCH] drm/mediatek: mark symbols static where possible Date: Sun, 18 Sep 2016 22:02:15 +0800 Message-Id: <1474207335-21966-1-git-send-email-baoyou.xie@linaro.org> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org We get 4 warnings when building kernel with W=1: drivers/gpu/drm/mediatek/mtk_hdmi.c:1089:6: warning: no previous prototype for 'mtk_hdmi_audio_enable' [-Wmissing-prototypes] drivers/gpu/drm/mediatek/mtk_hdmi.c:1095:6: warning: no previous prototype for 'mtk_hdmi_audio_disable' [-Wmissing-prototypes] drivers/gpu/drm/mediatek/mtk_hdmi.c:1101:5: warning: no previous prototype for 'mtk_hdmi_audio_set_param' [-Wmissing-prototypes] drivers/gpu/drm/mediatek/mtk_hdmi.c:1627:5: warning: no previous prototype for 'mtk_hdmi_audio_digital_mute' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks both functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/mediatek/mtk_hdmi.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) -- 2.7.4 diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c b/drivers/gpu/drm/mediatek/mtk_hdmi.c index 334562d..be4e19c 100644 --- a/drivers/gpu/drm/mediatek/mtk_hdmi.c +++ b/drivers/gpu/drm/mediatek/mtk_hdmi.c @@ -1086,19 +1086,19 @@ static int mtk_hdmi_output_init(struct mtk_hdmi *hdmi) return 0; } -void mtk_hdmi_audio_enable(struct mtk_hdmi *hdmi) +static void mtk_hdmi_audio_enable(struct mtk_hdmi *hdmi) { mtk_hdmi_aud_enable_packet(hdmi, true); hdmi->audio_enable = true; } -void mtk_hdmi_audio_disable(struct mtk_hdmi *hdmi) +static void mtk_hdmi_audio_disable(struct mtk_hdmi *hdmi) { mtk_hdmi_aud_enable_packet(hdmi, false); hdmi->audio_enable = false; } -int mtk_hdmi_audio_set_param(struct mtk_hdmi *hdmi, +static int mtk_hdmi_audio_set_param(struct mtk_hdmi *hdmi, struct hdmi_audio_param *param) { if (!hdmi->audio_enable) { @@ -1624,7 +1624,8 @@ static void mtk_hdmi_audio_shutdown(struct device *dev, void *data) mtk_hdmi_audio_disable(hdmi); } -int mtk_hdmi_audio_digital_mute(struct device *dev, void *data, bool enable) +static int +mtk_hdmi_audio_digital_mute(struct device *dev, void *data, bool enable) { struct mtk_hdmi *hdmi = dev_get_drvdata(dev);