From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759418AbcIWMNs (ORCPT ); Fri, 23 Sep 2016 08:13:48 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:52635 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757892AbcIWMNq (ORCPT ); Fri, 23 Sep 2016 08:13:46 -0400 Date: Fri, 23 Sep 2016 14:13:52 +0200 From: Greg KH To: Moshe Green Cc: sudipm.mukherjee@gmail.com, teddy.wang@siliconmotion.com, linux-fbdev@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: sm750fb: rename getChipType to get_chip_type Message-ID: <20160923121352.GA3556@kroah.com> References: <9a9ab953f0c55df0ffe75a7a3c35ba6dc9e2f742.1474568117.git.mgmoshes@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9a9ab953f0c55df0ffe75a7a3c35ba6dc9e2f742.1474568117.git.mgmoshes@gmail.com> User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote: > Rename CamelCased function getChipType to get_chip_type. > This issue was found by checkpatch.pl As this is a global function, can you rename it to something like "sm750_get_chip_type()"? Having a driver-specific function called get_chip_type() is not good. thanks, greg k-h