From patchwork Tue Aug 3 00:04:11 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Justin Mattock X-Patchwork-Id: 60702 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id BAF6CB70D8 for ; Tue, 3 Aug 2010 10:04:39 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753214Ab0HCAEQ (ORCPT ); Mon, 2 Aug 2010 20:04:16 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:40613 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751797Ab0HCAEP (ORCPT ); Mon, 2 Aug 2010 20:04:15 -0400 Received: by pzk26 with SMTP id 26so1489424pzk.19 for ; Mon, 02 Aug 2010 17:04:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer; bh=DIBMfll7JEwE77x5z+DjV94hIA+C7UKvJKinwvUBoys=; b=TKXY9vdh4DBb9xE96r3toudLJ/H68EQyfk6BGoNsCEqKkLHgmhRJay5IYJibRo3hhT Y7BRbCxs83Sc8EEri+e/GBUcseFdXpCeQjqJQly/G2rXHHo1X5L6Jx1lYDRghGptp/af N1lkGeK+YLfPMysccl62bRYuHkvVR1ZGuAw0g= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer; b=YfVspS3BXoOJduw74XhzFiZNGyS+2Su33fVEGuStMq6V9gE5Vjo6ghEubfo2LUoUWE hz8TGcxHLd7sHIBf/15SPhg5MK2axVqY7p5W7udNHREG0lTsyRVYHrJ/JDaVRYyvjQdV 2QwVauw0WMgM0+OXydVHX7wXIgPwtxH4IUYm8= Received: by 10.114.74.16 with SMTP id w16mr7975922waa.179.1280793854575; Mon, 02 Aug 2010 17:04:14 -0700 (PDT) Received: from localhost.localdomain ([76.91.45.220]) by mx.google.com with ESMTPS id s5sm12385651wak.12.2010.08.02.17.04.12 (version=TLSv1/SSLv3 cipher=RC4-MD5); Mon, 02 Aug 2010 17:04:13 -0700 (PDT) From: "Justin P. Mattock" To: linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, j@w1.fi, linville@tuxdriver.com, "Justin P. Mattock" Subject: [PATCH]hostap:hostap_ioctl.c Fix variable 'ret' set but not used Date: Mon, 2 Aug 2010 17:04:11 -0700 Message-Id: <1280793851-1719-1-git-send-email-justinmattock@gmail.com> X-Mailer: git-send-email 1.7.1.rc1.21.gf3bd6 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The below patch fixes a warning message generated by GCC: CC [M] drivers/net/wireless/hostap/hostap_ioctl.o drivers/net/wireless/hostap/hostap_ioctl.c: In function 'prism2_request_scan': drivers/net/wireless/hostap/hostap_ioctl.c:1666:6: warning: variable 'ret' set but not used Keep in mind Im not sure if this is the right fix for this, so please feedback is appreciated. Signed-off-by: Justin P. Mattock --- drivers/net/wireless/hostap/hostap_ioctl.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/hostap/hostap_ioctl.c b/drivers/net/wireless/hostap/hostap_ioctl.c index a85e43a..27d462f 100644 --- a/drivers/net/wireless/hostap/hostap_ioctl.c +++ b/drivers/net/wireless/hostap/hostap_ioctl.c @@ -1663,7 +1663,6 @@ static int prism2_request_scan(struct net_device *dev) struct hostap_interface *iface; local_info_t *local; struct hfa384x_scan_request scan_req; - int ret = 0; iface = netdev_priv(dev); local = iface->local; @@ -1689,7 +1688,7 @@ static int prism2_request_scan(struct net_device *dev) if (local->func->set_rid(dev, HFA384X_RID_SCANREQUEST, &scan_req, sizeof(scan_req))) { printk(KERN_DEBUG "SCANREQUEST failed\n"); - ret = -EINVAL; + return -EINVAL; } if (!local->host_roaming)