From mboxrd@z Thu Jan 1 00:00:00 1970 From: wangyufen Subject: Re: [PATCH net-next v2 3/3] ipv6: fix checkpatch errors of "foo*" and "foo * bar" Date: Fri, 28 Mar 2014 17:33:54 +0800 Message-ID: <53354202.4080108@huawei.com> References: <1395979624-8544-1-git-send-email-wangyufen@huawei.com> <1395979624-8544-4-git-send-email-wangyufen@huawei.com> <1395980521.19009.8.camel@joe-AO722> <20140328.014012.963819707045886028.davem@davemloft.net> <1395985645.31134.2.camel@joe-AO722> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: David Miller , To: Joe Perches Return-path: Received: from szxga02-in.huawei.com ([119.145.14.65]:33955 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750893AbaC1JfM (ORCPT ); Fri, 28 Mar 2014 05:35:12 -0400 In-Reply-To: <1395985645.31134.2.camel@joe-AO722> Sender: netdev-owner@vger.kernel.org List-ID: On 2014/3/28 13:47, Joe Perches wrote: > On Fri, 2014-03-28 at 01:40 -0400, David Miller wrote: >> From: Joe Perches >> Date: Thu, 27 Mar 2014 21:22:01 -0700 >> >>> On Fri, 2014-03-28 at 12:07 +0800, Wangyufen wrote: >>> [] >>>> diff --git a/net/ipv6/ip6_fib.c b/net/ipv6/ip6_fib.c >>> [] >>>> @@ -143,7 +143,7 @@ static __inline__ __be32 addr_bit_set(const void *token, int fn_bit) >>>> addr[fn_bit >> 5]; >>>> } >>> >>> Perhaps all the __inline__ uses could be changed to inline too. >> >> Or rather, deleted completely, this is a *.c file after all. > > Maybe right. There are a lot though just in net/ > > $ git ls-files net | grep "\.c$" | \ > xargs grep -Pw "_{0,2}inline_{0,2}" | wc -l > 1512 > > > > > Or change to inline and move the whole function to ip6_fib.h?