Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755189AbcJ0Qf4 (ORCPT ); Thu, 27 Oct 2016 12:35:56 -0400 Received: from mail-pf0-f196.google.com ([209.85.192.196]:35927 "EHLO mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750742AbcJ0Qfy (ORCPT ); Thu, 27 Oct 2016 12:35:54 -0400 Subject: Re: [RFC PATCH 06/13] of: Remove prefix "__of_" and prefix "__" from local function names To: Rob Herring References: <1477429146-27039-1-git-send-email-frowand.list@gmail.com> <1477429146-27039-7-git-send-email-frowand.list@gmail.com> Cc: Pantelis Antoniou , Pantelis Antoniou , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" From: Frank Rowand Message-ID: <58122CC0.3090700@gmail.com> Date: Thu, 27 Oct 2016 09:35:12 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1160 Lines: 32 On 10/27/16 05:47, Rob Herring wrote: > On Tue, Oct 25, 2016 at 3:58 PM, wrote: >> From: Frank Rowand > > I prefer to leave the prefixes and this is getting into pointless churn. > >> >> Signed-off-by: Frank Rowand >> --- >> drivers/of/resolver.c | 10 +++++----- >> 1 file changed, 5 insertions(+), 5 deletions(-) > If I was just submitting this as a single patch, I would agree. But since I am making so many other changes, I think it makes sense to do this as part of this series. It is broken apart as a separate patch to be easy to review and not pollute any of the other patches in the series. The prefixes add no value for a local function, but they do add noise when reading code. The changes are local to this file and do not impact anything else. Looking at the single patch, it does seem like churn. But looking at the entire file before the set of changes and after the set of changes, I find the file much easier to read afterwards. Each individual patch may make a small contribution to the end result, but the combination of all of them is significant.