Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754069AbbHRSn4 (ORCPT ); Tue, 18 Aug 2015 14:43:56 -0400 Received: from mail-ig0-f169.google.com ([209.85.213.169]:37482 "EHLO mail-ig0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753570AbbHRSnz (ORCPT ); Tue, 18 Aug 2015 14:43:55 -0400 MIME-Version: 1.0 In-Reply-To: <20150817224850.GM26431@google.com> References: <1438039809-24957-1-git-send-email-yinghai@kernel.org> <20150817224850.GM26431@google.com> Date: Tue, 18 Aug 2015 11:43:54 -0700 X-Google-Sender-Auth: TqY_N5t2caa9eHFOp0Rf_B2xIV0 Message-ID: Subject: Re: [PATCH v3 00/51] PCI: Resource allocation cleanup for v4.3 From: Yinghai Lu To: Bjorn Helgaas Cc: David Miller , Benjamin Herrenschmidt , Wei Yang , TJ , Yijing Wang , Andrew Morton , "linux-pci@vger.kernel.org" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1426 Lines: 40 On Mon, Aug 17, 2015 at 3:48 PM, Bjorn Helgaas wrote: > Overall comments: > > Thanks for paying attention to your subject lines. They are > capitalized and punctuated quite consistently. > > The one-line subject is a summary. The body of the changelog is not a > continuation of it. The body should stand alone even if you don't > read the subject. Sometimes that means you have to repeat the subject > line in the body. That's OK. > > Please use "()" when referring to functions. You do it sometimes, but > not always. Thanks for checking them. Will update those changelog. > > I think we should talk about "required/optional" resources instead of > "must/optional" ones. Is "addon" a third, separate, thing? yes, addon should be same as optional. Actually for devices resources, could safely use required/optional resources. those resources either required only or optional only resources. for bridge resources, that is little confusing, as those resources could have required part only or optional part only or have required part + optional part. will update changelog to make them consistent. also will check code/comment... Thanks Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/