From patchwork Fri Apr 4 20:39:36 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 3940851 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 4D56C9F1EE for ; Fri, 4 Apr 2014 20:39:44 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 78F1620396 for ; Fri, 4 Apr 2014 20:39:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9505E20394 for ; Fri, 4 Apr 2014 20:39:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752023AbaDDUjl (ORCPT ); Fri, 4 Apr 2014 16:39:41 -0400 Received: from mail-pb0-f47.google.com ([209.85.160.47]:63511 "EHLO mail-pb0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751688AbaDDUjk (ORCPT ); Fri, 4 Apr 2014 16:39:40 -0400 Received: by mail-pb0-f47.google.com with SMTP id up15so3959140pbc.34 for ; Fri, 04 Apr 2014 13:39:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=duA3hU+QD8aPXCYyRxhUySZo464NqnUVlaLLmP0KGbE=; b=UXyguqKUFsQd8LjzcmQqSNsGDxqqc7Wvy1VC+rEOCpWtv5RYGCelDYOjKU9fhV0yAQ y2+9FOpzuznBOXhD999D0AEM6DRonDU4h16lrjOTaO7LWya+Rp8/kTEJO4YZLH4kw6Mu idv9d6iasChRS5u0h4/cJXC/Gu+sMU3o0M5ECqMFWi9cIxqhFjD5teZiZsdz5NxmmxHH DBpAWZYhGRsowMZuebS56Hzf0KCRiEhIUl1JJL0DIYCtF238c7+/qODFm71fzsemdB+C QpOp1a6ze4nbYGt4L/xI83UdqamDHXieYGsMB1YmA5KWsDyRVRmRFARwZzHWTBrpDMnw g9Ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-type:content-disposition :content-transfer-encoding:in-reply-to:user-agent; bh=duA3hU+QD8aPXCYyRxhUySZo464NqnUVlaLLmP0KGbE=; b=NaduQufmmgcmbT0Vik1ZNcd8r7gq70N4X5ZgqbzZwyXw6FvfOng74eOJEBdZD9eWbu 5uYUGtEU02NCuXUlAPFp/1VDwUvMG+BuvDWNivtowC6PWYtgoq3Y/51VpSwKr+PG72tX 5hStzI+Md0WBuJu4BkPHg+5c5MuAzPoUzqqLFmDxdHXDCDZ+IC3mH/VtRXnJWonhd7XI V8eDLiptRiTM8TK/H5ayz9BWYNXtrHVs66b16yGumRY7JEF4dWX0ZaQEROp9A9EokoMF SFsZ5MvkdnMcLvnxvVY2x+YvgV8tboM6riEWqkKDjS+9MXtIpkKm/kRgvAahQBWet5Im HRCA== X-Gm-Message-State: ALoCoQl40JhYbHKaysg5ry5+xoegO7b7MXhOMQGhZSIV0AG6gI80vLMv8VKw+rqT+qxvV98HVUwqtM6EQima7DpUB0kpeWF+vTUOgcixUT92wbzPTIRpBmoECfGrPAA9fapjzjJ6WNOXajYVWJrdxo0JFF04C1xcIJrRK6HkdMyh3cCmAYoWohxephp4M8zgynBs0GpxtDB9FGBMPVK6y+XUeHa5jHHLhw== X-Received: by 10.67.13.134 with SMTP id ey6mr17066975pad.44.1396643980153; Fri, 04 Apr 2014 13:39:40 -0700 (PDT) Received: from google.com ([172.19.254.18]) by mx.google.com with ESMTPSA id ek2sm8406813pbd.30.2014.04.04.13.39.38 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Fri, 04 Apr 2014 13:39:39 -0700 (PDT) Date: Fri, 4 Apr 2014 14:39:36 -0600 From: Bjorn Helgaas To: Ryan Desfosses Cc: trivial@kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org Subject: Re: [PATCH 2/2] pci: added space before the open parenthesis Message-ID: <20140404203936.GA13931@google.com> References: <1396635490-3630-1-git-send-email-ryan@desfo.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1396635490-3630-1-git-send-email-ryan@desfo.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Fri, Apr 04, 2014 at 02:18:10PM -0400, Ryan Desfosses wrote: > change made to resolve checkpatch.pl error > > Signed-off-by: Ryan Desfosses Hi Ryan, Thanks for this and your previous fix.  This "Signed-off-by" looks good. Would you mind reposting the "pci_bus*" fix with a similar "Signed-off-by"? It's a small fix, but strictly speaking, I need the "Signed-off-by" before I can apply a patch. We normally write the changelog so it can stand alone, i.e., you shouldn't have to read the summary in addition to the changelog for it to make sense. It might be useful to browse other changelogs, e.g., with "git log --no-merges drivers/pci". I do often fix capitalization, spelling, and other errors because I think consistency makes it easier to read the history. The "[2/2]" notation suggests that there's another patch labeled "[1/2]" (and probably an overall summary labeled "[0/2]"), and that all were posted together.  In this case, I think you intended your "pci_bus*" patch to be [1/2], but since they weren't posted together, it's a bit harder for me to keep track of them as a group. If you don't mind, I'll collect up this and your previous fix (and any future ones you find) and squash them into a single "whitespace fixes" sort of patch to reduce the number of commits for obviously-correct changes. Thanks again, and welcome to Linux kernel development! Oh, and I applied the patch below to my pending/misc branch (and I'll update it with the other one when you post it). I have a few other things I hope to ask Linus to put in v3.15 yet, and I'll include this. Bjorn From: Ryan Desfosses PCI: Fix whitespace issues Resolve whitespace issues that checkpatch.pl complained about. No functional change. Signed-off-by: Ryan Desfosses Signed-off-by: Bjorn Helgaas --- drivers/pci/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index 7325d43bf030..1f0bf2eec127 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -114,7 +114,7 @@ unsigned char pci_bus_max_busnr(struct pci_bus* bus) max = bus->busn_res.end; list_for_each_entry(tmp, &bus->children, node) { n = pci_bus_max_busnr(tmp); - if(n > max) + if (n > max) max = n; } return max;