Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp1258624imm; Sun, 15 Jul 2018 03:22:15 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeR3RKpXcZ0YfetfNkQwEKIbsr3qIlpxli1nHfJYjqsOOVMI+CieLc0W2XMVLGwPP/v6q3s X-Received: by 2002:a63:5624:: with SMTP id k36-v6mr11955314pgb.146.1531650135706; Sun, 15 Jul 2018 03:22:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531650135; cv=none; d=google.com; s=arc-20160816; b=Kpg/vsYyZhiJYQBON34gvCHPnWTki3t69ainnL4LBcLBdWdyaJialPgqRQRIi1fSTC AVA2LXBZU7d1DOxXdJvZVJ/L8/rdfRmCLGz9bQdIS5yllB3qUGOQ6EAHKzDw6x+J2XZh 6SDq+Nfv/X9Kq88eoxGkYVK4SndeyHWifCII1YVKvsmqELTO6cRQA1yJ+QCpf6E4o5cC mf/CRUtXLMWuKconx1WurFMfnphoIKMph20jH4ayCqZeVzXeic09J+BtKATtLBU+4UUq CC66TPWgf2GOeIUNzxHWZs7DAwyrESTtoK8ApRwd2f0q3Pu5Ya0e7B+EzZ+iXnQuEfJV 6tHQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=hSCVnoW1y2rXJzKVL5ywQBjflJ8Iv9EyyPZzRQ6SlZU=; b=k1gAozxkKFjNSkf9yjb8hzypt9fUAjC/5m/zAOOpUTGGf2tKr2E9qGcYEQMXaqFpvY wG2sDlMThuXmMA4fa4Rb+xXIF5a/JzwjWDi35y5dt8PQfTi5iw4R7RakorH2JfXxrUta eKIv2pSsOCsRd/hfPUGjwzfSensntMVmrVNYytkw3a5sO+GdcJs67ugPOcBPzKWJDum0 Ow0yY8htq5yn3UHA8/nDrJ4Djx1a3JtIpA+1UXJFnkf7+JHI+oY4cVwOwZzPslMZXGj1 v5xAht/T06fXVAWYanJ64Ar7sI+BZWlxEfghvkrcuKY27YaTuI6gQOYXaJzuM80kx05E Teeg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q5-v6si19011248pgn.95.2018.07.15.03.22.00; Sun, 15 Jul 2018 03:22:15 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726184AbeGOKn4 (ORCPT + 99 others); Sun, 15 Jul 2018 06:43:56 -0400 Received: from www.osadl.org ([62.245.132.105]:37325 "EHLO www.osadl.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726060AbeGOKn4 (ORCPT ); Sun, 15 Jul 2018 06:43:56 -0400 Received: from debian01.hofrr.at (178.115.242.59.static.drei.at [178.115.242.59] (may be forged)) by www.osadl.org (8.13.8/8.13.8/OSADL-2007092901) with ESMTP id w6FAJBaI004985; Sun, 15 Jul 2018 12:19:12 +0200 From: Nicholas Mc Guire To: Michael Turquette Cc: Stephen Boyd , Kees Cook , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Nicholas Mc Guire Subject: [PATCH 1/2] drivers: clk: st: warn on iomap failure Date: Sun, 15 Jul 2018 12:18:23 +0200 Message-Id: <1531649904-6072-1-git-send-email-hofrat@osadl.org> X-Mailer: git-send-email 2.1.4 X-Spam-Status: No, score=-1.9 required=6.0 tests=BAYES_00 autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on www.osadl.org Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org While the return value of clkgen_get_register_base() is being checked at the call site, there is no indication of failure cause thus making diagnosis of the issues hard. The WARN_ON() allows to determine the cause of failure. Signed-off-by: Nicholas Mc Guire --- Problem found by an experimental coccinelle script Patch was compile tested with: multi_v7_defconfig (implies CONFIG_ARCH_STI=y) Patch is against 4.18-rc4 (localversion-next is next-20180713) drivers/clk/st/clkgen-pll.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c index cbb5184..aeb30ab 100644 --- a/drivers/clk/st/clkgen-pll.c +++ b/drivers/clk/st/clkgen-pll.c @@ -647,6 +647,7 @@ static void __iomem * __init clkgen_get_register_base( return NULL; reg = of_iomap(pnode, 0); + WARN_ON(!reg); of_node_put(pnode); return reg; -- 2.1.4