From patchwork Fri Jul 13 07:51:40 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hongbo Zhang X-Patchwork-Id: 9975 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id D52D623F2A for ; Fri, 13 Jul 2012 07:53:22 +0000 (UTC) Received: from mail-yw0-f52.google.com (mail-yw0-f52.google.com [209.85.213.52]) by fiordland.canonical.com (Postfix) with ESMTP id A547EA18193 for ; Fri, 13 Jul 2012 07:53:22 +0000 (UTC) Received: by yhpp61 with SMTP id p61so3711746yhp.11 for ; Fri, 13 Jul 2012 00:53:22 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-forwarded-to:x-forwarded-for:delivered-to:received-spf:from:to:cc :subject:date:message-id:x-mailer:in-reply-to:references :x-gm-message-state; bh=cP5wht8i0Kj8HTNncDt8iNEureu5zllxu8QzFj8AXEU=; b=ACIi1xyD7brZtc61XntMwx4XWIXdLRRRZA7QKpD3KBBZQTAC13eWMD4JYsxa85nURj NFRdDNFjiPq33grbxgw0Lb7BpweYD4zMZ1KebAISvqDH3XCKawB/q4pH4wHJkOj2jpLR i8XkRzT7IcN8YuXTx7N+3fad+yB6mCh01Ns0vzBfuyyCivE8DjTR+WdS/mgJSXtWvspc oIdFBne+LE1E4PNjHj1Zc1NNidQ3KFndXDzrmEJb6ydkiAFNoTorpV0Nmrd7uBIWrINs 8+e16mde9R84Fq38ylFhxWiz8t7d4uI0GSHY+dV0kxseutlpHSOpQMMQJapFeFh+W2I7 3OJg== Received: by 10.50.203.39 with SMTP id kn7mr54433igc.53.1342166001830; Fri, 13 Jul 2012 00:53:21 -0700 (PDT) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.231.241.2 with SMTP id lc2csp12556ibb; Fri, 13 Jul 2012 00:53:21 -0700 (PDT) Received: by 10.68.203.66 with SMTP id ko2mr1357596pbc.84.1342166001184; Fri, 13 Jul 2012 00:53:21 -0700 (PDT) Received: from mail-pb0-f50.google.com (mail-pb0-f50.google.com [209.85.160.50]) by mx.google.com with ESMTPS id gg3si13227368pbc.289.2012.07.13.00.53.20 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 13 Jul 2012 00:53:21 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.160.50 is neither permitted nor denied by best guess record for domain of hongbo.zhang@linaro.org) client-ip=209.85.160.50; Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.160.50 is neither permitted nor denied by best guess record for domain of hongbo.zhang@linaro.org) smtp.mail=hongbo.zhang@linaro.org Received: by pbbrr4 with SMTP id rr4so6149694pbb.37 for ; Fri, 13 Jul 2012 00:53:20 -0700 (PDT) Received: by 10.68.227.198 with SMTP id sc6mr1207301pbc.138.1342166000602; Fri, 13 Jul 2012 00:53:20 -0700 (PDT) Received: from stebjsxu0148.bjs.st.com ([124.127.135.94]) by mx.google.com with ESMTPS id oy8sm5522486pbc.52.2012.07.13.00.53.14 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 13 Jul 2012 00:53:19 -0700 (PDT) From: "hongbo.zhang" To: linaro-dev@lists.linaro.org Cc: patches@linaro.org, linaro-kernel@lists.linaro.org, STEricsson_nomadik_linux@list.st.com, kernel@igloocommunity.org, "hongbo.zhang" Subject: [PATCH 1/2] Remove un-necessary variable checking Date: Fri, 13 Jul 2012 15:51:40 +0800 Message-Id: <1342165901-24800-2-git-send-email-hongbo.zhang@linaro.com> X-Mailer: git-send-email 1.7.10 In-Reply-To: <1342165901-24800-1-git-send-email-hongbo.zhang@linaro.com> References: <1342165901-24800-1-git-send-email-hongbo.zhang@linaro.com> X-Gm-Message-State: ALoCoQlGwLgtXy1IyCfh46zXBz8Y9GELYU2kvOO9K2UmsFKMa599LexA4NONeICyN7kkJbl9e6Y4 From: "hongbo.zhang" Signed-off-by: hongbo.zhang --- drivers/thermal/cpu_cooling.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index c40d9a0..a8aa10f 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -399,8 +399,7 @@ struct thermal_cooling_device *cpufreq_cooling_register( /*Verify that all the entries of freq_clip_table are present*/ for (i = 0; i < tab_size; i++) { clip_tab = ((struct freq_clip_table *)&tab_ptr[i]); - if (!clip_tab->freq_clip_max || !clip_tab->mask_val - || !clip_tab->temp_level) { + if (!clip_tab->freq_clip_max || !clip_tab->mask_val) { kfree(cpufreq_dev); return ERR_PTR(-EINVAL); }