Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757536AbcJXO1w (ORCPT ); Mon, 24 Oct 2016 10:27:52 -0400 Received: from mail-wm0-f66.google.com ([74.125.82.66]:34016 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751147AbcJXO1t (ORCPT ); Mon, 24 Oct 2016 10:27:49 -0400 Date: Mon, 24 Oct 2016 16:27:45 +0200 From: Daniel Vetter To: Arnd Bergmann Cc: Baoyou Xie , tang.qiang007@zte.com.cn, xie.baoyou@zte.com.cn, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, rmk+kernel@armlinux.org.uk, han.fei@zte.com.cn Subject: Re: [PATCH] drm/i2c/tda998x: mark symbol static where possible Message-ID: <20161024142745.GQ20761@phenom.ffwll.local> Mail-Followup-To: Arnd Bergmann , Baoyou Xie , tang.qiang007@zte.com.cn, xie.baoyou@zte.com.cn, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, rmk+kernel@armlinux.org.uk, han.fei@zte.com.cn References: <1477127682-3615-1-git-send-email-baoyou.xie@linaro.org> <5367505.BUI00TSeFs@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5367505.BUI00TSeFs@wuerfel> X-Operating-System: Linux phenom 4.6.0-1-amd64 User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 732 Lines: 20 On Mon, Oct 24, 2016 at 12:14:14PM +0200, Arnd Bergmann wrote: > On Saturday, October 22, 2016 5:14:42 PM CEST Baoyou Xie wrote: > > We get 1 warning when building kernel with W=1: > > drivers/gpu/drm/i2c/tda998x_drv.c:1292:5: warning: no previous prototype for 'tda998x_audio_digital_mute' [-Wmissing-prototypes] > > > > In fact, this function is only used in the file in which it is > > declared and don't need a declaration, but can be made static. > > So this patch marks this function with 'static'. > > > > Signed-off-by: Baoyou Xie > > > > Reviewed-by: Arnd Bergmann Applied to drm-misc, thanks. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch