Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754928AbbLGHoX (ORCPT ); Mon, 7 Dec 2015 02:44:23 -0500 Received: from relmlor3.renesas.com ([210.160.252.173]:22462 "EHLO relmlie2.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753016AbbLGHoQ (ORCPT ); Mon, 7 Dec 2015 02:44:16 -0500 X-IronPort-AV: E=Sophos;i="5.20,393,1444662000"; d="scan'";a="201249715" Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Message-ID: <87r3iy66za.wl%kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto Subject: [PATCH 5/8 v3] thermal: rcar: enable to use thermal-zone on DT User-Agent: Wanderlust/2.15.9 Emacs/24.3 Mule/6.0 MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") To: Simon , Zhang Rui , Eduardo Valentin , Geert Uytterhoeven CC: Magnus , , , , "devicetree@vger.kernel.org" In-Reply-To: <87y4d6672t.wl%kuninori.morimoto.gx@renesas.com> References: <87y4d6672t.wl%kuninori.morimoto.gx@renesas.com> Content-Type: text/plain; charset="US-ASCII" Date: Mon, 7 Dec 2015 07:44:09 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: HKXPR03CA0034.apcprd03.prod.outlook.com (10.141.129.24) To SIXPR06MB1024.apcprd06.prod.outlook.com (25.160.242.28) X-Microsoft-Exchange-Diagnostics: 1;SIXPR06MB1024;2:NV78to/WIHnP3KwGKb9gII1NAGbFN7+216/wYjbTnLCTgupnGDyS7np7XKAF8u+7yrPj1qwvCT0byhiIW66t3vX+whYvYhu8r7vvrPpjEg0Nm8A+2FNwp4ueaBfXrAitPEUAvx9etaxkAfiqGzBZ/w==;3:Hlo1z+2LWfn+oSGDOAdP3csy+0qfUg6nBRAe1/2OQ/6dcbeLBbSbTzWTEzB7c/IgfYtBANgMBaRI3OwpFqZSZRklwOBvn10GkfSbKG10FC4ONz/D5pyi5AOMuTH3E0/I;25:PB8H31Dbsy6rVtT/Xhka1KsauoSJOJYwU7+kCWe2baBa6y4S5JpcOhDYoZ/xPdGa+8t6bk0xn4NUDkgG4IBzZzrkJT5H4N1XiQpyqMhvbi7XdLqP8ePopn8LPQQT3i5BlMhYU0Fs2FMUcz5IRKT0Nvw+e+8DHoTnpEFi2XKTPtBCxSJBbdI0AZJC2WFgkuaqV/WZ+lOf2MLmgeeXUgB0ke7rRyQbuBn1YNiJtrunAqywJ8QOZnsh+2Q0kPnCucHfU+8faTmPI96j1OawFLS9VQ== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SIXPR06MB1024; X-Microsoft-Exchange-Diagnostics: 1;SIXPR06MB1024;20:QOusD2VV8w5HrWDGpTbuksRJoQS/Sn0FWC8sqIU7rIKSEjXknHEcr7HiWFgamf49maBh7iPaqjHkorEslYnHhYv6543ACWoAUW94LzhoTweLUZZY+/pBeTSl6XR3UnkhqzHvPSkf1kr3B0w2I8XeXW6DTvyZR/jk+Dp24OYTNDtbUq5YXGbh+5Tm50B8FIqQrrxbNQ9HkaZLsq8bpJNxQRkY0ryFptwtCB/nrkooKRoKovWr5Uy8KR4K+ZUHgCf/01kgi0S9Le4vksubiq8Cnr7Le4su4c2z9Ei5U/87AD1bztMIvX3m3CDKL8FezzF1kozQBo6kmVxVirZPodNQyleawdmvce9YgTlElJa8r2mgqaK4chiZGKTN8PF8h0EiDDI0vpR6O6NccUG7lhBfoSUh7VYn/52wHMEbVsrfeuQrLy08uY5dxm7itKszuVHtuFsjucY77WvosQ9YRNmMWEve5rsuFZbD5XRPkHoAQOVxPZQ2TAhYdAwo9+8VCrlM;4:bCaxKrLq+61NdR4utWNoYq3EZk+15THRVGnwrNkUKk0Ia4hzxb0yUmEc0ZNbRDNyS39ioQHk1fqDAdlbUdjIOArww0KxLgkFRvqg2YVbXqttJ6ST4DW6jZ3pqGZrwKdim1pdDXjJUHD+NKDU7a+ekfiQujWW9ycw8fDNGr/i4vQ+X6nD2lyMafV/K7T//GpRNHqD67JZjkvcFT5m/G+m5qc88lgfSgW4mxRnqvdODX7sY0nBd/I/vOjmVEZG4CWvUqbhFLGy+XNkowx1MYqGPzkQJg8NL1wwb/CnP/276BJ4qbsY6l6asc32yGU00AEQZoqy3vENMRvAJme+GVnF97B3MD4jk83ztr4/6u93jb8hszglQV9fuGskQzFpnQDwOrERQDnXHvOUJraEd21ZLwATTWfBb3L2ZuEDfCYOHO4NxErAigRDD7dSX5nviOaf X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(85106069007906); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(3002001)(10201501046);SRVR:SIXPR06MB1024;BCL:0;PCL:0;RULEID:;SRVR:SIXPR06MB1024; X-Forefront-PRVS: 078310077C X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(6009001)(199003)(189002)(1096002)(5004730100002)(83506001)(97736004)(5001770100001)(4001350100001)(19580405001)(6116002)(86362001)(19580395003)(50466002)(42186005)(50986999)(76176999)(69596002)(81156007)(101416001)(77096005)(53416004)(3846002)(5008740100001)(33646002)(46406003)(66066001)(40100003)(2950100001)(23726003)(47776003)(36756003)(106356001)(54356999)(5001960100002)(229853001)(586003)(87976001)(189998001)(105586002)(92566002)(122386002);DIR:OUT;SFP:1102;SCL:1;SRVR:SIXPR06MB1024;H:morimoto-PC.renesas.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;SIXPR06MB1024;23:QX00ZA3g8SoLPlkfOY6Mipc6M4cm5twOBDSDV7AZq?= =?us-ascii?Q?Gy/OrBK2+pfTa7Zu5NzlsAKcU1RYfUXcsWoxWTqoA2Dalg1DaNhqxh/3JZlq?= =?us-ascii?Q?jc8/ioqAtxk7qVj5LaQ5p/TDwCfqvZyfvAQnmGXlVbMS9YdOCnrRHmnizrUr?= =?us-ascii?Q?9v/ktLLn+Svzx1qiv3+ECLEgutVZuZI3fJJc7bXNE5GqDwgCDTw0hHZ1cjQE?= =?us-ascii?Q?SpCbIcumgMNSdSrcEosddYMQXsUnQLo2PV43Q78An4o/dttGUfIqnfv/6/O7?= =?us-ascii?Q?GpSgwR47X4LMWbfbxF5JfxbAjMtHouPIDPpEYQjtnjjznhSwQ3NKCfbPVmOC?= =?us-ascii?Q?VEESBWvKcJdtqVeaLIjD3fc207c/ujNliRjTAjsvL7QhCKdk82pP/7mz4Eyf?= =?us-ascii?Q?Wa9JkU99U6RPLxHKW8bY6fvQo9qsSFQFGoLNOf9HP1a+5F6lbIbUJC88WEVR?= =?us-ascii?Q?D7bSIZ1WtWgcg54yqXFRyw0oY622djdh8N1vS92XjqeJOcPlZ5G9/aPzFthZ?= =?us-ascii?Q?bX8gxFAIqCXxtiHa69VMF0/IalJDv9UqjfRVUIOwh83Fd1f1dLr+k2dW6Zve?= =?us-ascii?Q?SedpIxkOLXKskZ3sIJKAvalt4k+aq8NsJOcXA25jdR9FRNJ0PrU8aQmy1h9D?= =?us-ascii?Q?yOFspCqR+M5tud8iSX8ldOepUpeXlj4+mGtZr4hhfA8JE8K9XCi53lJXUM+y?= =?us-ascii?Q?j9H8J2sKVUqAKBXpe7Sa4zuzudxlE1ALItrDdjXb3+fWWOdrZ793yj2HY1d4?= =?us-ascii?Q?c7HiffcresVKSxpfXMk7bFP8Xt4R0s7SQ3HyJK2j61IAcHHxAkq64dAsqYva?= =?us-ascii?Q?6ufaDi3jqrTGQhDXTwGytk0CuF1SFMH3xmfigT60gF5IKvPZqSehZIUXnA7Y?= =?us-ascii?Q?j/cJ8J4p+KcQi70QfX3vfYBa0/yuS/9lmw0w9oMF5+I+3SVFrxKDelXG/x/z?= =?us-ascii?Q?I2AOP5xQSPLkh6ytEUzt7bz3UyTDD7sDOKI5w0T4O4DWshqJLUrwve6a8Il0?= =?us-ascii?Q?884gy2svTh5QpcVnchna3UpL3JzZY7JoGIv4Hc2eCfsqtb78NzLIpFCA7pZt?= =?us-ascii?Q?lbZKLEhatHG4puknV3k+UZcfpIxkSNSFbiysUBOYFXGDZINuw+OSmJXs0W6y?= =?us-ascii?Q?2cs7G5g8qg=3D?= X-Microsoft-Exchange-Diagnostics: 1;SIXPR06MB1024;5:GE5ycAHCq+zWmSrRg9rewKr5hPJrKWK1O+XBKiamXn2V/6E9gMbk7DqBdnyN8Ku/FqdfPuexZP6koJJqx0Gu9iiPj1qV0S6AMd/h6JUHdriewc/04DOS/k/pkusRLCCRO2N3Xv6793I7y1U56Yn+yQ==;24:uGIVoE+9prmg5AEGK+c2WTkhn5jzAhlGUDUd0NytYUUCDnpzDPE/ZbBSqzAUE8Tl3sMiA0Lf003v9caIs4U3Afk60pjxzO8jssjmd9/EVvA=;20:+xSsS5OxJHSF/HFHWRPiVg3GlqIZCRELKPH899iIz/AUObylsu9Z28jMBqaTw2ariYsavn/80cxozQrjezC+v2A7O5KjqX19+u7xvjsF3pF5gVOP+OFy6WlKBSH45UmSxnhZp5AwUzKCarNbBee6xGX9cxgb1HoFKXxzZX20XZA= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2015 07:44:09.2545 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SIXPR06MB1024 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5894 Lines: 190 From: Kuninori Morimoto This patch enables to use thermal-zone on DT if it was call as "renesas,rcar-thermal-gen2". Previous style is still supported by "renesas,rcar-thermal". Signed-off-by: Kuninori Morimoto --- v2 -> v3 - compatible "renesas,rcar-thermal-gen2" -> "renesas,rcar-gen2-thermal" .../devicetree/bindings/thermal/rcar-thermal.txt | 37 +++++++++++++++++- drivers/thermal/rcar_thermal.c | 44 +++++++++++++++++++--- 2 files changed, 74 insertions(+), 7 deletions(-) diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt index 332e625..601a54d 100644 --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt @@ -1,8 +1,9 @@ * Renesas R-Car Thermal Required properties: -- compatible : "renesas,thermal-", "renesas,rcar-thermal" - as fallback. +- compatible : "renesas,thermal-", + "renesas,rcar-gen2-thermal" (with thermal-zone) or + "renesas,rcar-thermal" (without thermal-zone) as fallback. Examples with soctypes are: - "renesas,thermal-r8a73a4" (R-Mobile APE6) - "renesas,thermal-r8a7779" (R-Car H1) @@ -36,3 +37,35 @@ thermal@e61f0000 { 0xe61f0300 0x38>; interrupts = <0 69 IRQ_TYPE_LEVEL_HIGH>; }; + +Example (with thermal-zone): + +thermal-zones { + cpu_thermal: cpu-thermal { + polling-delay-passive = <1000>; + polling-delay = <5000>; + + thermal-sensors = <&thermal>; + + trips { + cpu-crit { + temperature = <1150000>; + hysteresis = <0>; + type = "critical"; + }; + }; + cooling-maps { + }; + }; +}; + +thermal: thermal@e61f0000 { + compatible = "renesas,thermal-r8a7790", + "renesas,rcar-gen2-thermal", + "renesas,rcar-thermal"; + reg = <0 0xe61f0000 0 0x14>, <0 0xe61f0100 0 0x38>; + interrupts = <0 69 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&mstp5_clks R8A7790_CLK_THERMAL>; + power-domains = <&cpg_clocks>; + #thermal-sensor-cells = <0>; +}; diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c index 52493b4..a8c88f4 100644 --- a/drivers/thermal/rcar_thermal.c +++ b/drivers/thermal/rcar_thermal.c @@ -23,6 +23,7 @@ #include #include #include +#include #include #include #include @@ -81,8 +82,10 @@ struct rcar_thermal_priv { # define rcar_force_update_temp(priv) 0 #endif +#define USE_OF_THERMAL 1 static const struct of_device_id rcar_thermal_dt_ids[] = { { .compatible = "renesas,rcar-thermal", }, + { .compatible = "renesas,rcar-gen2-thermal", .data = (void *)USE_OF_THERMAL }, {}, }; MODULE_DEVICE_TABLE(of, rcar_thermal_dt_ids); @@ -206,9 +209,8 @@ err_out_unlock: return ret; } -static int rcar_thermal_get_temp(struct thermal_zone_device *zone, int *temp) +static int rcar_thermal_get_current_temp(struct rcar_thermal_priv *priv, int *temp) { - struct rcar_thermal_priv *priv = rcar_zone_to_priv(zone); int tmp; if (!rcar_has_irq_support(priv) || rcar_force_update_temp(priv)) { @@ -233,6 +235,20 @@ static int rcar_thermal_get_temp(struct thermal_zone_device *zone, int *temp) return 0; } +static int rcar_thermal_of_get_temp(void *data, int *temp) +{ + struct rcar_thermal_priv *priv = data; + + return rcar_thermal_get_current_temp(priv, temp); +} + +static int rcar_thermal_get_temp(struct thermal_zone_device *zone, int *temp) +{ + struct rcar_thermal_priv *priv = rcar_zone_to_priv(zone); + + return rcar_thermal_get_current_temp(priv, temp); +} + static int rcar_thermal_get_trip_type(struct thermal_zone_device *zone, int trip, enum thermal_trip_type *type) { @@ -289,6 +305,10 @@ static int rcar_thermal_notify(struct thermal_zone_device *zone, return 0; } +static const struct thermal_zone_of_device_ops rcar_thermal_zone_of_ops = { + .get_temp = rcar_thermal_of_get_temp, +}; + static struct thermal_zone_device_ops rcar_thermal_zone_ops = { .get_temp = rcar_thermal_get_temp, .get_trip_type = rcar_thermal_get_trip_type, @@ -325,14 +345,20 @@ static void rcar_thermal_work(struct work_struct *work) priv = container_of(work, struct rcar_thermal_priv, work.work); - rcar_thermal_get_temp(priv->zone, &cctemp); + ret = rcar_thermal_get_current_temp(priv, &cctemp); + if (ret < 0) + return; + ret = rcar_thermal_update_temp(priv); if (ret < 0) return; rcar_thermal_irq_enable(priv); - rcar_thermal_get_temp(priv->zone, &nctemp); + ret = rcar_thermal_get_current_temp(priv, &nctemp); + if (ret < 0) + return; + if (nctemp != cctemp) thermal_zone_device_update(priv->zone); } @@ -410,6 +436,8 @@ static int rcar_thermal_probe(struct platform_device *pdev) struct rcar_thermal_priv *priv; struct device *dev = &pdev->dev; struct resource *res, *irq; + const struct of_device_id *of_id = of_match_device(rcar_thermal_dt_ids, dev); + unsigned long of_data = (unsigned long)of_id->data; int mres = 0; int i; int ret = -ENODEV; @@ -470,7 +498,13 @@ static int rcar_thermal_probe(struct platform_device *pdev) if (ret < 0) goto error_unregister; - priv->zone = thermal_zone_device_register("rcar_thermal", + if (of_data == USE_OF_THERMAL) + priv->zone = thermal_zone_of_sensor_register( + dev, i, priv, + &rcar_thermal_zone_of_ops); + else + priv->zone = thermal_zone_device_register( + "rcar_thermal", 1, 0, priv, &rcar_thermal_zone_ops, NULL, 0, idle); -- 1.9.1 -- 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/