From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751443AbdGRG63 (ORCPT ); Tue, 18 Jul 2017 02:58:29 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:36976 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751026AbdGRG62 (ORCPT ); Tue, 18 Jul 2017 02:58:28 -0400 Date: Tue, 18 Jul 2017 08:58:22 +0200 From: Greg KH To: Joseph Wright Cc: arve@android.com, riandrews@android.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 1/1] Staging: android/ion: declare function static Message-ID: <20170718065822.GA24804@kroah.com> References: <20170715114305.1336-1-rjosephwright@gmail.com> <20170715114305.1336-2-rjosephwright@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170715114305.1336-2-rjosephwright@gmail.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jul 15, 2017 at 11:43:05AM +0000, Joseph Wright wrote: > Declare private function static to fix sparse warning: > > ion_cma_heap.c:109:5: warning: symbol '__ion_add_cma_heaps' \ > was not declared. Should it be static? > > Signed-off-by: Joseph Wright > --- > Changes in v3: > - Make subject clearer > > drivers/staging/android/ion/ion_cma_heap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Sorry, does not apply to my tree as I think someone else already did this before you did :(