Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760291Ab3HOHBJ (ORCPT ); Thu, 15 Aug 2013 03:01:09 -0400 Received: from mga03.intel.com ([143.182.124.21]:5217 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757891Ab3HOHBH (ORCPT ); Thu, 15 Aug 2013 03:01:07 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.89,883,1367996400"; d="scan'208";a="346662905" Message-ID: <1376550128.2335.110.camel@rzhang-lenovo> Subject: Re: [PATCH] thermal: exynos_tmu: fix wrong error check for mapped memory From: Zhang Rui To: Naveen Krishna Chatradhi Cc: linux-pm@vger.kernel.org, eduardo.valentin@ti.com, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, amit.daniel@samsung.com, kgene.kim@samsung.com Date: Thu, 15 Aug 2013 15:02:08 +0800 In-Reply-To: <1375864269-9197-1-git-send-email-ch.naveen@samsung.com> References: <1375864269-9197-1-git-send-email-ch.naveen@samsung.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4-0ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1220 Lines: 35 On δΈ‰, 2013-08-07 at 14:01 +0530, Naveen Krishna Chatradhi wrote: > The error check is checking for a "base" mapped memory base > instead of "base_common". Fixing the same. > > Signed-off-by: Naveen Krishna Chatradhi applied. Thanks! BTW, can you rebase the other three exynos patches you submitted? thanks, rui > --- > drivers/thermal/samsung/exynos_tmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index ec01dfe..a033dbb 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -592,7 +592,7 @@ static int exynos_map_dt_data(struct platform_device *pdev) > > data->base_common = devm_ioremap(&pdev->dev, res.start, > resource_size(&res)); > - if (!data->base) { > + if (!data->base_common) { > dev_err(&pdev->dev, "Failed to ioremap memory\n"); > return -ENOMEM; > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/