Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp1725189ybg; Sat, 19 Oct 2019 01:08:47 -0700 (PDT) X-Google-Smtp-Source: APXvYqyAm09zc/BVh0g/RMRV77Lj6Y9DediGFjRju1R6Anvgyq+DqcHTvF6aIH5icNH58/sRDCHH X-Received: by 2002:a17:906:af46:: with SMTP id ly6mr12703148ejb.330.1571472527235; Sat, 19 Oct 2019 01:08:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571472527; cv=none; d=google.com; s=arc-20160816; b=wyFZVb95+mgfsY9qNeAlCFVop7a6CzWff8l+pcECwZ524vNYXMnhVA6Fb/dhzs7Kv4 Y/kVEQIPW/bRjvJ4HIA2AbjUTBWfSgZDG6fUWw980zt0KnMwg0Ksv1H2ckG+f4z6WhQ6 bEisoRyrcMAriUsKlg1I078YgrbpLGfj5NCmii3bVdk7VlJspQG8UpYp5jvRWG8tUTpS AnPHQvvlRuSdiYvinEFUISuuUd4FOlSeVmrG+PRdkyh/+KkDbg8CxOzM3gJwKMm1P1KQ GcZG57neMnrp70fdWEINaPl7yuqBhCKPQoqMY5Ikoc9CeV6qSl3dxzwwJOWUEyjKnCxU FwHw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-transfer-encoding:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=/uYaw2p2ptTno7fIrnLtgyVYV8MoeYukRKtDUtGvrcE=; b=ueAcfxJIFoXEYZktkv86plh44VHTgGJrLfnNq1O/x/X2BTw6PL01QNhmvEkt7xuQ3d 36WfjceVshyAnkfUbQuFRzevHLJvErcEMOxuPdTbuEJefHD/hcGo6A44AqvOCm4f4XHo bCE5vSFhot23o0W4XWmXFSgJlo8q+UIpsiUbqJWVlnlu+QBtuaAVKkgQVu3ugZGnm6la Hkr8m7CQWvMjcBwYYtJhDI/ixmQhmCQ+y5uO7dmXfry21KXcx8vE2ul+I8STbfcUUoN0 zrdRxC5+dwUIMQUMkP+37EW0YnsztHg5etvXO/dKwwFx7wvryB0peEF/de9xr1hMAFcc RzyA== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b25si4889575eja.198.2019.10.19.01.08.24; Sat, 19 Oct 2019 01:08:47 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2407709AbfJRJtd (ORCPT + 99 others); Fri, 18 Oct 2019 05:49:33 -0400 Received: from [217.140.110.172] ([217.140.110.172]:60164 "EHLO foss.arm.com" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S2391488AbfJRJtd (ORCPT ); Fri, 18 Oct 2019 05:49:33 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E1790492; Fri, 18 Oct 2019 02:49:08 -0700 (PDT) Received: from e110455-lin.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C26FD3F6C4; Fri, 18 Oct 2019 02:49:08 -0700 (PDT) Received: by e110455-lin.cambridge.arm.com (Postfix, from userid 1000) id 81002682189; Fri, 18 Oct 2019 10:49:07 +0100 (BST) Date: Fri, 18 Oct 2019 10:49:07 +0100 From: Liviu Dudau To: "Ben Dooks (Codethink)" Cc: linux-kernel@lists.codethink.co.uk, Brian Starkey , David Airlie , Daniel Vetter , malidp@foss.arm.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drm/arm: make undeclared items static Message-ID: <20191018094907.66ghzs3qiyelibzh@e110455-lin.cambridge.arm.com> References: <20191017111756.12861-1-ben.dooks@codethink.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20191017111756.12861-1-ben.dooks@codethink.co.uk> User-Agent: NeoMutt/20180716 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 17, 2019 at 12:17:55PM +0100, Ben Dooks (Codethink) wrote: > Make the following items static to avoid clashes with > other parts of the kernel (dev_attr_core_id) or just > silence the following sparse warning: > > drivers/gpu/drm/arm/malidp_drv.c:371:24: warning: symbol 'malidp_fb_create' was not declared. Should it be static? > drivers/gpu/drm/arm/malidp_drv.c:494:6: warning: symbol 'malidp_error_stats_dump' was not declared. Should it be static? > drivers/gpu/drm/arm/malidp_drv.c:668:1: warning: symbol 'dev_attr_core_id' was not declared. Should it be static? > > Signed-off-by: Ben Dooks Acked-by: Liviu Dudau Thanks for the patch! As a side note: the dim tool that we use in the DRM subsystem flags your S-o-b as being different from author, due to "(Codethink)" addition in the email name. Best regards, Liviu > --- > Cc: Liviu Dudau > Cc: Brian Starkey > Cc: David Airlie > Cc: Daniel Vetter > Cc: malidp@foss.arm.com > Cc: dri-devel@lists.freedesktop.org > Cc: linux-kernel@vger.kernel.org > .. (open list) > --- > drivers/gpu/drm/arm/malidp_drv.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/arm/malidp_drv.c b/drivers/gpu/drm/arm/malidp_drv.c > index 333b88a5efb0..18ca43c9cef4 100644 > --- a/drivers/gpu/drm/arm/malidp_drv.c > +++ b/drivers/gpu/drm/arm/malidp_drv.c > @@ -368,7 +368,7 @@ malidp_verify_afbc_framebuffer(struct drm_device *dev, struct drm_file *file, > return false; > } > > -struct drm_framebuffer * > +static struct drm_framebuffer * > malidp_fb_create(struct drm_device *dev, struct drm_file *file, > const struct drm_mode_fb_cmd2 *mode_cmd) > { > @@ -491,9 +491,9 @@ void malidp_error(struct malidp_drm *malidp, > spin_unlock_irqrestore(&malidp->errors_lock, irqflags); > } > > -void malidp_error_stats_dump(const char *prefix, > - struct malidp_error_stats error_stats, > - struct seq_file *m) > +static void malidp_error_stats_dump(const char *prefix, > + struct malidp_error_stats error_stats, > + struct seq_file *m) > { > seq_printf(m, "[%s] num_errors : %d\n", prefix, > error_stats.num_errors); > @@ -665,7 +665,7 @@ static ssize_t core_id_show(struct device *dev, struct device_attribute *attr, > return snprintf(buf, PAGE_SIZE, "%08x\n", malidp->core_id); > } > > -DEVICE_ATTR_RO(core_id); > +static DEVICE_ATTR_RO(core_id); > > static int malidp_init_sysfs(struct device *dev) > { > -- > 2.23.0 > -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯