From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PATCH V1 1/3] IB/core: Align coding style of ib_device_cap_flags structure Date: Mon, 21 Dec 2015 09:27:21 +0200 Message-ID: References: <1450606571-15877-1-git-send-email-leon@leon.nu> <1450606571-15877-2-git-send-email-leon@leon.nu> <20151221062252.GE3860@phlsvsds.ph.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Or Gerlitz Cc: "ira.weiny" , Doug Ledford , linux-rdma , Leon Romanovsky List-Id: linux-rdma@vger.kernel.org On Mon, Dec 21, 2015 at 8:52 AM, Or Gerlitz wrote: > On Mon, Dec 21, 2015 at 8:37 AM, Leon Romanovsky wrote: >> On Mon, Dec 21, 2015 at 8:22 AM, ira.weiny wrote: >>> On Sun, Dec 20, 2015 at 12:16:09PM +0200, Leon Romanovsky wrote: >>>> From: Leon Romanovsky > >>>> Modify enum ib_device_cap_flags such that other patches which add new >>>> enum values pass strict checkpatch.pl checks. > >>>> - IB_DEVICE_RESERVED = (1<<16), /* old SEND_W_INV */ >>>> - IB_DEVICE_MEM_WINDOW = (1<<17), >>>> + IB_DEVICE_RESIZE_MAX_WR = (1 << 0), > >> 2. Change the whole file => the work with "git blame" will be less >> straightforward. > > Agree. > > Leon, I don't think we need to take checkpatch-ing of things to that level. > > Indeed, we should make sure that whole new enums and such are done right -- > but avoid touching core structs/enums in a manner that disallows > simple git blaming of things, which is very useful for new comers and > old suffers. There are no doubts that standalone fixing checkpatch errors is more suitable to staging subsystem. In our case, it is part of coming changes in that structure. such change serves specific goal to minimize the possibility of error by seeing clean output from static analyser tool. For the new comers and old suffers checkpatch tool is extremely useful too. > >> I will do the change across whole file, If Doug accepts such change. > > Please don't... simple git blame is very powerful tool for kernel > developers everyday work. It is an open question what we prefer more: history with chance to sneak a mistake or less history with less chances to make a mistake. > > Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html