From patchwork Fri Feb 24 06:48:19 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Julia Lawall X-Patchwork-Id: 9589487 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5E7EF601AE for ; Fri, 24 Feb 2017 06:58:36 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4F99528744 for ; Fri, 24 Feb 2017 06:58:36 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 43E15287A1; Fri, 24 Feb 2017 06:58:36 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id DAB7E28744 for ; Fri, 24 Feb 2017 06:58:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Subject:To:From :Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=lYQ1vmb0bUIjxOSD2NHrcLn3onLjjtRri4lDAMS7rVE=; b=cgcS6WigDdTfS+ JrPeNmU96LQvt/snHF4bLugsoxDJZhwhjQ+whKrTi9DtxMInglXlulH2DBqHyt9o9UnibRwKJPva6 JEBTzmu6NZd/eg+UK1BmdOiXZ7L/vKWEPbB+MTM248z4a/ZiEW0DmThWYbTzo58q/A7yQpYRg8/Jp Wumjs8y8anQpgRjz3BtXYbErXIYfP0C3unuZ15fERofDT6grkRoqGMY7HFn54QHeZlceryHjUDNjO LjZgIMo7NS/wkra4aWtEClPPg7rgRMKDokm2dVP4uRwMDgeIVVkivaTHnrkLt+1rZ9WvXrceLt7Mt wtkB50K6ZPPR08DdbXOA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1ch9pm-00065G-PO; Fri, 24 Feb 2017 06:58:30 +0000 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1ch9pj-00064F-41 for linux-arm-kernel@lists.infradead.org; Fri, 24 Feb 2017 06:58:29 +0000 X-IronPort-AV: E=Sophos;i="5.35,200,1484002800"; d="scan'208";a="261885124" Received: from 198.67.28.109.rev.sfr.net (HELO hadrien) ([109.28.67.198]) by mail2-relais-roc.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Feb 2017 07:48:19 +0100 Date: Fri, 24 Feb 2017 07:48:19 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Shubhrajyoti Datta Subject: [PATCH] clk: zynqmp: fix itnull.cocci warnings Message-ID: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170223_225827_489932_F53875F2 X-CRM114-Status: UNSURE ( 8.35 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Michal Simek , Michael Turquette , Stephen Boyd , linux-kernel@vger.kernel.org, kbuild-all@01.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Iterator variable bound on line 220 cannot be NULL Generated by: scripts/coccinelle/iterators/itnull.cocci CC: Shubhrajyoti Datta Signed-off-by: Julia Lawall Signed-off-by: Fengguang Wu --- tree: https://github.com/Xilinx/linux-xlnx master head: 1741e191412444e3ab3308b407a9328186c248e4 commit: 75b49bd76a45208749235a7ba35dba96e3e706f0 [358/384] clk: zynqmp: Add check for no of users for vpll Actually, the patch needs to be extended to drop the braces as well. clk.c | 2 -- 1 file changed, 2 deletions(-) --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -218,8 +218,6 @@ static void clk_show_subtree(struct clk_ sibling++; hlist_for_each_entry(child, &c->children, child_node) { - if (!child) - return; clk_show_subtree(child, level + 1); } }