From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756617Ab3EJQqX (ORCPT ); Fri, 10 May 2013 12:46:23 -0400 Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:55077 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754459Ab3EJQqW (ORCPT ); Fri, 10 May 2013 12:46:22 -0400 X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 50.131.214.131 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX18sL37a7VWtwb7DdWC3xlmc Date: Fri, 10 May 2013 09:46:18 -0700 From: Tony Lindgren To: Wolfram Sang Cc: linux-kernel@vger.kernel.org, Russell King , linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org Subject: Re: [RFC 38/42] arch/arm/plat-omap: don't check resource with devm_ioremap_resource Message-ID: <20130510164617.GK31554@atomide.com> References: <1368173847-5661-1-git-send-email-wsa@the-dreams.de> <1368173847-5661-39-git-send-email-wsa@the-dreams.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1368173847-5661-39-git-send-email-wsa@the-dreams.de> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Wolfram Sang [130510 01:23]: > devm_ioremap_resource does sanity checks on the given resource. No need to > duplicate this in the driver. > > Signed-off-by: Wolfram Sang Acked-by: Tony Lindgren > --- > arch/arm/plat-omap/dmtimer.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/arch/arm/plat-omap/dmtimer.c b/arch/arm/plat-omap/dmtimer.c > index 869254c..da1a3f7 100644 > --- a/arch/arm/plat-omap/dmtimer.c > +++ b/arch/arm/plat-omap/dmtimer.c > @@ -814,12 +814,6 @@ static int omap_dm_timer_probe(struct platform_device *pdev) > return -ENODEV; > } > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - if (unlikely(!mem)) { > - dev_err(dev, "%s: no memory resource.\n", __func__); > - return -ENODEV; > - } > - > timer = devm_kzalloc(dev, sizeof(struct omap_dm_timer), GFP_KERNEL); > if (!timer) { > dev_err(dev, "%s: memory alloc failed!\n", __func__); > @@ -827,6 +821,7 @@ static int omap_dm_timer_probe(struct platform_device *pdev) > } > > timer->fclk = ERR_PTR(-ENODEV); > + mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > timer->io_base = devm_ioremap_resource(dev, mem); > if (IS_ERR(timer->io_base)) > return PTR_ERR(timer->io_base); > -- > 1.7.10.4 >