From patchwork Tue Apr 17 01:04:41 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hartley Sweeten X-Patchwork-Id: 153021 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 67ECEB7015 for ; Tue, 17 Apr 2012 11:04:58 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755832Ab2DQBE5 (ORCPT ); Mon, 16 Apr 2012 21:04:57 -0400 Received: from mail131.messagelabs.com ([216.82.242.99]:61817 "EHLO mail131.messagelabs.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755831Ab2DQBE4 (ORCPT ); Mon, 16 Apr 2012 21:04:56 -0400 X-Env-Sender: hartleys@visionengravers.com X-Msg-Ref: server-9.tower-131.messagelabs.com!1334624693!16502777!3 X-Originating-IP: [216.166.12.32] X-StarScan-Version: 6.5.7; banners=-,-,- X-VirusChecked: Checked Received: (qmail 11627 invoked from network); 17 Apr 2012 01:04:54 -0000 Received: from out001.collaborationhost.net (HELO out001.collaborationhost.net) (216.166.12.32) by server-9.tower-131.messagelabs.com with RC4-SHA encrypted SMTP; 17 Apr 2012 01:04:54 -0000 Received: from etch.local (10.2.3.210) by smtp.collaborationhost.net (10.2.0.47) with Microsoft SMTP Server (TLS) id 8.3.213.0; Mon, 16 Apr 2012 20:04:51 -0500 From: H Hartley Sweeten To: Linux Kernel Subject: [PATCH] ata: libata-transport.c: local functions should not be exposed globally Date: Mon, 16 Apr 2012 18:04:41 -0700 User-Agent: KMail/1.9.9 CC: , MIME-Version: 1.0 Content-Disposition: inline Message-ID: <201204161804.41337.hartleys@visionengravers.com> Sender: linux-ide-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org Functions not referenced outside of a source file should be marked static to prevent it from being exposed globally. This quiets the sparse warnings: warning: symbol 'ata_is_port' was not declared. Should it be static? warning: symbol 'ata_is_link' was not declared. Should it be static? warning: symbol 'ata_is_ata_dev' was not declared. Should it be static? Signed-off-by: H Hartley Sweeten Cc: Jeff Garzik --- -- To unsubscribe from this list: send the line "unsubscribe linux-ide" 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/ata/libata-transport.c b/drivers/ata/libata-transport.c index 74aaee3..201672a 100644 --- a/drivers/ata/libata-transport.c +++ b/drivers/ata/libata-transport.c @@ -232,7 +232,7 @@ static void ata_tport_release(struct device *dev) * Returns: * %1 if the device represents a ATA Port, %0 else */ -int ata_is_port(const struct device *dev) +static int ata_is_port(const struct device *dev) { return dev->release == ata_tport_release; } @@ -354,7 +354,7 @@ static void ata_tlink_release(struct device *dev) * Returns: * %1 if the device represents a ATA link, %0 else */ -int ata_is_link(const struct device *dev) +static int ata_is_link(const struct device *dev) { return dev->release == ata_tlink_release; } @@ -571,7 +571,7 @@ static void ata_tdev_release(struct device *dev) * Returns: * %1 if the device represents a ATA device, %0 else */ -int ata_is_ata_dev(const struct device *dev) +static int ata_is_ata_dev(const struct device *dev) { return dev->release == ata_tdev_release; }