Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753608AbaLGUnj (ORCPT ); Sun, 7 Dec 2014 15:43:39 -0500 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]:8490 "EHLO mail2-relais-roc.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753556AbaLGUnf (ORCPT ); Sun, 7 Dec 2014 15:43:35 -0500 X-IronPort-AV: E=Sophos;i="5.07,534,1413237600"; d="scan'208";a="111936499" Date: Sun, 7 Dec 2014 21:43:33 +0100 (CET) From: Julia Lawall X-X-Sender: jll@localhost6.localdomain6 To: Rasmus Villemoes cc: Bjorn Helgaas , kernel-janitors@vger.kernel.org, joe@perches.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/20] PCI: fix misspelling of current function in string In-Reply-To: <87lhmj42h3.fsf@rasmusvillemoes.dk> Message-ID: References: <1417980062-25151-1-git-send-email-Julia.Lawall@lip6.fr> <1417980062-25151-4-git-send-email-Julia.Lawall@lip6.fr> <87lhmj42h3.fsf@rasmusvillemoes.dk> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 7 Dec 2014, Rasmus Villemoes wrote: > On Sun, Dec 07 2014, Julia Lawall wrote: > > > Replace a misspelled function name by %s and then __func__. > > > > The function name begins with pci, not cpci. > ... > > result = cpci_hotplug_init(debug); > > if (result) { > > - err("cpci_hotplug_init with error %d\n", result); > > + err("%s with error %d\n", __func__, result); > > return result; > > } > > This one seems to be a false positive; I think the err() is reporting on > the result from the call of the function which is indeed called > cpci_hotplug_init. Agreed. Thanks. julia -- 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/