From patchwork Thu May 2 05:37:41 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amit Kachhap X-Patchwork-Id: 2510041 Return-Path: X-Original-To: patchwork-linux-samsung-soc@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 2F0C83FCA5 for ; Thu, 2 May 2013 05:39:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751934Ab3EBFjb (ORCPT ); Thu, 2 May 2013 01:39:31 -0400 Received: from mail-pb0-f45.google.com ([209.85.160.45]:45990 "EHLO mail-pb0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751397Ab3EBFj3 (ORCPT ); Thu, 2 May 2013 01:39:29 -0400 Received: by mail-pb0-f45.google.com with SMTP id ro12so155023pbb.4 for ; Wed, 01 May 2013 22:39:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:sender:from:to:cc:subject:date:message-id:x-mailer; bh=jzXYzolHAFWmEbFA76zQTb8RcWuAIiSwiAZ7Gv9OnEM=; b=hO9v4YUa7nv6px8PGt4aRaA7VLzTJoHnk1R/PNG4MP2AnFx9AmZADVhSihNsdhKxgc qmBEr0i0DqxItmGOyAtqIR3Vbw3kbqFu0US+BE5pJCAaYTMHH4YbnVtJhXC3ti6Wl9ol 4UR2NoQad9DRdKIGSlBebaaESfURbQbJ88vzDpXuxMNNmq0LOhIU31eaCL6b86xye347 8aASUIkde4arqpg2sKLbnRAsL1AMOaUVm6Iq4CQEpbnXXY7gB0dG8OlwpDmrwSdjJ+Wq 1+IE3x+4ofmYvsO4t5HOukqHOZLEw6yUIZ/kcFA78hspeYELHEa1c4aixo4Tca7lHoww aUQw== X-Received: by 10.68.175.97 with SMTP id bz1mr7393643pbc.37.1367473169179; Wed, 01 May 2013 22:39:29 -0700 (PDT) Received: from localhost.localdomain ([115.113.119.130]) by mx.google.com with ESMTPSA id dr6sm6767004pac.11.2013.05.01.22.37.58 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Wed, 01 May 2013 22:39:27 -0700 (PDT) From: Amit Daniel Kachhap To: linux-pm@vger.kernel.org Cc: Zhang Rui , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, amit.kachhap@gmail.com, Kukjin Kim , Lukasz Majewski , Eduardo Valentin Subject: [PATCH] thermal: exynos: Support for TMU regulator defined at device tree Date: Thu, 2 May 2013 11:07:41 +0530 Message-Id: <1367473061-28863-1-git-send-email-amit.daniel@samsung.com> X-Mailer: git-send-email 1.7.1 Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org TMU probe function now checks for a device tree defined regulator. For compatibility reasons it is allowed to probe driver even without this regulator defined. Signed-off-by: Lukasz Majewski Signed-off-by: Kyungmin Park Signed-off-by: Amit Daniel Kachhap --- This patch is repost of the patch posted by Lukasz Majewski (https://patchwork.kernel.org/patch/2488211/). I have rebased this patch on top of my TMU re-structured patch series (http://lwn.net/Articles/548634/). Although I thought of handling regulator as one type of feature (newly added) but could not do so as regulator is a board/platform property and not SOC property so leaving the device tree to define and handle it. .../devicetree/bindings/thermal/exynos-thermal.txt | 4 ++++ drivers/thermal/samsung/exynos_tmu.c | 17 +++++++++++++++++ 2 files changed, 21 insertions(+), 0 deletions(-) diff --git a/Documentation/devicetree/bindings/thermal/exynos-thermal.txt b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt index 970eeba..ff62f7a 100644 --- a/Documentation/devicetree/bindings/thermal/exynos-thermal.txt +++ b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt @@ -14,6 +14,9 @@ - interrupts : Should contain interrupt for thermal system - clocks : The main clock for TMU device - clock-names : Thermal system clock name +- vtmu-supply: This entry is optional and provides the regulator node supplying + voltage to TMU. If needed this entry can be placed inside + board/platform specific dts file. Example 1): @@ -25,6 +28,7 @@ Example 1): clocks = <&clock 383>; clock-names = "tmu_apbif"; status = "disabled"; + vtmu-supply = <&tmu_regulator_node>; }; Example 2): diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index 72446c9..45b50c1 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -32,6 +32,7 @@ #include #include #include +#include #include #include #include "exynos_thermal_common.h" @@ -52,6 +53,7 @@ * @clk: pointer to the clock structure. * @temp_error1: fused value of the first point trim. * @temp_error2: fused value of the second point trim. + * @regulator: pointer to the TMU regulator structure. * @reg_conf: pointer to structure to register with core thermal. */ struct exynos_tmu_data { @@ -65,6 +67,7 @@ struct exynos_tmu_data { struct mutex lock; struct clk *clk; u8 temp_error1, temp_error2; + struct regulator *regulator; struct thermal_sensor_conf *reg_conf; }; @@ -501,10 +504,21 @@ static int exynos_map_dt_data(struct platform_device *pdev) struct exynos_tmu_data *data = platform_get_drvdata(pdev); struct exynos_tmu_platform_data *pdata = data->pdata; struct resource res; + int ret; if (!data) return -ENODEV; + /* Try enabling the regulator if found */ + data->regulator = devm_regulator_get(&pdev->dev, "vtmu"); + if (!IS_ERR(data->regulator)) { + ret = regulator_enable(data->regulator); + if (ret) { + dev_err(&pdev->dev, "failed to enable vtmu\n"); + return ret; + } + } + data->id = of_alias_get_id(pdev->dev.of_node, "tmuctrl"); if (data->id < 0) data->id = 0; @@ -669,6 +683,9 @@ static int exynos_tmu_remove(struct platform_device *pdev) clk_unprepare(data->clk); + if (data->regulator) + regulator_disable(data->regulator); + platform_set_drvdata(pdev, NULL); return 0;