Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp1214261imm; Wed, 25 Jul 2018 13:43:07 -0700 (PDT) X-Google-Smtp-Source: AAOMgpc4CWqs/0/h1AolhIAZBlBgurZ8rvVkITazwS7jxxhRn2kUoua386+iXvzrWP9jEYDMTuOx X-Received: by 2002:a62:1219:: with SMTP id a25-v6mr23703634pfj.104.1532551387414; Wed, 25 Jul 2018 13:43:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532551387; cv=none; d=google.com; s=arc-20160816; b=FC0CDaHicil66Eqk/B5EK3vlLG3Nb1IDJkvJvUxgANm/KVegbxExADi76hsZ/Auhqn Gkk5AJIuZOqrZXn/EE769u8PRGWv2PYCv/kPeQ9DoF/Cp3Iy6mTJG0X551FsHNzhecXw QsIyChtxKPKSB0cEtly8xbJWBCgyaufOfP4o7g+84zWYqwzxOErzH+6PFtLJKt+YibBv e9G/dcGyzrjjllTLJIJzM5zA5p6MfzBb5oqLq5NrC5ZoU3nW3FhWFsqjolyH8gAFz+JI WUkfSj06l4msQoDYbRXFgmwEzc+Qu/ywAv1ztXz67sL7oklB9S86PDcfZqJ/9Y9otMa7 wpeg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:subject:user-agent:message-id :references:cc:in-reply-to:from:to:content-transfer-encoding :mime-version:dkim-signature:arc-authentication-results; bh=X5045Pj1xzWeBd2rNhYYCqEBnzbKheA/omrhKn8V+JA=; b=UbgyP0F/0SKv6WsfRjxcZVS/S+zanLen9h2qkAB5Fo2jsWE2tzAKWVBO8ShYr6wWi6 ZICLeu4FcTbwc67j00g5ebgoCiPAFnyClEWj8kU4or+d/2olhprKnEJDaPxQHLwqMkmE 79pauBt4q/1jjw1zjr0rgXadB39As6A7KpTiKsAnnKRNcA2Kp6e19G/SMwf+NGGnJgTN n6cL1R8XBWhyz/SHf+Dp3o6/OLDxrdQVV+8UM7vMdjdwIWxzUsHmfqRdVBdTSQa2pR+z g+KrBMVSamMelmNetTgvBWvTYqAM9ZAqglrZdOxUNFNMRjxWhvXp1Qnjnl/vKeTrlGXy QAyg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=wTKCEkly; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h71-v6si13660892pge.13.2018.07.25.13.42.51; Wed, 25 Jul 2018 13:43:07 -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; dkim=pass header.i=@kernel.org header.s=default header.b=wTKCEkly; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731378AbeGYVzA (ORCPT + 99 others); Wed, 25 Jul 2018 17:55:00 -0400 Received: from mail.kernel.org ([198.145.29.99]:48080 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730337AbeGYVzA (ORCPT ); Wed, 25 Jul 2018 17:55:00 -0400 Received: from localhost (unknown [104.132.1.75]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5DD0A20671; Wed, 25 Jul 2018 20:41:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1532551299; bh=PBtccEFy57UmBYlVldEJvevLkSpfzE29ezyDIabhv4A=; h=To:From:In-Reply-To:Cc:References:Subject:Date:From; b=wTKCEklyqeeXEWOb1gRTCejL5+Agn/CKjcfL8OqMSz+qQ0Nrsk/hpMvgWNlyNPrkc 0Hm04ooxK+1Q6SfLEAjemOfpkXYxq0+KFt/B9NBSqWaU9lArJI7tg6iGoL+o+JbWi4 0RkjpvISf3s36sgNJAx9aLRWWXTAvpK1q6bd+DCo= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: Michael Turquette , Nicholas Mc Guire From: Stephen Boyd In-Reply-To: <1531649904-6072-1-git-send-email-hofrat@osadl.org> Cc: Kees Cook , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Nicholas Mc Guire References: <1531649904-6072-1-git-send-email-hofrat@osadl.org> Message-ID: <153255129875.48062.4075642035553635128@swboyd.mtv.corp.google.com> User-Agent: alot/0.7 Subject: Re: [PATCH 1/2] drivers: clk: st: warn on iomap failure Date: Wed, 25 Jul 2018 13:41:38 -0700 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Nicholas Mc Guire (2018-07-15 03:18:23) > 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=3Dy) > = > 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 =3D of_iomap(pnode, 0); > + WARN_ON(!reg); > = > of_node_put(pnode); > return reg; Shouldn't the caller blow up on NULL pointer access? This patch doesn't seem useful, sorry.